fbdev: clps711x: Constify lcd_ops

krzk · lag-linaro · commit 9293c302f493 · 2024-05-03T10:45:40.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: Thomas Zimmermann <tzimmermann@suse.de> Link: https://lore.kernel.org/r/20240424-video-backlight-lcd-ops-v2-16-1aaa82b07bc6@kernel.org Signed-off-by: Lee Jones <lee@kernel.org>
diff --git a/drivers/video/fbdev/clps711x-fb.c b/drivers/video/fbdev/clps711x-fb.c
@@ -197,7 +197,7 @@ static int clps711x_lcd_set_power(struct lcd_device *lcddev, int blank)
 	return 0;
 }
 
-static struct lcd_ops clps711x_lcd_ops = {
+static const struct lcd_ops clps711x_lcd_ops = {
 	.check_fb	= clps711x_lcd_check_fb,
 	.get_power	= clps711x_lcd_get_power,
 	.set_power	= clps711x_lcd_set_power,

Original file line number	Diff line number	Diff line change
`@@ -197,7 +197,7 @@ static int clps711x_lcd_set_power(struct lcd_device *lcddev, int blank)`
`197`	`197`	`return 0;`
`198`	`198`	`}`
`199`	`199`
`200`		`-static struct lcd_ops clps711x_lcd_ops = {`
	`200`	`+static const struct lcd_ops clps711x_lcd_ops = {`
`201`	`201`	`.check_fb = clps711x_lcd_check_fb,`
`202`	`202`	`.get_power = clps711x_lcd_get_power,`
`203`	`203`	`.set_power = clps711x_lcd_set_power,`