diff mbox series

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

Message ID 20240119232630.2752239-1-colin.i.king@gmail.com
State New
Headers show
Series [next] leds: aw200xx: make read-only array coeff_table static const | expand

Commit Message

Colin Ian King Jan. 19, 2024, 11:26 p.m. UTC
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 <colin.i.king@gmail.com>
---
 drivers/leds/leds-aw200xx.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Lee Jones Jan. 25, 2024, 1:35 p.m. UTC | #1
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 [李琼斯]
diff mbox series

Patch

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;