Commit 327f46fe authored by Nuno Sa's avatar Nuno Sa Committed by Jonathan Cameron

iio: light: gp2ap020a00f: make use of iio_for_each_active_channel()

Use iio_for_each_active_channel() to iterate over active channels
accessing '.masklength' so it can be annotated as __private when there are
no more direct users of it.
Signed-off-by: default avatarNuno Sa <nuno.sa@analog.com>
Link: https://patch.msgid.link/20240718-dev-iio-masklength-private2-v1-13-8e12cd042906@analog.comSigned-off-by: default avatarJonathan Cameron <Jonathan.Cameron@huawei.com>
parent f801ab1a
...@@ -965,8 +965,7 @@ static irqreturn_t gp2ap020a00f_trigger_handler(int irq, void *data) ...@@ -965,8 +965,7 @@ static irqreturn_t gp2ap020a00f_trigger_handler(int irq, void *data)
size_t d_size = 0; size_t d_size = 0;
int i, out_val, ret; int i, out_val, ret;
for_each_set_bit(i, indio_dev->active_scan_mask, iio_for_each_active_channel(indio_dev, i) {
indio_dev->masklength) {
ret = regmap_bulk_read(priv->regmap, ret = regmap_bulk_read(priv->regmap,
GP2AP020A00F_DATA_REG(i), GP2AP020A00F_DATA_REG(i),
&priv->buffer[d_size], 2); &priv->buffer[d_size], 2);
...@@ -1397,8 +1396,7 @@ static int gp2ap020a00f_buffer_postenable(struct iio_dev *indio_dev) ...@@ -1397,8 +1396,7 @@ static int gp2ap020a00f_buffer_postenable(struct iio_dev *indio_dev)
* two separate IIO channels they are treated in the driver logic * two separate IIO channels they are treated in the driver logic
* as if they were controlled independently. * as if they were controlled independently.
*/ */
for_each_set_bit(i, indio_dev->active_scan_mask, iio_for_each_active_channel(indio_dev, i) {
indio_dev->masklength) {
switch (i) { switch (i) {
case GP2AP020A00F_SCAN_MODE_LIGHT_CLEAR: case GP2AP020A00F_SCAN_MODE_LIGHT_CLEAR:
err = gp2ap020a00f_exec_cmd(data, err = gp2ap020a00f_exec_cmd(data,
...@@ -1435,8 +1433,7 @@ static int gp2ap020a00f_buffer_predisable(struct iio_dev *indio_dev) ...@@ -1435,8 +1433,7 @@ static int gp2ap020a00f_buffer_predisable(struct iio_dev *indio_dev)
mutex_lock(&data->lock); mutex_lock(&data->lock);
for_each_set_bit(i, indio_dev->active_scan_mask, iio_for_each_active_channel(indio_dev, i) {
indio_dev->masklength) {
switch (i) { switch (i) {
case GP2AP020A00F_SCAN_MODE_LIGHT_CLEAR: case GP2AP020A00F_SCAN_MODE_LIGHT_CLEAR:
err = gp2ap020a00f_exec_cmd(data, err = gp2ap020a00f_exec_cmd(data,
......
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