Re: HWMON: (nct6775) Correct number of inputs on NCT6796D

On 7/5/21 5:30 AM, Henk wrote:

Noticed in the documentation that the NCT6796D actually has 16 analog voltage inputs, where only 15 are configured.

See: https://www.nuvoton.com/resource-files/NCT6796D_Datasheet_V0_6.pdf <https://www.nuvoton.com/resource-files/NCT6796D_Datasheet_V0_6.pdf> page 59-60:

8.6.2.1. Voltage Reading
NCT6796D has 16 voltage reading:
.....
and the missing one in the driver:
VIN9: Voltage reading Bank 4, Index 8F

This patch adds support for in15 on the NCT6796D.

Signed-off-by: Henk Vergonet <henk.vergonet@gmail.com <mailto:henk.vergonet@gmail.com>>

Please read and follow Documentation/process/submitting-patches.rst
when submitting patches.

As for the patch itself, it doesn't define the limit register addresses
in NCT6775_REG_IN_MAX[] and NCT6775_REG_IN_MIN[], and it doesn't define
the register address itself in NCT6779_REG_IN[]. That means it won't work.

Guenter