staging: iio: frequency: reorder dds.h macro parameters to match IIO convention

Browse source

The IIO subsystem convention requires that the file permission
(_mode) parameter be the first argument of IIO_DEV_ATTR_* macros.
The dds.h macros had _mode after _channel, causing checkpatch to
misinterpret the channel number as a permission value.

Reorder the parameters so _mode is first, matching the convention
established by IIO_DEV_ATTR_SAMP_FREQ in include/linux/iio/sysfs.h.
Where _mode is hard coded, make it a parameter in the interests
of consistency and update all callers to match previously hard coded
value.

Update all callers in ad9834.c and ad9832.c accordingly.

Compile tested.

Signed-off-by: Xiaofeng Yuan <xiaofengmian@163.com>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>

...

This commit is contained in:

Xiaofeng Yuan 2026-06-24 02:16:12 +00:00 • committed by Jonathan Cameron

parent cbfd04ca56

commit 8463551778

3 changed files with 33 additions and 33 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

20

drivers/staging/iio/frequency/ad9832.c

View file

22

drivers/staging/iio/frequency/ad9834.c

View file

24

drivers/staging/iio/frequency/dds.h

View file