Re: [PATCH v3 0/4] iio/adc: ingenic: add support for the JZ4760(B) Socs to the ingenic sadc driver

From: Jonathan Cameron
Date: Sun Jul 25 2021 - 11:19:33 EST


On Sat, 24 Jul 2021 21:04:44 +0200
Christophe Branchereau <cbranchereau@xxxxxxxxx> wrote:

> This is a set of patches to add support to the JZ4760(B) socs found in numerous gaming handhelds and some
> mp3 players to the ingenic-sadc driver.
>
> Changelog for this v3:
> - Minor formatting change in ingenic-adc.c to remain within 80 lines
> - Correctly set the ingenic,use-internal-divider property in the Documentation so it cannot be used on other
> socs than the jz4760b, and modify the description as requested.
>
> Thanks to Paul and Jonathan for their help

You are welcome.

Other than Paul's question on the missing entry in patch 4, these look good to me.
We should leave a bit of time for others to review but otherwise once that's
resolved I'll pick them up.

Thanks,

Jonathan

>
> Christophe Branchereau (5):
> iio/adc: ingenic: rename has_aux2 to has_aux_md
> dt-bindings: iio/adc: add an INGENIC_ADC_AUX0 entry
> iio/adc: ingenic: add JZ4760 support to the sadc driver
> iio/adc: ingenic: add JZ4760B support to the sadc driver
> dt-bindings: iio/adc: ingenic: add the JZ4760(B) socs to the sadc
> Documentation
>
> .../bindings/iio/adc/ingenic,adc.yaml | 19 ++++
> drivers/iio/adc/ingenic-adc.c | 101 ++++++++++++++++--
> include/dt-bindings/iio/adc/ingenic,adc.h | 1 +
> 3 files changed, 112 insertions(+), 9 deletions(-)
>