Message ID | 20240718-dev-iio-masklength-private2-v1-22-8e12cd042906@analog.com (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | iio: make masklength __private (round 2) | expand |
On Thu, 18 Jul 2024 15:38:08 +0200 Nuno Sa via B4 Relay <devnull+nuno.sa.analog.com@kernel.org> wrote: > From: Nuno Sa <nuno.sa@analog.com> > > 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: Nuno Sa <nuno.sa@analog.com> I applied them all to the testing branch of iio.git which will be rebased on rc1 once available then pushed out as togreg for linux-next to pick up. A few of the functions that got changes were pieces of nastiness that I'd long banished from my mind so at least I got to look at those again :) Datashuffling logic is 'fun'. Thanks, Jonathan
diff --git a/drivers/iio/proximity/sx_common.c b/drivers/iio/proximity/sx_common.c index a95e9814aaf2..71aa6dced7d3 100644 --- a/drivers/iio/proximity/sx_common.c +++ b/drivers/iio/proximity/sx_common.c @@ -369,8 +369,7 @@ static irqreturn_t sx_common_trigger_handler(int irq, void *private) mutex_lock(&data->mutex); - for_each_set_bit(bit, indio_dev->active_scan_mask, - indio_dev->masklength) { + iio_for_each_active_channel(indio_dev, bit) { ret = data->chip_info->ops.read_prox_data(data, &indio_dev->channels[bit], &val); @@ -398,8 +397,7 @@ static int sx_common_buffer_preenable(struct iio_dev *indio_dev) int bit, ret; mutex_lock(&data->mutex); - for_each_set_bit(bit, indio_dev->active_scan_mask, - indio_dev->masklength) + iio_for_each_active_channel(indio_dev, bit) __set_bit(indio_dev->channels[bit].channel, &channels); ret = sx_common_update_chan_en(data, channels, data->chan_event);