Commit fc89b38d authored by Jonathan Cameron's avatar Jonathan Cameron Committed by Greg Kroah-Hartman

staging: iio: amend macros for SCAN_EL creation to add separate index attr

As Manuel Stahl observed, putting data into the naming of an attribute
(beyond what it is for) breaks the one value per attribute rule (in spirit
at least).  Hence we introduce a separate _index attribute for each scan
element to tell userspace the ordering in output from the buffer.

This will generate a lot of sparse warnings as all drivers will have unused
iio_const_attrs created.  The rest of the set will clean these up.

The final patch will remove the old indexing method. It is not here so
as to maintain one or other interface for all devices as this set
progresses.
Signed-off-by: default avatarJonathan Cameron <jic23@cam.ac.uk>
Signed-off-by: default avatarManuel Stahl <manuel.stahl@iis.fraunhofer.de>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@suse.de>
parent 44f270de
...@@ -243,7 +243,8 @@ ssize_t iio_scan_el_ts_show(struct device *dev, struct device_attribute *attr, ...@@ -243,7 +243,8 @@ ssize_t iio_scan_el_ts_show(struct device *dev, struct device_attribute *attr,
.number = _number, \ .number = _number, \
.label = _label, \ .label = _label, \
.set_state = _controlfunc, \ .set_state = _controlfunc, \
} }; \
static IIO_CONST_ATTR(_name##_index, #_number)
#define IIO_SCAN_EL_C(_name, _number, _label, _controlfunc) \ #define IIO_SCAN_EL_C(_name, _number, _label, _controlfunc) \
__IIO_SCAN_EL_C(_name, _number, _label, _controlfunc) __IIO_SCAN_EL_C(_name, _number, _label, _controlfunc)
...@@ -257,7 +258,14 @@ ssize_t iio_scan_el_ts_show(struct device *dev, struct device_attribute *attr, ...@@ -257,7 +258,14 @@ ssize_t iio_scan_el_ts_show(struct device *dev, struct device_attribute *attr,
.number = _number, \ .number = _number, \
.label = _label, \ .label = _label, \
.set_state = _cf, \ .set_state = _cf, \
}; \
static struct iio_const_attr iio_const_attr_##_name##_index = { \
.string = #_number, \
.dev_attr = __ATTR(_string##_index, \
S_IRUGO, iio_read_const_attr, NULL) \
} }
#define IIO_SCAN_NAMED_EL_C(_name, _string, _number, _label, _cf) \ #define IIO_SCAN_NAMED_EL_C(_name, _string, _number, _label, _cf) \
__IIO_SCAN_NAMED_EL_C(_name, _string, _number, _label, _cf) __IIO_SCAN_NAMED_EL_C(_name, _string, _number, _label, _cf)
/** /**
...@@ -272,7 +280,8 @@ ssize_t iio_scan_el_ts_show(struct device *dev, struct device_attribute *attr, ...@@ -272,7 +280,8 @@ ssize_t iio_scan_el_ts_show(struct device *dev, struct device_attribute *attr,
S_IRUGO | S_IWUSR, \ S_IRUGO | S_IWUSR, \
iio_scan_el_ts_show, \ iio_scan_el_ts_show, \
iio_scan_el_ts_store), \ iio_scan_el_ts_store), \
} }; \
static IIO_CONST_ATTR(timestamp_index, #number)
/** /**
* IIO_CONST_ATTR_SCAN_EL_TYPE - attr to specify the data format of a scan el * IIO_CONST_ATTR_SCAN_EL_TYPE - attr to specify the data format of a scan el
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment