On Tue, Jul 2, 2024 at 7:03 PM Nuno Sa via B4 Relay <devnull+nuno.sa.analog.com@xxxxxxxxxx> wrote: > > From: Nuno Sa <nuno.sa@xxxxxxxxxx> > > 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. > Reviewed-by: Alexandru Ardelean <aardelean@xxxxxxxxxxxx> > Signed-off-by: Nuno Sa <nuno.sa@xxxxxxxxxx> > --- > drivers/iio/imu/adis16475.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/iio/imu/adis16475.c b/drivers/iio/imu/adis16475.c > index cf73c6f46c79d..1cc2a66a78ff3 100644 > --- a/drivers/iio/imu/adis16475.c > +++ b/drivers/iio/imu/adis16475.c > @@ -1608,8 +1608,7 @@ static int adis16475_push_single_sample(struct iio_poll_func *pf) > return -EINVAL; > } > > - for_each_set_bit(bit, indio_dev->active_scan_mask, > - indio_dev->masklength) { > + iio_for_each_active_channel(indio_dev, bit) { > /* > * When burst mode is used, system flags is the first data > * channel in the sequence, but the scan index is 7. > > -- > 2.45.2 > > >