thermal/drivers/intel_powerclamp: Constify static thermal_cooling_device_ops

rikardfalkeborn · dlezcano · commit 8152d2a9e73d · 2021-11-30T15:42:39.000+01:00
The only usage of powerclamp_cooling_ops is to pass its address to thermal_cooling_device_register(), which takes a pointer to const struct thermal_cooling_device_ops. Make it const to allow the compiler to put it in read-only memory. Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com> Link: https://lore.kernel.org/r/20211128214641.30953-1-rikard.falkeborn@gmail.com Signed-off-by: Daniel Lezcano <daniel.lezcano@linaro.org>
diff --git a/drivers/thermal/intel/intel_powerclamp.c b/drivers/thermal/intel/intel_powerclamp.c
@@ -641,7 +641,7 @@ static int powerclamp_set_cur_state(struct thermal_cooling_device *cdev,
 }
 
 /* bind to generic thermal layer as cooling device*/
-static struct thermal_cooling_device_ops powerclamp_cooling_ops = {
+static const struct thermal_cooling_device_ops powerclamp_cooling_ops = {
 	.get_max_state = powerclamp_get_max_state,
 	.get_cur_state = powerclamp_get_cur_state,
 	.set_cur_state = powerclamp_set_cur_state,

Original file line number	Diff line number	Diff line change
`@@ -641,7 +641,7 @@ static int powerclamp_set_cur_state(struct thermal_cooling_device *cdev,`
`641`	`641`	`}`
`642`	`642`
`643`	`643`	`/* bind to generic thermal layer as cooling device*/`
`644`		`-static struct thermal_cooling_device_ops powerclamp_cooling_ops = {`
	`644`	`+static const struct thermal_cooling_device_ops powerclamp_cooling_ops = {`
`645`	`645`	`.get_max_state = powerclamp_get_max_state,`
`646`	`646`	`.get_cur_state = powerclamp_get_cur_state,`
`647`	`647`	`.set_cur_state = powerclamp_set_cur_state,`