diff mbox series

[15/22] iio: light: si1145: use new '.masklength' accessors

Message ID	20240718-dev-iio-masklength-private2-v1-15-8e12cd042906@analog.com (mailing list archive)
State	Accepted
Headers	show Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 4E9DA13F458 for <linux-iio@vger.kernel.org>; Thu, 18 Jul 2024 13:38:26 +0000 (UTC) From: Nuno Sa via B4 Relay <devnull+nuno.sa.analog.com@kernel.org> Date: Thu, 18 Jul 2024 15:38:01 +0200 Subject: [PATCH 15/22] iio: light: si1145: use new '.masklength' accessors Precedence: bulk MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-Id: <20240718-dev-iio-masklength-private2-v1-15-8e12cd042906@analog.com> References: <20240718-dev-iio-masklength-private2-v1-0-8e12cd042906@analog.com> In-Reply-To: <20240718-dev-iio-masklength-private2-v1-0-8e12cd042906@analog.com> To: linux-iio@vger.kernel.org Cc: Jonathan Cameron <jic23@kernel.org>, Lars-Peter Clausen <lars@metafoo.de>, Andreas Klinger <ak@it-klinger.de>, Song Qiang <songqiang1304521@gmail.com> Reply-To: nuno.sa@analog.com
Series	iio: make masklength __private (round 2) \| expand [00/22] iio: make masklength __private (round 2) [01/22] iio: adc: cc10001_adc: make use of iio_for_each_active_channel() [02/22] iio: adc: dln2-adc: use new '.masklength' accessors [03/22] iio: adc: hx711: make use of iio_for_each_active_channel() [04/22] iio: dummy: iio_simple_dummy_buffer: use iio_for_each_active_channel() [05/22] iio: health: afe4403: make use of iio_for_each_active_channel() [06/22] iio: health: max30102: make use of iio_get_masklength() [07/22] iio: health: afe4404: make use of iio_for_each_active_channel() [08/22] iio: humidity: am2315: make use of iio_for_each_active_channel() [09/22] iio: imu: bmi160_core: make use of iio_for_each_active_channel() [10/22] iio: imu: bno055: make use of iio_get_masklength() [11/22] iio: imu: kmx61: make use of iio_for_each_active_channel() [12/22] iio: light: adjd_s311: make use of iio_for_each_active_channel() [13/22] iio: light: gp2ap020a00f: make use of iio_for_each_active_channel() [14/22] iio: light: isl29125: make use of iio_for_each_active_channel() [15/22] iio: light: si1145: use new '.masklength' accessors [16/22] iio: light: tcs3414: make use of iio_for_each_active_channel() [17/22] iio: light: tcs3472: make use of iio_for_each_active_channel() [18/22] iio: magnetometer: rm3100-core: make use of iio_get_masklength() [19/22] iio: pressure: dlhl60d: make use of iio_for_each_active_channel() [20/22] iio: proximity: hx9023s: make use of iio_for_each_active_channel() [21/22] iio: proximity: sx9500: make use of iio_for_each_active_channel() [22/22] iio: proximity: sx_common: make use of iio_for_each_active_channel()

Commit Message

Nuno Sa via B4 Relay July 18, 2024, 1:38 p.m. UTC

From: Nuno Sa <nuno.sa@analog.com>

Make use of iio_get_masklength) and iio_for_each_active_channel() to
access '.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>
---
 drivers/iio/light/si1145.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff mbox series

Patch

diff --git a/drivers/iio/light/si1145.c b/drivers/iio/light/si1145.c
index 77666b780a5c5..66abda021696a 100644
--- a/drivers/iio/light/si1145.c
+++ b/drivers/iio/light/si1145.c
@@ -465,11 +465,10 @@  static irqreturn_t si1145_trigger_handler(int irq, void *private)
 			goto done;
 	}
 
-	for_each_set_bit(i, indio_dev->active_scan_mask,
-		indio_dev->masklength) {
+	iio_for_each_active_channel(indio_dev, i) {
 		int run = 1;
 
-		while (i + run < indio_dev->masklength) {
+		while (i + run < iio_get_masklength(indio_dev)) {
 			if (!test_bit(i + run, indio_dev->active_scan_mask))
 				break;
 			if (indio_dev->channels[i + run].address !=
@@ -514,7 +513,7 @@  static int si1145_set_chlist(struct iio_dev *indio_dev, unsigned long scan_mask)
 	if (data->scan_mask == scan_mask)
 		return 0;
 
-	for_each_set_bit(i, &scan_mask, indio_dev->masklength) {
+	for_each_set_bit(i, &scan_mask, iio_get_masklength(indio_dev)) {
 		switch (indio_dev->channels[i].address) {
 		case SI1145_REG_ALSVIS_DATA:
 			reg |= SI1145_CHLIST_EN_ALSVIS;