clk: pxa: Constify static struct clk_ops
authorRikard Falkeborn <rikard.falkeborn@gmail.com>
Tue, 22 Sep 2020 18:47:15 +0000 (20:47 +0200)
committerStephen Boyd <sboyd@kernel.org>
Wed, 14 Oct 2020 02:49:11 +0000 (19:49 -0700)
Constify a couple of static struct clk_ops that are not modified. Their
only usage is inside the macros and their address is passed to
clk_register_composite() which takes pointers to const struct clk_ops.
This allows the compiler to put them in read-only memory.

Signed-off-by: Rikard Falkeborn <rikard.falkeborn@gmail.com>
Link: https://lore.kernel.org/r/20200922184715.1854-1-rikard.falkeborn@gmail.com
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/pxa/clk-pxa.h

index f131d28..5768e0f 100644 (file)
 #define MUX_RO_RATE_RO_OPS(name, clk_name)                     \
        static struct clk_hw name ## _mux_hw;                   \
        static struct clk_hw name ## _rate_hw;                  \
-       static struct clk_ops name ## _mux_ops = {              \
+       static const struct clk_ops name ## _mux_ops = {        \
                .get_parent = name ## _get_parent,              \
                .set_parent = dummy_clk_set_parent,             \
        };                                                      \
-       static struct clk_ops name ## _rate_ops = {             \
+       static const struct clk_ops name ## _rate_ops = {       \
                .recalc_rate = name ## _get_rate,               \
        };                                                      \
        static struct clk * __init clk_register_ ## name(void)  \
@@ -38,7 +38,7 @@
 
 #define RATE_RO_OPS(name, clk_name)                            \
        static struct clk_hw name ## _rate_hw;                  \
-       static const struct clk_ops name ## _rate_ops = {               \
+       static const struct clk_ops name ## _rate_ops = {       \
                .recalc_rate = name ## _get_rate,               \
        };                                                      \
        static struct clk * __init clk_register_ ## name(void)  \
@@ -53,7 +53,7 @@
 
 #define RATE_OPS(name, clk_name)                               \
        static struct clk_hw name ## _rate_hw;                  \
-       static struct clk_ops name ## _rate_ops = {             \
+       static const struct clk_ops name ## _rate_ops = {       \
                .recalc_rate = name ## _get_rate,               \
                .set_rate = name ## _set_rate,                  \
                .determine_rate = name ## _determine_rate,      \