linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/2] ARM: S3C64XX: Remove duplicated camera clock
@ 2012-11-10 14:07 dron0gus at gmail.com
  2012-11-10 15:00 ` Tomasz Figa
  2012-11-10 21:23 ` Sylwester Nawrocki
  0 siblings, 2 replies; 4+ messages in thread
From: dron0gus at gmail.com @ 2012-11-10 14:07 UTC (permalink / raw)
  To: linux-arm-kernel

From: Andrey Gusakov <dron_gus@mail.ru>

Camera clock defined two times. One in init_clocks_off array with
"cam" name, second in clksrcs array with "camera" name. Leave
second definition because clock have divider.

Signed-off-by: Andrey Gusakov <dron0gus@gmail.com>
---
 arch/arm/mach-s3c64xx/clock.c |    4 ----
 1 files changed, 0 insertions(+), 4 deletions(-)

diff --git a/arch/arm/mach-s3c64xx/clock.c b/arch/arm/mach-s3c64xx/clock.c
index cacbe45..b2d08fa 100644
--- a/arch/arm/mach-s3c64xx/clock.c
+++ b/arch/arm/mach-s3c64xx/clock.c
@@ -319,10 +319,6 @@ static struct clk init_clocks_off[] = {
 		.enable		= s3c64xx_sclk_ctrl,
 		.ctrlbit	= S3C_CLKCON_SCLK_MFC,
 	}, {
-		.name		= "cam",
-		.enable		= s3c64xx_sclk_ctrl,
-		.ctrlbit	= S3C_CLKCON_SCLK_CAM,
-	}, {
 		.name		= "sclk_jpeg",
 		.enable		= s3c64xx_sclk_ctrl,
 		.ctrlbit	= S3C_CLKCON_SCLK_JPEG,
-- 
1.7.0.4

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* [PATCH 1/2] ARM: S3C64XX: Remove duplicated camera clock
  2012-11-10 14:07 [PATCH 1/2] ARM: S3C64XX: Remove duplicated camera clock dron0gus at gmail.com
@ 2012-11-10 15:00 ` Tomasz Figa
  2012-11-10 21:23 ` Sylwester Nawrocki
  1 sibling, 0 replies; 4+ messages in thread
From: Tomasz Figa @ 2012-11-10 15:00 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Andrey,

On Saturday 10 of November 2012 18:07:56 dron0gus at gmail.com wrote:
> From: Andrey Gusakov <dron_gus@mail.ru>
> 
> Camera clock defined two times. One in init_clocks_off array with
> "cam" name, second in clksrcs array with "camera" name. Leave
> second definition because clock have divider.
> 
> Signed-off-by: Andrey Gusakov <dron0gus@gmail.com>
> ---
>  arch/arm/mach-s3c64xx/clock.c |    4 ----
>  1 files changed, 0 insertions(+), 4 deletions(-)
> 
> diff --git a/arch/arm/mach-s3c64xx/clock.c
> b/arch/arm/mach-s3c64xx/clock.c index cacbe45..b2d08fa 100644
> --- a/arch/arm/mach-s3c64xx/clock.c
> +++ b/arch/arm/mach-s3c64xx/clock.c
> @@ -319,10 +319,6 @@ static struct clk init_clocks_off[] = {
>  		.enable		= s3c64xx_sclk_ctrl,
>  		.ctrlbit	= S3C_CLKCON_SCLK_MFC,
>  	}, {
> -		.name		= "cam",
> -		.enable		= s3c64xx_sclk_ctrl,
> -		.ctrlbit	= S3C_CLKCON_SCLK_CAM,
> -	}, {
>  		.name		= "sclk_jpeg",
>  		.enable		= s3c64xx_sclk_ctrl,
>  		.ctrlbit	= S3C_CLKCON_SCLK_JPEG,

Reviewed-by: Tomasz Figa <tomasz.figa@gmail.com>

Best regards,
Tomasz Figa

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] ARM: S3C64XX: Remove duplicated camera clock
  2012-11-10 14:07 [PATCH 1/2] ARM: S3C64XX: Remove duplicated camera clock dron0gus at gmail.com
  2012-11-10 15:00 ` Tomasz Figa
@ 2012-11-10 21:23 ` Sylwester Nawrocki
  2012-11-12  5:02   ` Kukjin Kim
  1 sibling, 1 reply; 4+ messages in thread
From: Sylwester Nawrocki @ 2012-11-10 21:23 UTC (permalink / raw)
  To: linux-arm-kernel

On 11/10/2012 03:07 PM, dron0gus at gmail.com wrote:
> From: Andrey Gusakov<dron_gus@mail.ru>
>
> Camera clock defined two times. One in init_clocks_off array with
> "cam" name, second in clksrcs array with "camera" name. Leave
> second definition because clock have divider.
>
> Signed-off-by: Andrey Gusakov<dron0gus@gmail.com>

Reviewed-by: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH 1/2] ARM: S3C64XX: Remove duplicated camera clock
  2012-11-10 21:23 ` Sylwester Nawrocki
@ 2012-11-12  5:02   ` Kukjin Kim
  0 siblings, 0 replies; 4+ messages in thread
From: Kukjin Kim @ 2012-11-12  5:02 UTC (permalink / raw)
  To: linux-arm-kernel

Sylwester Nawrocki wrote:
> 
> On 11/10/2012 03:07 PM, dron0gus at gmail.com wrote:
> > From: Andrey Gusakov<dron_gus@mail.ru>
> >
> > Camera clock defined two times. One in init_clocks_off array with
> > "cam" name, second in clksrcs array with "camera" name. Leave
> > second definition because clock have divider.
> >
> > Signed-off-by: Andrey Gusakov<dron0gus@gmail.com>
> 
> Reviewed-by: Sylwester Nawrocki <sylvester.nawrocki@gmail.com>

Applied, thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2012-11-12  5:02 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-11-10 14:07 [PATCH 1/2] ARM: S3C64XX: Remove duplicated camera clock dron0gus at gmail.com
2012-11-10 15:00 ` Tomasz Figa
2012-11-10 21:23 ` Sylwester Nawrocki
2012-11-12  5:02   ` Kukjin Kim

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).