[PATCH v5 11/12] iio: dac: ad5686: cleanup doc header of local structs

From: Rodrigo Alencar via B4 Relay

Date: Fri May 01 2026 - 05:18:37 EST


From: Rodrigo Alencar <rodrigo.alencar@xxxxxxxxxx>

Review documentation comment header for ad5686_chip_info and ad5686_state.
Update variable names and description and remove unnecessary blank line
between comment and struct declaration.

Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>
Signed-off-by: Rodrigo Alencar <rodrigo.alencar@xxxxxxxxxx>
---
drivers/iio/dac/ad5686.h | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/drivers/iio/dac/ad5686.h b/drivers/iio/dac/ad5686.h
index dc1961b8c15e..4c38dd43d4bb 100644
--- a/drivers/iio/dac/ad5686.h
+++ b/drivers/iio/dac/ad5686.h
@@ -71,12 +71,11 @@ typedef int (*ad5686_read_func)(struct ad5686_state *st, u8 addr);

/**
* struct ad5686_chip_info - chip specific information
- * @int_vref_mv: AD5620/40/60: the internal reference voltage
+ * @int_vref_mv: the internal reference voltage
* @num_channels: number of channels
* @channel: channel specification
* @regmap_type: register map layout variant
*/
-
struct ad5686_chip_info {
u16 int_vref_mv;
unsigned int num_channels;
@@ -114,16 +113,16 @@ extern const struct ad5686_chip_info ad5679r_chip_info;

/**
* struct ad5686_state - driver instance specific data
- * @spi: spi_device
+ * @dev: device instance
* @chip_info: chip model specific constants, available modes etc
* @vref_mv: actual reference voltage used
* @pwr_down_mask: power down mask
* @pwr_down_mode: current power down mode
* @use_internal_vref: set to true if the internal reference voltage is used
- * @lock lock to protect the data buffer during regmap ops
- * @data: spi transfer buffers
+ * @lock: lock to protect access to state fields, which includes
+ * the data buffer during regmap ops
+ * @data: transfer buffers
*/
-
struct ad5686_state {
struct device *dev;
const struct ad5686_chip_info *chip_info;

--
2.43.0