Re: [PATCH 4/4] iio: dac: ad5686: update device list and file header comments

[Jonathan Cameron <jonathan.cameron@oss.qualcomm.com>]

On Sun, 19 Jul 2026 10:52:00 +0100
Rodrigo Alencar via B4 Relay <devnull+rodrigo.alencar.analog.com@kernel.org> wrote:

> From: Rodrigo Alencar <rodrigo.alencar@analog.com>
> 
> Update device list entries in Kconfig and remove device description from
> file headers. One can rely on the ID table(s) to check for supported
> devices. Adjust comment header accordingly and update Copyright notice in
> comment file headers with the current year.
> 
> Signed-off-by: Rodrigo Alencar <rodrigo.alencar@analog.com>
> ---

Main comments here are about the patch break up.  I think it needs
just enough modification for a v2 to be needed.

>  drivers/iio/dac/Kconfig      | 31 ++++++++++++++++++++++++-------
>  drivers/iio/dac/ad5686-spi.c |  7 ++-----
>  drivers/iio/dac/ad5686.c     |  4 ++--
>  drivers/iio/dac/ad5696-i2c.c |  6 ++----
>  4 files changed, 30 insertions(+), 18 deletions(-)
> 
> diff --git a/drivers/iio/dac/Kconfig b/drivers/iio/dac/Kconfig
> index d6d560c09e25..ba2065241af3 100644
> --- a/drivers/iio/dac/Kconfig
> +++ b/drivers/iio/dac/Kconfig
> @@ -251,9 +251,18 @@ config AD5686_SPI
>  	depends on SPI
>  	select AD5686
>  	help
> -	  Say yes here to build support for Analog Devices AD5672R, AD5674R,
> -	  AD5676, AD5676R, AD5679R, AD5684, AD5684R, AD5684R, AD5685R, AD5686,
> -	  AD5686R Voltage Output Digital to Analog Converter.

Given this is adding entrees that were in the previous patch, I'd have
preferred to see the more complex dance of a precursor patch before that
which did the reformat (and dropped the stuff form the c files)

That would then be followed the new stuff being added as part of the previous
patch.

Whilst the ends result is the same I think it is worth doing a v2 just
to clean that break down of patches up.

> +	  Say yes here to build support for Analog Devices Voltage Output
> +	  Digital to Analog Converters:
> +	  - Single-channel:
> +	    AD5310R, AD5681R, AD5682R, AD5683R, AD5683R

Eagle eyed Sashiko spotted last entry is a duplicate.

> +	  - Dual-channel:
> +	    AD5313R, AD5687, AD5687R, AD5689, AD5689R
> +	  - Quad-channel:
> +	    AD5317R, AD5684, AD5684R, AD5685R, AD5686, AD5686R
> +	  - 8-channel:
> +	    AD5672R, AD5676, AD5676R
> +	  - 16-channel:
> +	    AD5674, AD5674R, AD5679, AD5679R
>  
>  	  To compile this driver as a module, choose M here: the
>  	  module will be called ad5686.
> @@ -263,10 +272,18 @@ config AD5696_I2C
>  	depends on I2C
>  	select AD5686
>  	help
> -	  Say yes here to build support for Analog Devices AD5311R, AD5337,
> -	  AD5338R, AD5671R, AD5673R, AD5675R, AD5677R, AD5691R, AD5692R, AD5693,
> -	  AD5693R, AD5694, AD5694R, AD5695R, AD5696, and AD5696R Digital to
> -	  Analog converters.
> +	  Say yes here to build support for Analog Devices Voltage Output
> +	  Digital to Analog Converters:
> +	  - Single-channel:
> +	    AD5311R, AD5691R, AD5692R, AD5693, AD5693R
> +	  - Dual-channel:
> +	    AD5338R, AD5697R
> +	  - Quad-channel:
> +	    AD5316R, AD5694, AD5694R, AD5695R, AD5696, AD5696R
> +	  - 8-channel:
> +	    AD5671R, AD5675, AD5675R
> +	  - 16-channel:
> +	    AD5673R, AD5677R
>  
>  	  To compile this driver as a module, choose M here: the module will be
>  	  called ad5696.
> diff --git a/drivers/iio/dac/ad5686-spi.c b/drivers/iio/dac/ad5686-spi.c
> index 003541edb742..b4531b49cb22 100644
> --- a/drivers/iio/dac/ad5686-spi.c
> +++ b/drivers/iio/dac/ad5686-spi.c
> @@ -1,11 +1,8 @@
>  // SPDX-License-Identifier: GPL-2.0
>  /*
> - * AD5672R, AD5674R, AD5676, AD5676R, AD5679R,
> - * AD5681R, AD5682R, AD5683, AD5683R, AD5684,
> - * AD5684R, AD5685R, AD5686, AD5686R
> - * Digital to analog converters driver
> + * SPI driver for AD5686 and similar Digital to Analog Converters
>   *
> - * Copyright 2018 Analog Devices Inc.
> + * Copyright 2018-2026 Analog Devices Inc.

That definitely belongs in the patch that made a real change, not one
that is just messing with comments.

>   */
>  
>  #include <linux/array_size.h>