From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kukjin Kim Subject: Re: [PATCH 1/3] clk: exynos4: Staticize local symbols Date: Wed, 19 Jun 2013 23:24:00 +0900 Message-ID: <51C1BF00.3060805@samsung.com> References: <1366106719-26342-1-git-send-email-sachin.kamat@linaro.org> <08f401ce61e6$2ecc33f0$8c649bd0$%kim@samsung.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mail-pb0-f49.google.com ([209.85.160.49]:50322 "EHLO mail-pb0-f49.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756740Ab3FSOYF (ORCPT ); Wed, 19 Jun 2013 10:24:05 -0400 Received: by mail-pb0-f49.google.com with SMTP id jt11so5140373pbb.36 for ; Wed, 19 Jun 2013 07:24:04 -0700 (PDT) In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: Sachin Kamat Cc: Kukjin Kim , Mike Turquette , linux-samsung-soc@vger.kernel.org On 06/17/13 12:14, Sachin Kamat wrote: > On 13 June 2013 21:06, Sachin Kamat wrote: >> On 11 June 2013 09:46, Sachin Kamat wrote: >>> On 5 June 2013 17:44, Kukjin Kim wrote: >>>> Sachin Kamat wrote: >>>>> >>>>> These symbols are used only in this file and hence should be >>>>> static. >>>>> >>>>> Signed-off-by: Sachin Kamat >>>>> --- >>>>> drivers/clk/samsung/clk-exynos4.c | 26 ++++++++++++++------------ >>>>> 1 file changed, 14 insertions(+), 12 deletions(-) >>>>> >>>>> diff --git a/drivers/clk/samsung/clk-exynos4.c b/drivers/clk/samsung/clk- >>>>> exynos4.c >>>>> index 7104669..26f2a85 100644 >>>>> --- a/drivers/clk/samsung/clk-exynos4.c >>>>> +++ b/drivers/clk/samsung/clk-exynos4.c >>>>> @@ -339,24 +339,26 @@ PNAME(mout_user_aclk200_p4x12) = {"fin_pll", >>>>> "div_aclk200", }; >>>>> PNAME(mout_user_aclk266_gps_p4x12) = {"fin_pll", "div_aclk266_gps", }; >>>>> >>>>> /* fixed rate clocks generated outside the soc */ >>>>> -struct samsung_fixed_rate_clock exynos4_fixed_rate_ext_clks[] __initdata >>>>> = { >>>>> +static struct >>>>> +samsung_fixed_rate_clock exynos4_fixed_rate_ext_clks[] __initdata = { >>>> >>>> Any reason to use double lines? >>> >>> This is one of the ways to avoid exceeding 80 column limit. >>> >>>> >>>>> FRATE(xxti, "xxti", NULL, CLK_IS_ROOT, 0), >>>>> FRATE(xusbxti, "xusbxti", NULL, CLK_IS_ROOT, 0), >>>>> }; >>>> >>>> [...] >>>> >>>>> -struct samsung_fixed_rate_clock exynos4210_fixed_rate_clks[] __initdata = >>>>> { >>>>> +static struct >>>>> +samsung_fixed_rate_clock exynos4210_fixed_rate_clks[] __initdata = { >>>> >>>> Same as above. >>>> >>>>> FRATE(none, "sclk_usbphy1", NULL, CLK_IS_ROOT, 48000000), >>>>> }; >>>>> >>>> [...] >>>> >>>> Others look good to me, >>>> Acked-by: Kukjin Kim >>>> >>>> Mike, please pick this into the clk tree if you're ok. >>>> >>> >>> Mike, >>> If you are taking this through your tree, please also take the below >>> patch [1] as it is dependent on this series. >>> [1] https://patchwork.kernel.org/patch/2469891/ >> >> >> Ping Mike.. > > > Kukjin, > > Haven't heard back from Mike regarding this. Can you please take this > through your tree? > OK, I will. BTW, Sachin, let me modify the line added static for regarding 80 columns...I think, just one like is better... Anyway, Mike, if you have any objection, please kindly let me know before pulling into arm-soc tree. Thanks, - Kukjin