linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] iio: adc: axp20x_adc: Add missing sentinel to AXP717 ADC channel maps
@ 2025-06-07 13:56 Chen-Yu Tsai
  2025-06-07 15:30 ` Jonathan Cameron
  0 siblings, 1 reply; 5+ messages in thread
From: Chen-Yu Tsai @ 2025-06-07 13:56 UTC (permalink / raw)
  To: Jonathan Cameron, David Lechner, Nuno Sá, Andy Shevchenko
  Cc: Chen-Yu Tsai, linux-iio, linux-arm-kernel, linux-sunxi,
	linux-kernel, Andre Przywara

From: Chen-Yu Tsai <wens@csie.org>

The AXP717 ADC channel maps is missing a sentinel entry at the end. This
causes a KASAN warning.

Add the missing sentinel entry.

Fixes: 5ba0cb92584b ("iio: adc: axp20x_adc: add support for AXP717 ADC")
Signed-off-by: Chen-Yu Tsai <wens@csie.org>
---
 drivers/iio/adc/axp20x_adc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c
index 71584ffd3632..1b49325ec1ce 100644
--- a/drivers/iio/adc/axp20x_adc.c
+++ b/drivers/iio/adc/axp20x_adc.c
@@ -187,6 +187,7 @@ static struct iio_map axp717_maps[] = {
 		.consumer_channel = "batt_chrg_i",
 		.adc_channel_label = "batt_chrg_i",
 	},
+	{ }
 };
 
 /*
-- 
2.39.5


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] iio: adc: axp20x_adc: Add missing sentinel to AXP717 ADC channel maps
  2025-06-07 13:56 [PATCH] iio: adc: axp20x_adc: Add missing sentinel to AXP717 ADC channel maps Chen-Yu Tsai
@ 2025-06-07 15:30 ` Jonathan Cameron
  0 siblings, 0 replies; 5+ messages in thread
From: Jonathan Cameron @ 2025-06-07 15:30 UTC (permalink / raw)
  To: Chen-Yu Tsai
  Cc: David Lechner, Nuno Sá, Andy Shevchenko, Chen-Yu Tsai,
	linux-iio, linux-arm-kernel, linux-sunxi, linux-kernel,
	Andre Przywara

On Sat,  7 Jun 2025 21:56:27 +0800
Chen-Yu Tsai <wens@kernel.org> wrote:

> From: Chen-Yu Tsai <wens@csie.org>
> 
> The AXP717 ADC channel maps is missing a sentinel entry at the end. This
> causes a KASAN warning.
> 
> Add the missing sentinel entry.
> 
> Fixes: 5ba0cb92584b ("iio: adc: axp20x_adc: add support for AXP717 ADC")
> Signed-off-by: Chen-Yu Tsai <wens@csie.org>
Applied to my temporary fixes tree.  I'll rebase that on rc1 in next few days.

J
> ---
>  drivers/iio/adc/axp20x_adc.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c
> index 71584ffd3632..1b49325ec1ce 100644
> --- a/drivers/iio/adc/axp20x_adc.c
> +++ b/drivers/iio/adc/axp20x_adc.c
> @@ -187,6 +187,7 @@ static struct iio_map axp717_maps[] = {
>  		.consumer_channel = "batt_chrg_i",
>  		.adc_channel_label = "batt_chrg_i",
>  	},
> +	{ }
>  };
>  
>  /*


^ permalink raw reply	[flat|nested] 5+ messages in thread

* [PATCH] iio: adc: axp20x_adc: Add missing sentinel to AXP717 ADC channel maps
@ 2025-06-28  4:43 Chen-Yu Tsai
  2025-06-28 14:51 ` Jonathan Cameron
  0 siblings, 1 reply; 5+ messages in thread
From: Chen-Yu Tsai @ 2025-06-28  4:43 UTC (permalink / raw)
  To: Jonathan Cameron, David Lechner, Nuno Sá, Andy Shevchenko
  Cc: Chen-Yu Tsai, linux-iio, linux-sunxi, linux-kernel

From: Chen-Yu Tsai <wens@csie.org>

The AXP717 ADC channel maps is missing a sentinel entry at the end. This
causes a KASAN warning.

Add the missing sentinel entry.

Fixes: 5ba0cb92584b ("iio: adc: axp20x_adc: add support for AXP717 ADC")
Signed-off-by: Chen-Yu Tsai <wens@csie.org>
---
 drivers/iio/adc/axp20x_adc.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c
index 71584ffd3632..1b49325ec1ce 100644
--- a/drivers/iio/adc/axp20x_adc.c
+++ b/drivers/iio/adc/axp20x_adc.c
@@ -187,6 +187,7 @@ static struct iio_map axp717_maps[] = {
 		.consumer_channel = "batt_chrg_i",
 		.adc_channel_label = "batt_chrg_i",
 	},
+	{ }
 };
 
 /*
-- 
2.39.5


^ permalink raw reply related	[flat|nested] 5+ messages in thread

* Re: [PATCH] iio: adc: axp20x_adc: Add missing sentinel to AXP717 ADC channel maps
  2025-06-28  4:43 Chen-Yu Tsai
@ 2025-06-28 14:51 ` Jonathan Cameron
  2025-06-28 15:03   ` Chen-Yu Tsai
  0 siblings, 1 reply; 5+ messages in thread
From: Jonathan Cameron @ 2025-06-28 14:51 UTC (permalink / raw)
  To: Chen-Yu Tsai
  Cc: David Lechner, Nuno Sá, Andy Shevchenko, Chen-Yu Tsai,
	linux-iio, linux-sunxi, linux-kernel

On Sat, 28 Jun 2025 12:43:26 +0800
Chen-Yu Tsai <wens@kernel.org> wrote:

> From: Chen-Yu Tsai <wens@csie.org>
> 
> The AXP717 ADC channel maps is missing a sentinel entry at the end. This
> causes a KASAN warning.
> 
> Add the missing sentinel entry.
> 
> Fixes: 5ba0cb92584b ("iio: adc: axp20x_adc: add support for AXP717 ADC")
> Signed-off-by: Chen-Yu Tsai <wens@csie.org>
I picked up the version you posted on 7th June.
I've been a bit slow to send a fixed pull request - sorry about that; busy
few weeks.

Jonathan


> ---
>  drivers/iio/adc/axp20x_adc.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c
> index 71584ffd3632..1b49325ec1ce 100644
> --- a/drivers/iio/adc/axp20x_adc.c
> +++ b/drivers/iio/adc/axp20x_adc.c
> @@ -187,6 +187,7 @@ static struct iio_map axp717_maps[] = {
>  		.consumer_channel = "batt_chrg_i",
>  		.adc_channel_label = "batt_chrg_i",
>  	},
> +	{ }
>  };
>  
>  /*


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] iio: adc: axp20x_adc: Add missing sentinel to AXP717 ADC channel maps
  2025-06-28 14:51 ` Jonathan Cameron
@ 2025-06-28 15:03   ` Chen-Yu Tsai
  0 siblings, 0 replies; 5+ messages in thread
From: Chen-Yu Tsai @ 2025-06-28 15:03 UTC (permalink / raw)
  To: Jonathan Cameron
  Cc: David Lechner, Nuno Sá, Andy Shevchenko, linux-iio,
	linux-sunxi, linux-kernel

On Sat, Jun 28, 2025 at 10:51 PM Jonathan Cameron <jic23@kernel.org> wrote:
>
> On Sat, 28 Jun 2025 12:43:26 +0800
> Chen-Yu Tsai <wens@kernel.org> wrote:
>
> > From: Chen-Yu Tsai <wens@csie.org>
> >
> > The AXP717 ADC channel maps is missing a sentinel entry at the end. This
> > causes a KASAN warning.
> >
> > Add the missing sentinel entry.
> >
> > Fixes: 5ba0cb92584b ("iio: adc: axp20x_adc: add support for AXP717 ADC")
> > Signed-off-by: Chen-Yu Tsai <wens@csie.org>
> I picked up the version you posted on 7th June.
> I've been a bit slow to send a fixed pull request - sorry about that; busy
> few weeks.

Ah, sorry about that. I was looking through my work directory and trees,
didn't find a patch file and thought I hadn't sent it yet.

Good to know it's already queued up.

Thanks
ChenYu


> Jonathan
>
>
> > ---
> >  drivers/iio/adc/axp20x_adc.c | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c
> > index 71584ffd3632..1b49325ec1ce 100644
> > --- a/drivers/iio/adc/axp20x_adc.c
> > +++ b/drivers/iio/adc/axp20x_adc.c
> > @@ -187,6 +187,7 @@ static struct iio_map axp717_maps[] = {
> >               .consumer_channel = "batt_chrg_i",
> >               .adc_channel_label = "batt_chrg_i",
> >       },
> > +     { }
> >  };
> >
> >  /*
>

^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2025-06-28 15:03 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-06-07 13:56 [PATCH] iio: adc: axp20x_adc: Add missing sentinel to AXP717 ADC channel maps Chen-Yu Tsai
2025-06-07 15:30 ` Jonathan Cameron
  -- strict thread matches above, loose matches on Subject: below --
2025-06-28  4:43 Chen-Yu Tsai
2025-06-28 14:51 ` Jonathan Cameron
2025-06-28 15:03   ` Chen-Yu Tsai

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).