Re: [PATCH v2 2/3] iio: dac: Add AD5529R DAC driver support

[David Lechner <dlechner@baylibre.com>]

On 5/8/26 6:55 AM, Janani Sunil wrote:
> Add support for AD5529R 16-channel, 12/16 bit Digital to Analog Converter
> 

...

> +#define AD5529R_DAC_CHANNEL(chan, bits) {			\
> +	.type = IIO_VOLTAGE,					\
> +	.indexed = 1,						\
> +	.output = 1,						\
> +	.channel = (chan),					\
> +	.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |		\
> +			      BIT(IIO_CHAN_INFO_SCALE),		\
> +	.scan_type = {						\

> +		.sign = 'u',					\

This field has a new name `.format`.

> +		.realbits = (bits),				\
> +		.storagebits = 16,				\
> +	},							\
> +}
> +static struct regmap *ad5529r_get_regmap(struct ad5529r_state *st, unsigned int reg)
> +{
> +	if (reg <= AD5529R_8BIT_REG_MAX)
> +		return st->regmap_8bit;
> +
> +	return st->regmap_16bit;
> +}

Another way we have done this is make custom read/write functions for the
regmap itself so that we don't have to have two regmaps.

> +
> +static int ad5529r_debugfs_reg_read(struct ad5529r_state *st, unsigned int reg,
> +				    unsigned int *val)
> +{
> +	return regmap_read(ad5529r_get_regmap(st, reg), reg, val);
> +}
> +
> +static int ad5529r_debugfs_reg_write(struct ad5529r_state *st, unsigned int reg,
> +				     unsigned int val)
> +{
> +	return regmap_write(ad5529r_get_regmap(st, reg), reg, val);
> +}

Would be more logical to move these closer to the struct that
references them.

(I snipped a bunch of functions here)

> +
> +static int ad5529r_reg_access(struct iio_dev *indio_dev,
> +			      unsigned int reg,
> +			      unsigned int writeval,
> +			      unsigned int *readval)
> +{
> +	struct ad5529r_state *st = iio_priv(indio_dev);
> +
> +	if (!readval)

Might as well swap these and avoid the !.

> +		return ad5529r_debugfs_reg_write(st, reg, writeval);
> +
> +	return ad5529r_debugfs_reg_read(st, reg, readval);
> +}
> +