2017-06-02 17:58:36

by Stephen Boyd

[permalink] [raw]
Subject: [PATCH] clk: meson-gxbb: Add const to some parent name arrays

These can be marked as const * const.

Cc: Neil Armstrong <[email protected]>
Cc: Jerome Brunet <[email protected]>
Signed-off-by: Stephen Boyd <[email protected]>
---
drivers/clk/meson/gxbb.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/clk/meson/gxbb.c b/drivers/clk/meson/gxbb.c
index 2919a0e044a5..36c0e455437e 100644
--- a/drivers/clk/meson/gxbb.c
+++ b/drivers/clk/meson/gxbb.c
@@ -701,7 +701,7 @@ static struct clk_gate gxbb_sar_adc_clk = {
*/

static u32 mux_table_mali_0_1[] = {0, 1, 2, 3, 4, 5, 6, 7};
-static const char *gxbb_mali_0_1_parent_names[] = {
+static const char * const gxbb_mali_0_1_parent_names[] = {
"xtal", "gp0_pll", "mpll2", "mpll1", "fclk_div7",
"fclk_div4", "fclk_div3", "fclk_div5"
};
@@ -801,7 +801,7 @@ static struct clk_gate gxbb_mali_1 = {
};

static u32 mux_table_mali[] = {0, 1};
-static const char *gxbb_mali_parent_names[] = {
+static const char * const gxbb_mali_parent_names[] = {
"mali_0", "mali_1"
};

@@ -953,7 +953,7 @@ static struct clk_gate gxbb_32k_clk = {
},
};

-static const char *gxbb_32k_clk_parent_names[] = {
+static const char * const gxbb_32k_clk_parent_names[] = {
"xtal", "cts_slow_oscin", "fclk_div3", "fclk_div5"
};

--
The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum,
a Linux Foundation Collaborative Project


2017-06-02 22:40:53

by Stephen Boyd

[permalink] [raw]
Subject: Re: [PATCH] clk: meson-gxbb: Add const to some parent name arrays

On 06/02, Stephen Boyd wrote:
> These can be marked as const * const.
>
> Cc: Neil Armstrong <[email protected]>
> Cc: Jerome Brunet <[email protected]>
> Signed-off-by: Stephen Boyd <[email protected]>
> ---

Applied to clk-next

--
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
a Linux Foundation Collaborative Project

2017-06-08 13:17:12

by Jerome Brunet

[permalink] [raw]
Subject: Re: [PATCH] clk: meson-gxbb: Add const to some parent name arrays

On Fri, 2017-06-02 at 15:40 -0700, Stephen Boyd wrote:
> On 06/02, Stephen Boyd wrote:
> > These can be marked as const * const.
> >
> > Cc: Neil Armstrong <[email protected]>
> > Cc: Jerome Brunet <[email protected]>
> > Signed-off-by: Stephen Boyd <[email protected]>
> > ---
>
> Applied to clk-next
>

Thx Stephen. Neil And I were discussing this particular warning but you've been
quicker than us to send the fix ;)

Cheers
Jerome