backlight: tdo24m: Constify lcd_ops

krzk · lag-linaro · commit ee7b1e8465d5 · 2024-05-03T10:45:31.000+01:00
'struct lcd_ops' is not modified by core backlight code, so it can be made const for increased code safety. Signed-off-by: Krzysztof Kozlowski <krzk@kernel.org> Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org> Link: https://lore.kernel.org/r/20240424-video-backlight-lcd-ops-v2-14-1aaa82b07bc6@kernel.org Signed-off-by: Lee Jones <lee@kernel.org>
diff --git a/drivers/video/backlight/tdo24m.c b/drivers/video/backlight/tdo24m.c
@@ -322,7 +322,7 @@ static int tdo24m_set_mode(struct lcd_device *ld, struct fb_videomode *m)
 	return lcd->adj_mode(lcd, mode);
 }
 
-static struct lcd_ops tdo24m_ops = {
+static const struct lcd_ops tdo24m_ops = {
 	.get_power	= tdo24m_get_power,
 	.set_power	= tdo24m_set_power,
 	.set_mode	= tdo24m_set_mode,

Original file line number	Diff line number	Diff line change
`@@ -322,7 +322,7 @@ static int tdo24m_set_mode(struct lcd_device ld, struct fb_videomode m)`
`322`	`322`	`return lcd->adj_mode(lcd, mode);`
`323`	`323`	`}`
`324`	`324`
`325`		`-static struct lcd_ops tdo24m_ops = {`
	`325`	`+static const struct lcd_ops tdo24m_ops = {`
`326`	`326`	`.get_power = tdo24m_get_power,`
`327`	`327`	`.set_power = tdo24m_set_power,`
`328`	`328`	`.set_mode = tdo24m_set_mode,`