[PATCH 0/4 v2] Miscellaneous s3c64xx fixes.
From: Tomasz Figa <hidden>
Date: 2011-08-21 23:48:25
(no body)
Revision v2 of 7 in this series.
8 messages, 2 authors, 2011-08-25 · open the first message on its own page
From: Tomasz Figa <hidden>
Date: 2011-08-21 23:48:25
(no body)
From: Tomasz Figa <hidden>
Date: 2011-08-21 23:48:57
(no body)
From: Tomasz Figa <hidden>
Date: 2011-08-21 23:49:24
(no body)
From: Tomasz Figa <hidden>
Date: 2011-08-21 23:49:50
(no body)
From: Tomasz Figa <hidden>
Date: 2011-08-21 23:50:34
(no body)
From: Kukjin Kim <hidden>
Date: 2011-08-23 02:42:05
Tomasz Figa wrote:
From 4ebfc49563b24eea6a689d0323bb3524b1bcbabf Mon Sep 17 00:00:00 2001 From: Tomasz Figa <redacted> Date: Fri, 19 Aug 2011 11:54:31 +0200 Subject: [PATCH 4/4 v2] ARM: s3c64xx: Add support for synchronous clock operation. Some boards based on S3C6410 use synchronous clocking, which means that HCLKx2 and other system clocks are generated from APLL instead of MPLL. This patch adds support for such boards, by calculating hclk2 depending on the status of S3C_OTHERS_SYNCMUXSEL bit in S3C64XX_OTHERS registers. Signed-off-by: Tomasz Figa <redacted> --- arch/arm/mach-s3c64xx/clock.c | 8 +++++++- arch/arm/mach-s3c64xx/include/mach/regs-sys.h | 3 ++- 2 files changed, 9 insertions(+), 2 deletions(-)diff --git a/arch/arm/mach-s3c64xx/clock.c b/arch/arm/mach-s3c64xx/clock.c index fdfc4d5..a7dab43 100644 --- a/arch/arm/mach-s3c64xx/clock.c +++ b/arch/arm/mach-s3c64xx/clock.c@@ -780,7 +780,13 @@ void __init_or_cpufreq s3c6400_setup_clocks(void) printk(KERN_INFO "S3C64XX: PLL settings, A=%ld, M=%ld, E=%ld\n", apll, mpll, epll); - hclk2 = mpll / GET_DIV(clkdiv0, S3C6400_CLKDIV0_HCLK2); + if(__raw_readl(S3C64XX_OTHERS) & S3C64XX_OTHERS_SYNCMUXSEL) + /* Synchronous mode */ + hclk2 = apll / GET_DIV(clkdiv0, S3C6400_CLKDIV0_HCLK2); + else + /* Asynchronous mode */ + hclk2 = mpll / GET_DIV(clkdiv0, S3C6400_CLKDIV0_HCLK2); + hclk = hclk2 / GET_DIV(clkdiv0, S3C6400_CLKDIV0_HCLK); pclk = hclk2 / GET_DIV(clkdiv0, S3C6400_CLKDIV0_PCLK);diff --git a/arch/arm/mach-s3c64xx/include/mach/regs-sys.h
b/arch/arm/mach-
s3c64xx/include/mach/regs-sys.h index 774e0de..00c5d8b 100644--- a/arch/arm/mach-s3c64xx/include/mach/regs-sys.h +++ b/arch/arm/mach-s3c64xx/include/mach/regs-sys.h@@ -25,6 +25,7 @@ #define S3C64XX_OTHERS S3C_SYSREG(0x900) -#define S3C64XX_OTHERS_USBMASK (1 << 16) +#define S3C64XX_OTHERS_USBMASK (1 << 16)
Hi Tomasz, Please don't include other changes in this patch which is for supporting synchronous mode. If required, should be separated patch. As a note, others looks ok to me. I think, you need to re-submit only this one and I will apply your whole series after getting updated this one. Thanks. Best regards, Kgene. -- Kukjin Kim [off-list ref], Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd.
+#define S3C64XX_OTHERS_SYNCMUXSEL (1 << 6) #endif /* _PLAT_REGS_SYS_H */ -- 1.7.6
From: Tomasz Figa <hidden>
Date: 2011-08-23 10:40:44
(no body)
From: Kukjin Kim <hidden>
Date: 2011-08-25 23:33:07
Tomasz Figa wrote:
From 1518d67b93432272312a55c4602040c580f46a0c Mon Sep 17 00:00:00 2001 From: Tomasz Figa <redacted> Date: Fri, 19 Aug 2011 11:54:31 +0200 Subject: [PATCH 4/4 v3] ARM: s3c64xx: Add support for synchronous clock operation. Some boards based on S3C6410 use synchronous clocking, which means that HCLKx2 and other system clocks are generated from APLL instead of MPLL. This patch adds support for such boards, by calculating hclk2 depending on the status of S3C_OTHERS_SYNCMUXSEL bit in S3C64XX_OTHERS regist Signed-off-by: Tomasz Figa <redacted> --- arch/arm/mach-s3c64xx/clock.c | 8 +++++++- arch/arm/mach-s3c64xx/include/mach/regs-sys.h | 1 + 2 files changed, 8 insertions(+), 1 deletions(-)diff --git a/arch/arm/mach-s3c64xx/clock.c b/arch/arm/mach-s3c64xx/clock.c index fdfc4d5..a7dab43 100644 --- a/arch/arm/mach-s3c64xx/clock.c +++ b/arch/arm/mach-s3c64xx/clock.c@@ -780,7 +780,13 @@ void __init_or_cpufreq s3c6400_setup_clocks(void) printk(KERN_INFO "S3C64XX: PLL settings, A=%ld, M=%ld, E=%ld\n", apll, mpll, epll); - hclk2 = mpll / GET_DIV(clkdiv0, S3C6400_CLKDIV0_HCLK2); + if(__raw_readl(S3C64XX_OTHERS) & S3C64XX_OTHERS_SYNCMUXSEL) + /* Synchronous mode */ + hclk2 = apll / GET_DIV(clkdiv0, S3C6400_CLKDIV0_HCLK2); + else + /* Asynchronous mode */ + hclk2 = mpll / GET_DIV(clkdiv0, S3C6400_CLKDIV0_HCLK2); + hclk = hclk2 / GET_DIV(clkdiv0, S3C6400_CLKDIV0_HCLK); pclk = hclk2 / GET_DIV(clkdiv0, S3C6400_CLKDIV0_PCLK);diff --git a/arch/arm/mach-s3c64xx/include/mach/regs-sys.h
b/arch/arm/mach-
s3c64xx/include/mach/regs-sys.h index 774e0de..b91e020 100644--- a/arch/arm/mach-s3c64xx/include/mach/regs-sys.h +++ b/arch/arm/mach-s3c64xx/include/mach/regs-sys.h@@ -26,5 +26,6 @@ #define S3C64XX_OTHERS S3C_SYSREG(0x900) #define S3C64XX_OTHERS_USBMASK (1 << 16) +#define S3C64XX_OTHERS_SYNCMUXSEL (1 << 6) #endif /* _PLAT_REGS_SYS_H */ --1.7.6
OK, applied. Thanks. Best regards, Kgene. -- Kukjin Kim [off-list ref], Senior Engineer, SW Solution Development Team, Samsung Electronics Co., Ltd.