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@xxxxxxxxx <mailto:henk.vergonet@xxxxxxxxx>>
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
