Re: [PATCH v4 2/2] iio: adc: add support for Nuvoton NCT7201

[Paul Menzel]

Dear Eason,


Thank you for your patch.

Am 21.02.25 um 10:09 schrieb Eason Yang:
> Add Nuvoton NCT7201/NCT7202 system voltage monitor 12-bit ADC driver
>
> NCT7201/NCT7202 supports up to 12 analog voltage monitor inputs and up to
> 4 SMBus addresses by ADDR pin. Meanwhile, ALERT# hardware event pins for
> independent alarm signals, and the all threshold values could be set for

… and all the threshold values …

> system protection without any timing delay. It also supports reset input
> RSTIN# to recover system from a fault condition.
>
> Currently, only single-edge mode conversion and threshold events support.

… are supported.

It’d be great if you added a datasheet name and revision, and, if 
publicly available, a URL.

> Signed-off-by: Eason Yang <j2anfernee@xxxxxxxxx>
> ---
>   MAINTAINERS               |   1 +
>   drivers/iio/adc/Kconfig   |  11 +
>   drivers/iio/adc/Makefile  |   1 +
>   drivers/iio/adc/nct7201.c | 487 ++++++++++++++++++++++++++++++++++++++
>   4 files changed, 500 insertions(+)
>   create mode 100644 drivers/iio/adc/nct7201.c

[…]


Kind regards,

Paul