backlight: corgi_lcd: Constify lcd_ops

krzk · lag-linaro · commit 94194e314ec8 · 2024-05-03T10:44:56.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-3-1aaa82b07bc6@kernel.org Signed-off-by: Lee Jones <lee@kernel.org>
diff --git a/drivers/video/backlight/corgi_lcd.c b/drivers/video/backlight/corgi_lcd.c
@@ -380,7 +380,7 @@ static int corgi_lcd_get_power(struct lcd_device *ld)
 	return lcd->power;
 }
 
-static struct lcd_ops corgi_lcd_ops = {
+static const struct lcd_ops corgi_lcd_ops = {
 	.get_power	= corgi_lcd_get_power,
 	.set_power	= corgi_lcd_set_power,
 	.set_mode	= corgi_lcd_set_mode,

Original file line number	Diff line number	Diff line change
`@@ -380,7 +380,7 @@ static int corgi_lcd_get_power(struct lcd_device *ld)`
`380`	`380`	`return lcd->power;`
`381`	`381`	`}`
`382`	`382`
`383`		`-static struct lcd_ops corgi_lcd_ops = {`
	`383`	`+static const struct lcd_ops corgi_lcd_ops = {`
`384`	`384`	`.get_power = corgi_lcd_get_power,`
`385`	`385`	`.set_power = corgi_lcd_set_power,`
`386`	`386`	`.set_mode = corgi_lcd_set_mode,`