OSDN Git Service

clk: pistachio: Fix initconst confusion
authorAndi Kleen <ak@linux.intel.com>
Tue, 20 Sep 2022 05:58:38 +0000 (07:58 +0200)
committerStephen Boyd <sboyd@kernel.org>
Thu, 29 Sep 2022 01:35:45 +0000 (18:35 -0700)
A variable pointing to const isn't const itself. It has to contain
"const" keyword after "*" too. So to keep it in __initconst (and not
mark properly as __initdata), add the "const" keyword exactly there.

Note we need to update struct pistachio_mux too. On the other hand, the
clk core already counts with "const char *const" already.

[js] more explanatory commit message.

Cc: Michael Turquette <mturquette@baylibre.com>
Cc: Stephen Boyd <sboyd@kernel.org>
Cc: linux-clk@vger.kernel.org
Cc: Martin Liska <mliska@suse.cz>
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Link: https://lore.kernel.org/r/20220920055838.22637-1-jslaby@suse.cz
Signed-off-by: Stephen Boyd <sboyd@kernel.org>
drivers/clk/pistachio/clk.h

index f9c31e3..2f4ffbd 100644 (file)
@@ -31,10 +31,10 @@ struct pistachio_mux {
        unsigned int shift;
        unsigned int num_parents;
        const char *name;
-       const char **parents;
+       const char *const *parents;
 };
 
-#define PNAME(x) static const char *x[] __initconst
+#define PNAME(x) static const char *const x[] __initconst
 
 #define MUX(_id, _name, _pnames, _reg, _shift)                 \
        {                                                       \