Consider '*' alignment in the comments of struct ad5380_chip_info declaration. ... > * @channel_template: channel specification template > * @num_channels: number of channels > * @int_vref: internal vref in uV > -*/ > + */ > While at it, drop the unneeded blank line here. > struct ad5380_chip_info { ... } Signed-off-by: Jiang Jian <jiangjian@xxxxxxxxxx> --- drivers/iio/dac/ad5380.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/iio/dac/ad5380.c b/drivers/iio/dac/ad5380.c index a44c83242fb1..81775152aac6 100644 --- a/drivers/iio/dac/ad5380.c +++ b/drivers/iio/dac/ad5380.c @@ -36,8 +36,7 @@ * @channel_template: channel specification template * @num_channels: number of channels * @int_vref: internal vref in uV -*/ - + */ struct ad5380_chip_info { struct iio_chan_spec channel_template; unsigned int num_channels; @@ -53,7 +52,6 @@ struct ad5380_chip_info { * @pwr_down: whether the chip is currently in power down mode * @lock: lock to protect the data buffer during regmap ops */ - struct ad5380_state { struct regmap *regmap; const struct ad5380_chip_info *chip_info; -- 2.17.1