Message ID | 20201109221012.177478-1-rikard.falkeborn@gmail.com |
---|---|
State | New |
Headers | show |
Series | pinctrl: renesas: Constify sh73a0_vccq_mc0_ops | expand |
On Mon, Nov 9, 2020 at 11:10 PM Rikard Falkeborn <rikard.falkeborn@gmail.com> wrote: > The only usage of sh73a0_vccq_mc0_ops is to assign its address to the > ops field in the regulator_desc struct, which is a const pointer. Make > it const to allow the compiler to put it in read-only memory. > > Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> i.e. will queue in renesas-pinctrl-for-v5.11. Gr{oetje,eeting}s, Geert
diff --git a/drivers/pinctrl/renesas/pfc-sh73a0.c b/drivers/pinctrl/renesas/pfc-sh73a0.c index afabd95105d5..96b91e95b1e1 100644 --- a/drivers/pinctrl/renesas/pfc-sh73a0.c +++ b/drivers/pinctrl/renesas/pfc-sh73a0.c @@ -4279,7 +4279,7 @@ static int sh73a0_vccq_mc0_get_voltage(struct regulator_dev *reg) return 3300000; } -static struct regulator_ops sh73a0_vccq_mc0_ops = { +static const struct regulator_ops sh73a0_vccq_mc0_ops = { .enable = sh73a0_vccq_mc0_enable, .disable = sh73a0_vccq_mc0_disable, .is_enabled = sh73a0_vccq_mc0_is_enabled,
The only usage of sh73a0_vccq_mc0_ops is to assign its address to the ops field in the regulator_desc struct, which is a const pointer. Make it const to allow the compiler to put it in read-only memory. Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com> --- drivers/pinctrl/renesas/pfc-sh73a0.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)