From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomasz Figa Subject: Re: [PATCH 2/2] ARM: S3C64XX: Staticly define parent clock of "camera" clock Date: Sat, 10 Nov 2012 16:01:29 +0100 Message-ID: <2352709.lKq9axvLZf@flatron> References: <509e6047.a766980a.0f91.14f6@mx.google.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Return-path: Received: from mail-ea0-f174.google.com ([209.85.215.174]:48307 "EHLO mail-ea0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751727Ab2KJPBa (ORCPT ); Sat, 10 Nov 2012 10:01:30 -0500 Received: by mail-ea0-f174.google.com with SMTP id c13so1830416eaa.19 for ; Sat, 10 Nov 2012 07:01:29 -0800 (PST) In-Reply-To: <509e6047.a766980a.0f91.14f6@mx.google.com> Sender: linux-samsung-soc-owner@vger.kernel.org List-Id: linux-samsung-soc@vger.kernel.org To: dron0gus@gmail.com Cc: linux-samsung-soc@vger.kernel.org, kgene.kim@samsung.com, linux-arm-kernel@lists.infradead.org, sylvester.nawrocki@gmail.com, Andrey Gusakov Hi Andrey, On Saturday 10 of November 2012 18:07:57 dron0gus@gmail.com wrote: > From: Andrey Gusakov > > The "camera" clock have only one parent. Define it staticly and > remove unused source clock list. > > Signed-off-by: Andrey Gusakov > --- > arch/arm/mach-s3c64xx/clock.c | 12 +----------- > 1 files changed, 1 insertions(+), 11 deletions(-) > > diff --git a/arch/arm/mach-s3c64xx/clock.c > b/arch/arm/mach-s3c64xx/clock.c index b2d08fa..19228f3 100644 > --- a/arch/arm/mach-s3c64xx/clock.c > +++ b/arch/arm/mach-s3c64xx/clock.c > @@ -677,15 +677,6 @@ static struct clksrc_sources clkset_audio2 = { > .nr_sources = ARRAY_SIZE(clkset_audio2_list), > }; > > -static struct clk *clkset_camif_list[] = { > - &clk_h2, > -}; > - > -static struct clksrc_sources clkset_camif = { > - .sources = clkset_camif_list, > - .nr_sources = ARRAY_SIZE(clkset_camif_list), > -}; > - > static struct clk *clkset_mfc_list[] = { > &clk_h2, > &clk_mout_epll.clk, > @@ -750,10 +741,9 @@ static struct clksrc_clk clksrcs[] = { > .name = "camera", > .ctrlbit = S3C_CLKCON_SCLK_CAM, > .enable = s3c64xx_sclk_ctrl, > + .parent = &clk_h2, > }, > .reg_div = { .reg = S3C_CLK_DIV0, .shift = 20, .size = 4 }, > - .reg_src = { .reg = NULL, .shift = 0, .size = 0 }, > - .sources = &clkset_camif, > }, { > .clk = { > .name = "sclk_mfc", Reviewed-by: Tomasz Figa Best regards, Tomasz Figa From mboxrd@z Thu Jan 1 00:00:00 1970 From: tomasz.figa@gmail.com (Tomasz Figa) Date: Sat, 10 Nov 2012 16:01:29 +0100 Subject: [PATCH 2/2] ARM: S3C64XX: Staticly define parent clock of "camera" clock In-Reply-To: <509e6047.a766980a.0f91.14f6@mx.google.com> References: <509e6047.a766980a.0f91.14f6@mx.google.com> Message-ID: <2352709.lKq9axvLZf@flatron> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi Andrey, On Saturday 10 of November 2012 18:07:57 dron0gus at gmail.com wrote: > From: Andrey Gusakov > > The "camera" clock have only one parent. Define it staticly and > remove unused source clock list. > > Signed-off-by: Andrey Gusakov > --- > arch/arm/mach-s3c64xx/clock.c | 12 +----------- > 1 files changed, 1 insertions(+), 11 deletions(-) > > diff --git a/arch/arm/mach-s3c64xx/clock.c > b/arch/arm/mach-s3c64xx/clock.c index b2d08fa..19228f3 100644 > --- a/arch/arm/mach-s3c64xx/clock.c > +++ b/arch/arm/mach-s3c64xx/clock.c > @@ -677,15 +677,6 @@ static struct clksrc_sources clkset_audio2 = { > .nr_sources = ARRAY_SIZE(clkset_audio2_list), > }; > > -static struct clk *clkset_camif_list[] = { > - &clk_h2, > -}; > - > -static struct clksrc_sources clkset_camif = { > - .sources = clkset_camif_list, > - .nr_sources = ARRAY_SIZE(clkset_camif_list), > -}; > - > static struct clk *clkset_mfc_list[] = { > &clk_h2, > &clk_mout_epll.clk, > @@ -750,10 +741,9 @@ static struct clksrc_clk clksrcs[] = { > .name = "camera", > .ctrlbit = S3C_CLKCON_SCLK_CAM, > .enable = s3c64xx_sclk_ctrl, > + .parent = &clk_h2, > }, > .reg_div = { .reg = S3C_CLK_DIV0, .shift = 20, .size = 4 }, > - .reg_src = { .reg = NULL, .shift = 0, .size = 0 }, > - .sources = &clkset_camif, > }, { > .clk = { > .name = "sclk_mfc", Reviewed-by: Tomasz Figa Best regards, Tomasz Figa