backlight: platform_lcd: Constify lcd_ops

krzk · lag-linaro · commit 02bc4c447e29 · 2024-05-03T10:45:28.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-13-1aaa82b07bc6@kernel.org Signed-off-by: Lee Jones <lee@kernel.org>
diff --git a/drivers/video/backlight/platform_lcd.c b/drivers/video/backlight/platform_lcd.c
@@ -62,7 +62,7 @@ static int platform_lcd_match(struct lcd_device *lcd, struct fb_info *info)
 	return plcd->us->parent == info->device;
 }
 
-static struct lcd_ops platform_lcd_ops = {
+static const struct lcd_ops platform_lcd_ops = {
 	.get_power	= platform_lcd_get_power,
 	.set_power	= platform_lcd_set_power,
 	.check_fb	= platform_lcd_match,

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ static int platform_lcd_match(struct lcd_device lcd, struct fb_info info)`
`62`	`62`	`return plcd->us->parent == info->device;`
`63`	`63`	`}`
`64`	`64`
`65`		`-static struct lcd_ops platform_lcd_ops = {`
	`65`	`+static const struct lcd_ops platform_lcd_ops = {`
`66`	`66`	`.get_power = platform_lcd_get_power,`
`67`	`67`	`.set_power = platform_lcd_set_power,`
`68`	`68`	`.check_fb = platform_lcd_match,`