From: Subhajit Ghosh <subhajit.ghosh@tweaklogic.com>
To: "Antoniu Miclaus" <antoniu.miclaus@analog.com>,
"Jonathan Cameron" <jic23@kernel.org>,
"David Lechner" <dlechner@baylibre.com>,
"Nuno Sá" <nuno.sa@analog.com>,
"Andy Shevchenko" <andy@kernel.org>,
"Jagath Jog J" <jagathjog1996@gmail.com>,
"Eugene Zaikonnikov" <ez@norphonic.com>,
"Marcelo Schmitt" <marcelo.schmitt1@gmail.com>,
linux-iio@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] iio: light: apds9306: remove unused nlux_per_count
Date: Fri, 30 Jan 2026 09:06:00 +1030 [thread overview]
Message-ID: <be6e6492-b3f9-4a0b-87b6-3816eff6d395@tweaklogic.com> (raw)
In-Reply-To: <20260129174253.39574-3-antoniu.miclaus@analog.com>
On 30/1/26 04:12, Antoniu Miclaus wrote:
> Remove unused nlux_per_count field from apds9306_data
> struct. The field is declared but never accessed in the
> driver.
>
> Signed-off-by: Antoniu Miclaus <antoniu.miclaus@analog.com>
> ---
> drivers/iio/light/apds9306.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/drivers/iio/light/apds9306.c b/drivers/iio/light/apds9306.c
> index 7e68cca0edfa..ef6429bde193 100644
> --- a/drivers/iio/light/apds9306.c
> +++ b/drivers/iio/light/apds9306.c
> @@ -168,7 +168,6 @@ struct apds9306_regfields {
> * respectively.
> * @regmap: Regmap structure pointer
> * @rf: Regmap register fields structure
> - * @nlux_per_count: Nano lux per ADC count for a particular model
> * @read_data_available: Flag set by IRQ handler for ADC data available
> */
> struct apds9306_data {
> @@ -180,7 +179,6 @@ struct apds9306_data {
> struct regmap *regmap;
> struct apds9306_regfields rf;
>
> - int nlux_per_count;
> int read_data_available;
> };
>
Good find.
Acked-by: Subhajit Ghosh <subhajit.ghosh@tweaklogic.com>
Regards,
Subhajit Ghosh
next prev parent reply other threads:[~2026-01-29 22:34 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-01-29 17:42 [PATCH] iio: humidity: hdc2010: remove unused interrupt_config Antoniu Miclaus
2026-01-29 17:42 ` [PATCH] iio: imu: bmi323: remove unused drdy_trigger_enabled Antoniu Miclaus
2026-01-29 17:42 ` [PATCH] iio: light: apds9306: remove unused nlux_per_count Antoniu Miclaus
2026-01-29 22:36 ` Subhajit Ghosh [this message]
2026-01-29 17:42 ` [PATCH] iio: light: gp2ap020a00f: remove unused debug_reg_addr Antoniu Miclaus
2026-01-31 17:01 ` [PATCH] iio: humidity: hdc2010: remove unused interrupt_config Jonathan Cameron
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=be6e6492-b3f9-4a0b-87b6-3816eff6d395@tweaklogic.com \
--to=subhajit.ghosh@tweaklogic.com \
--cc=andy@kernel.org \
--cc=antoniu.miclaus@analog.com \
--cc=dlechner@baylibre.com \
--cc=ez@norphonic.com \
--cc=jagathjog1996@gmail.com \
--cc=jic23@kernel.org \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=marcelo.schmitt1@gmail.com \
--cc=nuno.sa@analog.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox