fbdev: omap: lcd_ams_delta: Constify lcd_ops

krzk · lag-linaro · commit feb61a4b34a8 · 2024-05-03T10:45:50.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-18-1aaa82b07bc6@kernel.org Signed-off-by: Lee Jones <lee@kernel.org>
diff --git a/drivers/video/fbdev/omap/lcd_ams_delta.c b/drivers/video/fbdev/omap/lcd_ams_delta.c
@@ -76,7 +76,7 @@ static int ams_delta_lcd_get_contrast(struct lcd_device *dev)
 	return ams_delta_lcd & AMS_DELTA_MAX_CONTRAST;
 }
 
-static struct lcd_ops ams_delta_lcd_ops = {
+static const struct lcd_ops ams_delta_lcd_ops = {
 	.get_power = ams_delta_lcd_get_power,
 	.set_power = ams_delta_lcd_set_power,
 	.get_contrast = ams_delta_lcd_get_contrast,

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,7 @@ static int ams_delta_lcd_get_contrast(struct lcd_device *dev)`
`76`	`76`	`return ams_delta_lcd & AMS_DELTA_MAX_CONTRAST;`
`77`	`77`	`}`
`78`	`78`
`79`		`-static struct lcd_ops ams_delta_lcd_ops = {`
	`79`	`+static const struct lcd_ops ams_delta_lcd_ops = {`
`80`	`80`	`.get_power = ams_delta_lcd_get_power,`
`81`	`81`	`.set_power = ams_delta_lcd_set_power,`
`82`	`82`	`.get_contrast = ams_delta_lcd_get_contrast,`