dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH][next] drm/bridge: lontium-lt8912b: make read-only const array supply_names static
@ 2026-07-14 19:03 Colin Ian King
  2026-07-15  6:42 ` Francesco Dolcini
  2026-07-15 10:21 ` Luca Ceresoli
  0 siblings, 2 replies; 3+ messages in thread
From: Colin Ian King @ 2026-07-14 19:03 UTC (permalink / raw)
  To: Francesco Dolcini, Andrzej Hajda, Neil Armstrong, Robert Foss,
	Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Luca Ceresoli,
	Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, David Airlie,
	Simona Vetter, dri-devel
  Cc: kernel-janitors, linux-kernel

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

Signed-off-by: Colin Ian King <colin.i.king@gmail.com>
---
 drivers/gpu/drm/bridge/lontium-lt8912b.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/gpu/drm/bridge/lontium-lt8912b.c b/drivers/gpu/drm/bridge/lontium-lt8912b.c
index 729b12b67470..62eabf0a3b88 100644
--- a/drivers/gpu/drm/bridge/lontium-lt8912b.c
+++ b/drivers/gpu/drm/bridge/lontium-lt8912b.c
@@ -673,7 +673,7 @@ static DEFINE_SIMPLE_DEV_PM_OPS(lt8912_bridge_pm_ops, lt8912_bridge_suspend, lt8
 static int lt8912_get_regulators(struct lt8912 *lt)
 {
 	unsigned int i;
-	const char * const supply_names[] = {
+	static const char * const supply_names[] = {
 		"vdd", "vccmipirx", "vccsysclk", "vcclvdstx",
 		"vcchdmitx", "vcclvdspll", "vcchdmipll"
 	};
-- 
2.53.0


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

* Re: [PATCH][next] drm/bridge: lontium-lt8912b: make read-only const array supply_names static
  2026-07-14 19:03 [PATCH][next] drm/bridge: lontium-lt8912b: make read-only const array supply_names static Colin Ian King
@ 2026-07-15  6:42 ` Francesco Dolcini
  2026-07-15 10:21 ` Luca Ceresoli
  1 sibling, 0 replies; 3+ messages in thread
From: Francesco Dolcini @ 2026-07-15  6:42 UTC (permalink / raw)
  To: Colin Ian King
  Cc: Francesco Dolcini, Andrzej Hajda, Neil Armstrong, Robert Foss,
	Laurent Pinchart, Jonas Karlman, Jernej Skrabec, Luca Ceresoli,
	Maarten Lankhorst, Maxime Ripard, Thomas Zimmermann, David Airlie,
	Simona Vetter, dri-devel, kernel-janitors, linux-kernel

On Tue, Jul 14, 2026 at 08:03:59PM +0100, Colin Ian King wrote:
> Don't populate the read-only const array supply_names on the stack
> at run time, instead make it static
> 
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>

Acked-by: Francesco Dolcini <francesco.dolcini@toradex.com>

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

* Re: [PATCH][next] drm/bridge: lontium-lt8912b: make read-only const array supply_names static
  2026-07-14 19:03 [PATCH][next] drm/bridge: lontium-lt8912b: make read-only const array supply_names static Colin Ian King
  2026-07-15  6:42 ` Francesco Dolcini
@ 2026-07-15 10:21 ` Luca Ceresoli
  1 sibling, 0 replies; 3+ messages in thread
From: Luca Ceresoli @ 2026-07-15 10:21 UTC (permalink / raw)
  To: Colin Ian King, Francesco Dolcini, Andrzej Hajda, Neil Armstrong,
	Robert Foss, Laurent Pinchart, Jonas Karlman, Jernej Skrabec,
	Luca Ceresoli, Maarten Lankhorst, Maxime Ripard,
	Thomas Zimmermann, David Airlie, Simona Vetter, dri-devel
  Cc: kernel-janitors, linux-kernel

On Tue Jul 14, 2026 at 9:03 PM CEST, Colin Ian King wrote:
> Don't populate the read-only const array supply_names on the stack
> at run time, instead make it static
>
> Signed-off-by: Colin Ian King <colin.i.king@gmail.com>

Reviewed-by: Luca Ceresoli <luca.ceresoli@bootlin.com>

--
Luca Ceresoli, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

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

end of thread, other threads:[~2026-07-15 10:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-14 19:03 [PATCH][next] drm/bridge: lontium-lt8912b: make read-only const array supply_names static Colin Ian King
2026-07-15  6:42 ` Francesco Dolcini
2026-07-15 10:21 ` Luca Ceresoli

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox