pinctrl: actions: pinctrl-owl: Constify owl_pinctrl_ops and owl_pinmux_ops
authorRikard Falkeborn <rikard.falkeborn@gmail.com>
Sun, 30 Aug 2020 22:43:09 +0000 (00:43 +0200)
committerLinus Walleij <linus.walleij@linaro.org>
Sat, 12 Sep 2020 10:15:19 +0000 (12:15 +0200)
The only usage of owl_pinctrl_ops and owl_pinmux_ops is to assign their
addresses to the pctlops and the pmxops fields in the pinctrl_desc struct,
which are const pointers. Make them const to allow the compiler to put them
in read-only memory.

Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org>
Link: https://lore.kernel.org/r/20200830224311.36994-2-rikard.falkeborn@gmail.com
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
drivers/pinctrl/actions/pinctrl-owl.c

index 7efdfb4..903a4ba 100644 (file)
@@ -125,7 +125,7 @@ static void owl_pin_dbg_show(struct pinctrl_dev *pctrldev,
        seq_printf(s, "%s", dev_name(pctrl->dev));
 }
 
-static struct pinctrl_ops owl_pinctrl_ops = {
+static const struct pinctrl_ops owl_pinctrl_ops = {
        .get_groups_count = owl_get_groups_count,
        .get_group_name = owl_get_group_name,
        .get_group_pins = owl_get_group_pins,
@@ -212,7 +212,7 @@ static int owl_set_mux(struct pinctrl_dev *pctrldev,
        return 0;
 }
 
-static struct pinmux_ops owl_pinmux_ops = {
+static const struct pinmux_ops owl_pinmux_ops = {
        .get_functions_count = owl_get_funcs_count,
        .get_function_name = owl_get_func_name,
        .get_function_groups = owl_get_func_groups,