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
