2024-01-19 23:26:47

by Colin Ian King

Subject: [PATCH][next] leds: aw200xx: make read-only array coeff_table static const

Don't populate the read-only array coeff_table on the stack at
run time, instead make it static const.

Signed-off-by: Colin Ian King <[email protected]>
---
drivers/leds/leds-aw200xx.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/leds/leds-aw200xx.c b/drivers/leds/leds-aw200xx.c
index f584a7f98fc5..6c8c9f2c19e3 100644
--- a/drivers/leds/leds-aw200xx.c
+++ b/drivers/leds/leds-aw200xx.c
@@ -282,7 +282,7 @@ static int aw200xx_set_imax(const struct aw200xx *const chip,
u32 led_imax_uA)
{
u32 g_imax_uA = aw200xx_imax_to_global(chip, led_imax_uA);
- u32 coeff_table[] = {1, 2, 3, 4, 6, 8, 12, 16};
+ static const u32 coeff_table[] = {1, 2, 3, 4, 6, 8, 12, 16};
u32 gccr_imax = UINT_MAX;
u32 cur_imax = 0;
int i;
--
2.39.2

2024-01-25 13:35:57

by Lee Jones

Subject: Re: (subset) [PATCH][next] leds: aw200xx: make read-only array coeff_table static const

On Fri, 19 Jan 2024 23:26:30 +0000, Colin Ian King wrote:
> Don't populate the read-only array coeff_table on the stack at
> run time, instead make it static const.
>
>

Applied, thanks!

[1/1] leds: aw200xx: make read-only array coeff_table static const
commit: bcc7f2ee83dc1dace6eeb6b25014f253ca2a2a51

--
Lee Jones [李琼斯]