Re: [PATCH 0/9] iio: adc: at91_adc: cleanup DT bindings

2020-11-15 Thread Ludovic Desroches
On Fri, Nov 13, 2020 at 10:26:41PM +0100, Alexandre Belloni wrote:
> EXTERNAL EMAIL: Do not click links or open attachments unless you know the 
> content is safe
> 
> Hello,
> 
> This series cleans up the at91_adc devicetree bindings. This mainly
> moves back the resolution options and names and the triggers description
> back in the driver.
> 
> There are also other cleanups, like removing platform data support, this
> was pending for a while.
>

Nice cleanup of this old and obscur binding.

Reviewed-by: Ludovic Desroches 

Thanks.

> Alexandre Belloni (8):
>   iio: adc: at91_adc: remove platform data
>   iio: adc: at91_adc: rework resolution selection
>   iio: adc: at91_adc: rework trigger definition
>   iio: adc: at91_adc: merge at91_adc_probe_dt back in at91_adc_probe
>   iio: adc: at91_adc: remove forward declaration
>   iio: adc: at91_adc: use devm_input_allocate_device
>   ARM: dts: at91: sama5d3: use proper ADC compatible
>   ARM: dts: at91: remove deprecated ADC properties
> 
> Jonathan Cameron (1):
>   dt-bindings:iio:adc:atmel,sama9260-adc: conversion to yaml from
> at91_adc.txt
> 
>  .../devicetree/bindings/iio/adc/at91_adc.txt  |  83 
>  .../bindings/iio/adc/atmel,sama9260-adc.yaml  | 121 ++
>  arch/arm/boot/dts/at91sam9260.dtsi|  25 --
>  arch/arm/boot/dts/at91sam9g45.dtsi|  27 --
>  arch/arm/boot/dts/at91sam9rl.dtsi |  25 --
>  arch/arm/boot/dts/at91sam9x5.dtsi |  28 --
>  arch/arm/boot/dts/sama5d3.dtsi|  26 +-
>  arch/arm/boot/dts/sama5d4.dtsi|  22 -
>  drivers/iio/adc/at91_adc.c| 377 +++---
>  include/linux/platform_data/at91_adc.h|  49 ---
>  10 files changed, 259 insertions(+), 524 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/iio/adc/at91_adc.txt
>  create mode 100644 
> Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
>  delete mode 100644 include/linux/platform_data/at91_adc.h
> 
> --
> 2.28.0


Re: [PATCH 0/9] iio: adc: at91_adc: cleanup DT bindings

2020-11-14 Thread Jonathan Cameron
On Fri, 13 Nov 2020 22:26:41 +0100
Alexandre Belloni  wrote:

> Hello,
> 
> This series cleans up the at91_adc devicetree bindings. This mainly
> moves back the resolution options and names and the triggers description
> back in the driver.
> 
> There are also other cleanups, like removing platform data support, this
> was pending for a while.

I'm fine with everything up to patch 7.  Given it touches DT I'll leave
a bit of time for Rob to take a quick look (can't see him objecting to the
removal of sections of the binding though!)

If you don't mind me dropping 7 I can pick 1-6 up in a week or two.
Thanks,

Jonathan

> 
> Alexandre Belloni (8):
>   iio: adc: at91_adc: remove platform data
>   iio: adc: at91_adc: rework resolution selection
>   iio: adc: at91_adc: rework trigger definition
>   iio: adc: at91_adc: merge at91_adc_probe_dt back in at91_adc_probe
>   iio: adc: at91_adc: remove forward declaration
>   iio: adc: at91_adc: use devm_input_allocate_device
>   ARM: dts: at91: sama5d3: use proper ADC compatible
>   ARM: dts: at91: remove deprecated ADC properties
> 
> Jonathan Cameron (1):
>   dt-bindings:iio:adc:atmel,sama9260-adc: conversion to yaml from
> at91_adc.txt
> 
>  .../devicetree/bindings/iio/adc/at91_adc.txt  |  83 
>  .../bindings/iio/adc/atmel,sama9260-adc.yaml  | 121 ++
>  arch/arm/boot/dts/at91sam9260.dtsi|  25 --
>  arch/arm/boot/dts/at91sam9g45.dtsi|  27 --
>  arch/arm/boot/dts/at91sam9rl.dtsi |  25 --
>  arch/arm/boot/dts/at91sam9x5.dtsi |  28 --
>  arch/arm/boot/dts/sama5d3.dtsi|  26 +-
>  arch/arm/boot/dts/sama5d4.dtsi|  22 -
>  drivers/iio/adc/at91_adc.c| 377 +++---
>  include/linux/platform_data/at91_adc.h|  49 ---
>  10 files changed, 259 insertions(+), 524 deletions(-)
>  delete mode 100644 Documentation/devicetree/bindings/iio/adc/at91_adc.txt
>  create mode 100644 
> Documentation/devicetree/bindings/iio/adc/atmel,sama9260-adc.yaml
>  delete mode 100644 include/linux/platform_data/at91_adc.h
>