lkml.org 
[lkml]   [2024]   [Feb]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v3] Watchdog: New module for ITE 5632 watchdog
From
On 2/29/24 10:12, Mark Pearson wrote:
[ ... ]

>> Is it necessary to treat this differently than, say,
>> drivers/hwmon/it87.c and drivers/watchdog/it87_wdt.c ?
>> Those work together nicely because most of the address space is
>> separate; access through Super-IO registers
>> is limited enough that it can be shared by using request_muxed_region()
>> in both drivers.
>>
>> I'll have to look deeper into NCT6692D (and NCT6686, for that matter),
>> to see if those require mfd drivers.
>> I'll also need to get the datasheets for those chips and confirm that
>> they really need different watchdog
>> drivers to start with.
>>
> Ack - I'll look at those and see. Quick look at the watchdog driver and it looks possible but I need to check the details more carefully.
> Afraid I can't share the datasheet as I don't have permission to release it. How much of a blocker is that for you?
>

I do have the datasheet for the NCT6683 EC space. I'll need to check if
it matches the code submitted for the NCT6686 watchdog.

For NCT6692D, I asked Nuvoton if they can share the datasheet.
Given that it is some kind of security controller chip, it may well be
that Lenovo has an NDA with Nuvoton which prevents them from sharing
the datasheet. We'll see.

Undocumented ITE chips are simply not supportable. No matter what, someone
will have to step up as maintainer for those chips.

Guenter


\
 
 \ /
  Last update: 2024-05-27 15:28    [W:0.063 / U:1.388 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site