* [PATCH 1/1] clk: exynos5420: Staticize local variables @ 2014-05-21 11:58 Sachin Kamat 2014-05-26 3:39 ` Sachin Kamat 2014-05-28 8:01 ` Sachin Kamat 0 siblings, 2 replies; 5+ messages in thread From: Sachin Kamat @ 2014-05-21 11:58 UTC (permalink / raw) To: linux-arm-kernel Local symbols are made static. Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> --- drivers/clk/samsung/clk-exynos5420.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c index 9d7d7eed03fd..41b507baf5d6 100644 --- a/drivers/clk/samsung/clk-exynos5420.c +++ b/drivers/clk/samsung/clk-exynos5420.c @@ -474,7 +474,7 @@ static struct samsung_fixed_factor_clock FFACTOR(0, "ff_dout_spll2", "mout_sclk_spll", 1, 2, 0), }; -struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { +static struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { MUX(0, "mout_aclk400_isp", mout_group3_5800_p, SRC_TOP0, 0, 3), MUX(0, "mout_aclk400_mscl", mout_group3_5800_p, SRC_TOP0, 4, 3), MUX(0, "mout_aclk400_wcore", mout_group2_5800_p, SRC_TOP0, 16, 3), @@ -523,7 +523,7 @@ struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { MUX(0, "mout_fimd1", mout_group2_p, SRC_DISP10, 4, 3), }; -struct samsung_div_clock exynos5800_div_clks[] __initdata = { +static struct samsung_div_clock exynos5800_div_clks[] __initdata = { DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore", DIV_TOP0, 16, 3), DIV(0, "dout_aclk550_cam", "mout_aclk550_cam", @@ -539,14 +539,14 @@ struct samsung_div_clock exynos5800_div_clks[] __initdata = { DIV(0, "dout_sclk_sw", "sclk_spll", DIV_TOP9, 24, 6), }; -struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { +static struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { GATE(CLK_ACLK550_CAM, "aclk550_cam", "mout_user_aclk550_cam", GATE_BUS_TOP, 24, 0, 0), GATE(CLK_ACLK432_SCALER, "aclk432_scaler", "mout_user_aclk432_scaler", GATE_BUS_TOP, 27, 0, 0), }; -struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { +static struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { MUX(0, "sclk_bpll", mout_bpll_p, TOP_SPARE2, 0, 1), MUX(0, "mout_aclk400_wcore_bpll", mout_aclk400_wcore_bpll_p, TOP_SPARE2, 4, 1), @@ -576,7 +576,7 @@ struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { MUX(0, "mout_fimd1", mout_group3_p, SRC_DISP10, 4, 1), }; -struct samsung_div_clock exynos5420_div_clks[] __initdata = { +static struct samsung_div_clock exynos5420_div_clks[] __initdata = { DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore_bpll", DIV_TOP0, 16, 3), }; -- 1.7.9.5 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* [PATCH 1/1] clk: exynos5420: Staticize local variables 2014-05-21 11:58 [PATCH 1/1] clk: exynos5420: Staticize local variables Sachin Kamat @ 2014-05-26 3:39 ` Sachin Kamat 2014-05-28 8:01 ` Sachin Kamat 1 sibling, 0 replies; 5+ messages in thread From: Sachin Kamat @ 2014-05-26 3:39 UTC (permalink / raw) To: linux-arm-kernel On 21 May 2014 17:28, Sachin Kamat <sachin.kamat@linaro.org> wrote: > Local symbols are made static. > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > --- > drivers/clk/samsung/clk-exynos5420.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c > index 9d7d7eed03fd..41b507baf5d6 100644 > --- a/drivers/clk/samsung/clk-exynos5420.c > +++ b/drivers/clk/samsung/clk-exynos5420.c > @@ -474,7 +474,7 @@ static struct samsung_fixed_factor_clock > FFACTOR(0, "ff_dout_spll2", "mout_sclk_spll", 1, 2, 0), > }; > > -struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { > +static struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { > MUX(0, "mout_aclk400_isp", mout_group3_5800_p, SRC_TOP0, 0, 3), > MUX(0, "mout_aclk400_mscl", mout_group3_5800_p, SRC_TOP0, 4, 3), > MUX(0, "mout_aclk400_wcore", mout_group2_5800_p, SRC_TOP0, 16, 3), > @@ -523,7 +523,7 @@ struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { > MUX(0, "mout_fimd1", mout_group2_p, SRC_DISP10, 4, 3), > }; > > -struct samsung_div_clock exynos5800_div_clks[] __initdata = { > +static struct samsung_div_clock exynos5800_div_clks[] __initdata = { > DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore", DIV_TOP0, 16, 3), > > DIV(0, "dout_aclk550_cam", "mout_aclk550_cam", > @@ -539,14 +539,14 @@ struct samsung_div_clock exynos5800_div_clks[] __initdata = { > DIV(0, "dout_sclk_sw", "sclk_spll", DIV_TOP9, 24, 6), > }; > > -struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { > +static struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { > GATE(CLK_ACLK550_CAM, "aclk550_cam", "mout_user_aclk550_cam", > GATE_BUS_TOP, 24, 0, 0), > GATE(CLK_ACLK432_SCALER, "aclk432_scaler", "mout_user_aclk432_scaler", > GATE_BUS_TOP, 27, 0, 0), > }; > > -struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { > +static struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { > MUX(0, "sclk_bpll", mout_bpll_p, TOP_SPARE2, 0, 1), > MUX(0, "mout_aclk400_wcore_bpll", mout_aclk400_wcore_bpll_p, > TOP_SPARE2, 4, 1), > @@ -576,7 +576,7 @@ struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { > MUX(0, "mout_fimd1", mout_group3_p, SRC_DISP10, 4, 1), > }; > > -struct samsung_div_clock exynos5420_div_clks[] __initdata = { > +static struct samsung_div_clock exynos5420_div_clks[] __initdata = { > DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore_bpll", > DIV_TOP0, 16, 3), > }; > -- > 1.7.9.5 > Gentle ping, Tomasz :) -- With warm regards, Sachin ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/1] clk: exynos5420: Staticize local variables 2014-05-21 11:58 [PATCH 1/1] clk: exynos5420: Staticize local variables Sachin Kamat 2014-05-26 3:39 ` Sachin Kamat @ 2014-05-28 8:01 ` Sachin Kamat 2014-05-28 11:02 ` Tomasz Figa 1 sibling, 1 reply; 5+ messages in thread From: Sachin Kamat @ 2014-05-28 8:01 UTC (permalink / raw) To: linux-arm-kernel On 21 May 2014 17:28, Sachin Kamat <sachin.kamat@linaro.org> wrote: > Local symbols are made static. > > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> > --- > drivers/clk/samsung/clk-exynos5420.c | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-) > > diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c > index 9d7d7eed03fd..41b507baf5d6 100644 > --- a/drivers/clk/samsung/clk-exynos5420.c > +++ b/drivers/clk/samsung/clk-exynos5420.c > @@ -474,7 +474,7 @@ static struct samsung_fixed_factor_clock > FFACTOR(0, "ff_dout_spll2", "mout_sclk_spll", 1, 2, 0), > }; > > -struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { > +static struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { > MUX(0, "mout_aclk400_isp", mout_group3_5800_p, SRC_TOP0, 0, 3), > MUX(0, "mout_aclk400_mscl", mout_group3_5800_p, SRC_TOP0, 4, 3), > MUX(0, "mout_aclk400_wcore", mout_group2_5800_p, SRC_TOP0, 16, 3), > @@ -523,7 +523,7 @@ struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { > MUX(0, "mout_fimd1", mout_group2_p, SRC_DISP10, 4, 3), > }; > > -struct samsung_div_clock exynos5800_div_clks[] __initdata = { > +static struct samsung_div_clock exynos5800_div_clks[] __initdata = { > DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore", DIV_TOP0, 16, 3), > > DIV(0, "dout_aclk550_cam", "mout_aclk550_cam", > @@ -539,14 +539,14 @@ struct samsung_div_clock exynos5800_div_clks[] __initdata = { > DIV(0, "dout_sclk_sw", "sclk_spll", DIV_TOP9, 24, 6), > }; > > -struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { > +static struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { > GATE(CLK_ACLK550_CAM, "aclk550_cam", "mout_user_aclk550_cam", > GATE_BUS_TOP, 24, 0, 0), > GATE(CLK_ACLK432_SCALER, "aclk432_scaler", "mout_user_aclk432_scaler", > GATE_BUS_TOP, 27, 0, 0), > }; > > -struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { > +static struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { > MUX(0, "sclk_bpll", mout_bpll_p, TOP_SPARE2, 0, 1), > MUX(0, "mout_aclk400_wcore_bpll", mout_aclk400_wcore_bpll_p, > TOP_SPARE2, 4, 1), > @@ -576,7 +576,7 @@ struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { > MUX(0, "mout_fimd1", mout_group3_p, SRC_DISP10, 4, 1), > }; > > -struct samsung_div_clock exynos5420_div_clks[] __initdata = { > +static struct samsung_div_clock exynos5420_div_clks[] __initdata = { > DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore_bpll", > DIV_TOP0, 16, 3), > }; > -- Mike, Tomasz is probably on vacation. Can you please take this patch? -- With warm regards, Sachin ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/1] clk: exynos5420: Staticize local variables 2014-05-28 8:01 ` Sachin Kamat @ 2014-05-28 11:02 ` Tomasz Figa 2014-05-28 11:18 ` Sachin Kamat 0 siblings, 1 reply; 5+ messages in thread From: Tomasz Figa @ 2014-05-28 11:02 UTC (permalink / raw) To: linux-arm-kernel Hi Sachin, On 28.05.2014 10:01, Sachin Kamat wrote: > On 21 May 2014 17:28, Sachin Kamat <sachin.kamat@linaro.org> wrote: >> Local symbols are made static. >> >> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> >> --- >> drivers/clk/samsung/clk-exynos5420.c | 10 +++++----- >> 1 file changed, 5 insertions(+), 5 deletions(-) >> >> diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c >> index 9d7d7eed03fd..41b507baf5d6 100644 >> --- a/drivers/clk/samsung/clk-exynos5420.c >> +++ b/drivers/clk/samsung/clk-exynos5420.c >> @@ -474,7 +474,7 @@ static struct samsung_fixed_factor_clock >> FFACTOR(0, "ff_dout_spll2", "mout_sclk_spll", 1, 2, 0), >> }; >> >> -struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { >> +static struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { >> MUX(0, "mout_aclk400_isp", mout_group3_5800_p, SRC_TOP0, 0, 3), >> MUX(0, "mout_aclk400_mscl", mout_group3_5800_p, SRC_TOP0, 4, 3), >> MUX(0, "mout_aclk400_wcore", mout_group2_5800_p, SRC_TOP0, 16, 3), >> @@ -523,7 +523,7 @@ struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { >> MUX(0, "mout_fimd1", mout_group2_p, SRC_DISP10, 4, 3), >> }; >> >> -struct samsung_div_clock exynos5800_div_clks[] __initdata = { >> +static struct samsung_div_clock exynos5800_div_clks[] __initdata = { >> DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore", DIV_TOP0, 16, 3), >> >> DIV(0, "dout_aclk550_cam", "mout_aclk550_cam", >> @@ -539,14 +539,14 @@ struct samsung_div_clock exynos5800_div_clks[] __initdata = { >> DIV(0, "dout_sclk_sw", "sclk_spll", DIV_TOP9, 24, 6), >> }; >> >> -struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { >> +static struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { >> GATE(CLK_ACLK550_CAM, "aclk550_cam", "mout_user_aclk550_cam", >> GATE_BUS_TOP, 24, 0, 0), >> GATE(CLK_ACLK432_SCALER, "aclk432_scaler", "mout_user_aclk432_scaler", >> GATE_BUS_TOP, 27, 0, 0), >> }; >> >> -struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { >> +static struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { >> MUX(0, "sclk_bpll", mout_bpll_p, TOP_SPARE2, 0, 1), >> MUX(0, "mout_aclk400_wcore_bpll", mout_aclk400_wcore_bpll_p, >> TOP_SPARE2, 4, 1), >> @@ -576,7 +576,7 @@ struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { >> MUX(0, "mout_fimd1", mout_group3_p, SRC_DISP10, 4, 1), >> }; >> >> -struct samsung_div_clock exynos5420_div_clks[] __initdata = { >> +static struct samsung_div_clock exynos5420_div_clks[] __initdata = { >> DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore_bpll", >> DIV_TOP0, 16, 3), >> }; >> -- > > Mike, > > Tomasz is probably on vacation. Can you please take this patch? > Sorry for late reply. Yes, I'm on vacation right now until 9.06, so I'm going to be less active through this period of time. As for this patch, I'm not sure if it will apply to Mike's tree, as it seems to depend on patches that went through Kukjin's tree. Since this isn't anything critical, I'd suggest deferring this for next release, so I can pick this patch up directly after 3.16-rc1 is released. In case you want to proceed with it now, feel free to add my Acked-by. Best regards, Tomasz ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/1] clk: exynos5420: Staticize local variables 2014-05-28 11:02 ` Tomasz Figa @ 2014-05-28 11:18 ` Sachin Kamat 0 siblings, 0 replies; 5+ messages in thread From: Sachin Kamat @ 2014-05-28 11:18 UTC (permalink / raw) To: linux-arm-kernel Hi Tomasz, On 28 May 2014 16:32, Tomasz Figa <tomasz.figa@gmail.com> wrote: > Hi Sachin, > > On 28.05.2014 10:01, Sachin Kamat wrote: >> On 21 May 2014 17:28, Sachin Kamat <sachin.kamat@linaro.org> wrote: >>> Local symbols are made static. >>> >>> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org> >>> --- >>> drivers/clk/samsung/clk-exynos5420.c | 10 +++++----- >>> 1 file changed, 5 insertions(+), 5 deletions(-) >>> >>> diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c >>> index 9d7d7eed03fd..41b507baf5d6 100644 >>> --- a/drivers/clk/samsung/clk-exynos5420.c >>> +++ b/drivers/clk/samsung/clk-exynos5420.c >>> @@ -474,7 +474,7 @@ static struct samsung_fixed_factor_clock >>> FFACTOR(0, "ff_dout_spll2", "mout_sclk_spll", 1, 2, 0), >>> }; >>> >>> -struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { >>> +static struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { >>> MUX(0, "mout_aclk400_isp", mout_group3_5800_p, SRC_TOP0, 0, 3), >>> MUX(0, "mout_aclk400_mscl", mout_group3_5800_p, SRC_TOP0, 4, 3), >>> MUX(0, "mout_aclk400_wcore", mout_group2_5800_p, SRC_TOP0, 16, 3), >>> @@ -523,7 +523,7 @@ struct samsung_mux_clock exynos5800_mux_clks[] __initdata = { >>> MUX(0, "mout_fimd1", mout_group2_p, SRC_DISP10, 4, 3), >>> }; >>> >>> -struct samsung_div_clock exynos5800_div_clks[] __initdata = { >>> +static struct samsung_div_clock exynos5800_div_clks[] __initdata = { >>> DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore", DIV_TOP0, 16, 3), >>> >>> DIV(0, "dout_aclk550_cam", "mout_aclk550_cam", >>> @@ -539,14 +539,14 @@ struct samsung_div_clock exynos5800_div_clks[] __initdata = { >>> DIV(0, "dout_sclk_sw", "sclk_spll", DIV_TOP9, 24, 6), >>> }; >>> >>> -struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { >>> +static struct samsung_gate_clock exynos5800_gate_clks[] __initdata = { >>> GATE(CLK_ACLK550_CAM, "aclk550_cam", "mout_user_aclk550_cam", >>> GATE_BUS_TOP, 24, 0, 0), >>> GATE(CLK_ACLK432_SCALER, "aclk432_scaler", "mout_user_aclk432_scaler", >>> GATE_BUS_TOP, 27, 0, 0), >>> }; >>> >>> -struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { >>> +static struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { >>> MUX(0, "sclk_bpll", mout_bpll_p, TOP_SPARE2, 0, 1), >>> MUX(0, "mout_aclk400_wcore_bpll", mout_aclk400_wcore_bpll_p, >>> TOP_SPARE2, 4, 1), >>> @@ -576,7 +576,7 @@ struct samsung_mux_clock exynos5420_mux_clks[] __initdata = { >>> MUX(0, "mout_fimd1", mout_group3_p, SRC_DISP10, 4, 1), >>> }; >>> >>> -struct samsung_div_clock exynos5420_div_clks[] __initdata = { >>> +static struct samsung_div_clock exynos5420_div_clks[] __initdata = { >>> DIV(0, "dout_aclk400_wcore", "mout_aclk400_wcore_bpll", >>> DIV_TOP0, 16, 3), >>> }; >>> -- >> >> Mike, >> >> Tomasz is probably on vacation. Can you please take this patch? >> > > Sorry for late reply. Yes, I'm on vacation right now until 9.06, so I'm > going to be less active through this period of time. No worries. > As for this patch, I'm not sure if it will apply to Mike's tree, as it > seems to depend on patches that went through Kukjin's tree. Since this > isn't anything critical, I'd suggest deferring this for next release, so > I can pick this patch up directly after 3.16-rc1 is released. > > In case you want to proceed with it now, feel free to add my Acked-by. Thanks Tomasz. Kukjin, If you still can take it, please do so with Tomasz's ack. Else he will handle it after the merge window. -- With warm regards, Sachin ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-05-28 11:18 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-05-21 11:58 [PATCH 1/1] clk: exynos5420: Staticize local variables Sachin Kamat 2014-05-26 3:39 ` Sachin Kamat 2014-05-28 8:01 ` Sachin Kamat 2014-05-28 11:02 ` Tomasz Figa 2014-05-28 11:18 ` Sachin Kamat
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).