linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH V2 0/3] Add PMU and S2R support for exynos5420
@ 2014-03-27  6:13 Vikas Sajjan
  2014-03-27  6:13 ` [PATCH V2 1/3] ARM: EXYNOS5: Add PMU settings " Vikas Sajjan
                   ` (3 more replies)
  0 siblings, 4 replies; 12+ messages in thread
From: Vikas Sajjan @ 2014-03-27  6:13 UTC (permalink / raw)
  To: linux-arm-kernel

rebased on kgene tree for-next branch
https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/log/?h=for-next

changes since v1:
	- Addressed comments from Tomasz figa.
	- restructured/consolidated as per Tomasz figa's PM consolidations for exynos 

Abhilash Kesavan (1):
  ARM: dts: Add node for GPIO keys on SMDK5420

Vikas Sajjan (2):
  ARM: EXYNOS5: Add PMU settings for exynos5420
  ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420

 arch/arm/boot/dts/exynos5420-smdk5420.dts    |   11 +
 arch/arm/mach-exynos/common.h                |   10 +
 arch/arm/mach-exynos/pm.c                    |  148 +++++++++++--
 arch/arm/mach-exynos/pmu.c                   |  307 ++++++++++++++++++++++++++
 arch/arm/mach-exynos/regs-pmu.h              |  230 +++++++++++++++++++
 arch/arm/plat-samsung/include/plat/map-s5p.h |    2 +
 drivers/clk/samsung/clk-exynos5420.c         |   32 +++
 7 files changed, 722 insertions(+), 18 deletions(-)

-- 
1.7.9.5

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

* [PATCH V2 1/3] ARM: EXYNOS5: Add PMU settings for exynos5420
  2014-03-27  6:13 [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
@ 2014-03-27  6:13 ` Vikas Sajjan
  2014-04-15 18:57   ` Tomasz Figa
  2014-03-27  6:13 ` [PATCH V2 2/3] ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420 Vikas Sajjan
                   ` (2 subsequent siblings)
  3 siblings, 1 reply; 12+ messages in thread
From: Vikas Sajjan @ 2014-03-27  6:13 UTC (permalink / raw)
  To: linux-arm-kernel

Add intial PMU settings for exynos5420. This is required for
future S2R and Switching support.

Signed-off-by: Thomas Abraham <thomas.ab@samsung.com>
Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
---
 arch/arm/mach-exynos/common.h   |   10 ++
 arch/arm/mach-exynos/pmu.c      |  307 +++++++++++++++++++++++++++++++++++++++
 arch/arm/mach-exynos/regs-pmu.h |  228 +++++++++++++++++++++++++++++
 3 files changed, 545 insertions(+)

diff --git a/arch/arm/mach-exynos/common.h b/arch/arm/mach-exynos/common.h
index 9ef3f83..347afc2 100644
--- a/arch/arm/mach-exynos/common.h
+++ b/arch/arm/mach-exynos/common.h
@@ -15,6 +15,16 @@
 #include <linux/reboot.h>
 #include <linux/of.h>
 
+#define EXYNOS5420_USE_STANDBY_WFI_ALL	(EXYNOS5420_ARM_USE_STANDBY_WFI0  \
+					 | EXYNOS5420_ARM_USE_STANDBY_WFI1  \
+					 | EXYNOS5420_ARM_USE_STANDBY_WFI2  \
+					 | EXYNOS5420_ARM_USE_STANDBY_WFI3  \
+					 | EXYNOS5420_KFC_USE_STANDBY_WFI0  \
+					 | EXYNOS5420_KFC_USE_STANDBY_WFI1  \
+					 | EXYNOS5420_KFC_USE_STANDBY_WFI2  \
+					 | EXYNOS5420_KFC_USE_STANDBY_WFI3)
+
+
 void mct_init(void __iomem *base, int irq_g0, int irq_l0, int irq_l1);
 
 struct map_desc;
diff --git a/arch/arm/mach-exynos/pmu.c b/arch/arm/mach-exynos/pmu.c
index 05c7ce1..f69a6ed 100644
--- a/arch/arm/mach-exynos/pmu.c
+++ b/arch/arm/mach-exynos/pmu.c
@@ -12,9 +12,14 @@
 #include <linux/io.h>
 #include <linux/kernel.h>
 #include <linux/bug.h>
+#include <linux/cpumask.h>
+#include <linux/delay.h>
+#include <linux/pm.h>
 
 #include <plat/cpu.h>
 
+#include <asm/cputype.h>
+
 #include "common.h"
 #include "regs-pmu.h"
 
@@ -318,6 +323,212 @@ static const struct exynos_pmu_conf exynos5250_pmu_config[] = {
 	{ PMU_TABLE_END,},
 };
 
+static const struct exynos_pmu_conf exynos5420_pmu_config[] = {
+	/* { .reg = address, .val = { AFTR, LPA, SLEEP } */
+	{ EXYNOS5_ARM_CORE0_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_DIS_IRQ_ARM_CORE0_LOCAL_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_DIS_IRQ_ARM_CORE0_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_ARM_CORE1_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_DIS_IRQ_ARM_CORE1_LOCAL_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_DIS_IRQ_ARM_CORE1_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_ARM_CORE2_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_ARM_CORE2_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_ARM_CORE2_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_ARM_CORE3_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_ARM_CORE3_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_ARM_CORE3_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_KFC_CORE0_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_KFC_CORE0_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_KFC_CORE0_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_KFC_CORE1_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_KFC_CORE1_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_KFC_CORE1_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_KFC_CORE2_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_KFC_CORE2_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_KFC_CORE2_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_KFC_CORE3_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_KFC_CORE3_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_DIS_IRQ_KFC_CORE3_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_ISP_ARM_SYS_PWR_REG,				{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_DIS_IRQ_ISP_ARM_LOCAL_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_DIS_IRQ_ISP_ARM_CENTRAL_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_ARM_COMMON_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_KFC_COMMON_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_ARM_L2_SYS_PWR_REG,				{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_KFC_L2_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_ACLKSTOP_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_CMU_SCLKSTOP_SYS_PWR_REG,			{ 0x1, 0x0, 0x1} },
+	{ EXYNOS5_CMU_RESET_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_CMU_ACLKSTOP_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_CMU_SCLKSTOP_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x1} },
+	{ EXYNOS5_CMU_RESET_SYSMEM_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_DRAM_FREQ_DOWN_SYS_PWR_REG,			{ 0x1, 0x0, 0x1} },
+	{ EXYNOS5_DDRPHY_DLLOFF_SYS_PWR_REG,			{ 0x1, 0x1, 0x1} },
+	{ EXYNOS5_DDRPHY_DLLLOCK_SYS_PWR_REG,			{ 0x1, 0x0, 0x1} },
+	{ EXYNOS5_APLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_MPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_VPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_EPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_BPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_CPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_DPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_IPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_KPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_MPLLUSER_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_BPLLUSER_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_RPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_SPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_TOP_BUS_SYS_PWR_REG,				{ 0x3, 0x0, 0x0} },
+	{ EXYNOS5_TOP_RETENTION_SYS_PWR_REG,			{ 0x1, 0x1, 0x1} },
+	{ EXYNOS5_TOP_PWR_SYS_PWR_REG,				{ 0x3, 0x3, 0x0} },
+	{ EXYNOS5_TOP_BUS_SYSMEM_SYS_PWR_REG,			{ 0x3, 0x0, 0x0} },
+	{ EXYNOS5_TOP_RETENTION_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x1} },
+	{ EXYNOS5_TOP_PWR_SYSMEM_SYS_PWR_REG,			{ 0x3, 0x0, 0x0} },
+	{ EXYNOS5_LOGIC_RESET_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_OSCCLK_GATE_SYS_PWR_REG,			{ 0x1, 0x0, 0x1} },
+	{ EXYNOS5_LOGIC_RESET_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_OSCCLK_GATE_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_INTRAM_MEM_SYS_PWR_REG,			{ 0x3, 0x0, 0x3} },
+	{ EXYNOS5420_INTROM_MEM_SYS_PWR_REG,			{ 0x3, 0x0, 0x3} },
+	{ EXYNOS5_PAD_RETENTION_DRAM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_PAD_RETENTION_MAU_SYS_PWR_REG,		{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_JTAG_SYS_PWR_REG,		{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_DRAM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_UART_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_MMC0_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_MMC1_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_MMC2_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_HSI_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_EBIA_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_EBIB_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_SPI_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5420_PAD_RETENTION_DRAM_COREBLK_SYS_PWR_REG,	{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_PAD_ISOLATION_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_PAD_ISOLATION_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_PAD_ALV_SEL_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_XUSBXTI_SYS_PWR_REG,				{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_XXTI_SYS_PWR_REG,				{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_EXT_REGULATOR_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_GPIO_MODE_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_GPIO_MODE_SYSMEM_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_GPIO_MODE_MAU_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_TOP_ASB_RESET_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
+	{ EXYNOS5_TOP_ASB_ISOLATION_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
+	{ EXYNOS5_GSCL_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5_ISP_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5_MFC_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5_G3D_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5420_DISP1_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5420_MAU_SYS_PWR_REG,				{ 0x7, 0x7, 0x0} },
+	{ EXYNOS5420_G2D_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5420_MSC_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5420_FSYS_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5420_FSYS2_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5420_PSGEN_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5420_PERIC_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5420_WCORE_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
+	{ EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_RESET_ISP_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
+	{ EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
+	{ PMU_TABLE_END,},
+};
+
+void __iomem *exynos5_list_disable_pmu_reg[] = {
+	EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,
+	EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,
+	EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,
+	EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,
+	EXYNOS5_CMU_CLKSTOP_DISP1_SYS_PWR_REG,
+	EXYNOS5_CMU_CLKSTOP_MAU_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_DISP1_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_MAU_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_DISP1_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_MAU_SYS_PWR_REG,
+};
+
+void __iomem *exynos5420_list_disable_pmu_reg[] = {
+	EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,
+	EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,
+	EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,
+	EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,
+	EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG,
+	EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG,
+	EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG,
+	EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG,
+	EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG,
+	EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG,
+	EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG,
+	EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,
+	EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG,
+	EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_ISP_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,
+	EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG,
+	EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG,
+};
+
 static void __iomem * const exynos5_list_both_cnt_feed[] = {
 	EXYNOS5_ARM_CORE0_OPTION,
 	EXYNOS5_ARM_CORE1_OPTION,
@@ -371,6 +582,21 @@ static void exynos5_init_pmu(void)
 	}
 }
 
+/*
+ * exynos_set_core_flag - set the cluster id to IROM register
+ *			  to ensure that we wake up with the
+ *			  current cluster.
+ */
+static void exynos_set_core_flag(void)
+{
+	int cluster_id = (read_cpuid_mpidr() >> 8) & 0xf;
+
+	if (cluster_id)
+		__raw_writel(1, EXYNOS_IROM_DATA2);
+	else
+		__raw_writel(0, EXYNOS_IROM_DATA2);
+}
+
 void exynos_sys_powerdown_conf(enum sys_powerdown mode)
 {
 	unsigned int i;
@@ -378,6 +604,9 @@ void exynos_sys_powerdown_conf(enum sys_powerdown mode)
 	if (soc_is_exynos5250())
 		exynos5_init_pmu();
 
+	if (soc_is_exynos5420())
+		exynos_set_core_flag();
+
 	for (i = 0; (exynos_pmu_config[i].reg != PMU_TABLE_END) ; i++)
 		__raw_writel(exynos_pmu_config[i].val[mode],
 				exynos_pmu_config[i].reg);
@@ -389,9 +618,27 @@ void exynos_sys_powerdown_conf(enum sys_powerdown mode)
 	}
 }
 
+static void exynos5_power_off(void)
+{
+	unsigned int tmp;
+
+	pr_info("Power down.\n");
+	tmp = __raw_readl(EXYNOS_PS_HOLD_CONTROL);
+	tmp &= ~(1 << 8);
+	__raw_writel(tmp, EXYNOS_PS_HOLD_CONTROL);
+
+	/* Wait a little so we don't give a false warning below */
+	mdelay(100);
+
+	pr_err("Power down failed, please power off system manually.\n");
+	while (1)
+		;
+}
+
 static int __init exynos_pmu_init(void)
 {
 	unsigned int value;
+	int i;
 
 	exynos_pmu_config = exynos4210_pmu_config;
 
@@ -416,6 +663,66 @@ static int __init exynos_pmu_init(void)
 
 		exynos_pmu_config = exynos5250_pmu_config;
 		pr_info("EXYNOS5250 PMU Initialize\n");
+	} else if (soc_is_exynos5420()) {
+		/*
+		 * Set the CMU_RESET, CMU_SYSCLK and CMU_CLKSTOP registers
+		 * for local power blocks to Low initially as per Table 8-4:
+		 * "System-Level Power-Down Configuration Registers".
+		 */
+		for (i = 0;
+			i < ARRAY_SIZE(exynos5420_list_disable_pmu_reg);
+			i++)
+			__raw_writel(0x0, exynos5420_list_disable_pmu_reg[i]);
+		/* Time taken to stabilized XXTI clock */
+		__raw_writel(0x005dc, EXYNOS5_XXTI_DURATION3);
+
+		/* Enable USE_STANDBY_WFI for all CORE */
+		__raw_writel(EXYNOS5420_USE_STANDBY_WFI_ALL,
+				S5P_CENTRAL_SEQ_OPTION);
+
+		value = __raw_readl(EXYNOS_L2_OPTION(0));
+		value &= ~EXYNOS5_USE_RETENTION;
+		__raw_writel(value, EXYNOS_L2_OPTION(0));
+
+		value = __raw_readl(EXYNOS_L2_OPTION(1));
+		value &= ~EXYNOS5_USE_RETENTION;
+		__raw_writel(value, EXYNOS_L2_OPTION(1));
+
+		/*
+		 * If L2_COMMON is turned off, clocks related to ATB async
+		 * bridge are gated. Thus, when ISP power is gated, LPI
+		 * may get stuck.
+		 */
+		value = __raw_readl(EXYNOS5420_LPI_MASK);
+		value |= EXYNOS5420_ATB_ISP_ARM;
+		__raw_writel(value, EXYNOS5420_LPI_MASK);
+		value = __raw_readl(EXYNOS5420_LPI_MASK1);
+		value |= EXYNOS5420_ATB_KFC;
+		__raw_writel(value, EXYNOS5420_LPI_MASK1);
+
+		/* Prevent issue of new bus request from L2 memory */
+		value = __raw_readl(EXYNOS5420_ARM_COMMON_OPTION);
+		value |= EXYNOS5_SKIP_DEACTIVATE_ACEACP_IN_PWDN;
+		__raw_writel(value, EXYNOS5420_ARM_COMMON_OPTION);
+
+		value = __raw_readl(EXYNOS5420_KFC_COMMON_OPTION);
+		value |= EXYNOS5_SKIP_DEACTIVATE_ACEACP_IN_PWDN;
+		__raw_writel(value, EXYNOS5420_KFC_COMMON_OPTION);
+
+		/*
+		 * This setting is to reduce suspend/resume time.
+		 */
+		__raw_writel(DUR_WAIT_RESET, EXYNOS5420_LOGIC_RESET_DURATION3);
+
+		/* Serialized CPU wakeup of Eagle */
+		__raw_writel(SPREAD_ENABLE, EXYNOS5420_ARM_INTR_SPREAD_ENABLE);
+		__raw_writel(SPREAD_USE_STANDWFI,
+			EXYNOS5420_ARM_INTR_SPREAD_USE_STANDBYWFI);
+		__raw_writel(0x1, EXYNOS5420_UP_SCHEDULER);
+
+		exynos_pmu_config = exynos5420_pmu_config;
+		pm_power_off = exynos5_power_off;
+		pr_info("EXYNOS5420 PMU Initialized\n");
 	} else {
 		pr_info("EXYNOS: PMU not supported\n");
 	}
diff --git a/arch/arm/mach-exynos/regs-pmu.h b/arch/arm/mach-exynos/regs-pmu.h
index 4f6a256..cfbfc575 100644
--- a/arch/arm/mach-exynos/regs-pmu.h
+++ b/arch/arm/mach-exynos/regs-pmu.h
@@ -39,6 +39,8 @@
 #define S5P_INFORM6				S5P_PMUREG(0x0818)
 #define S5P_INFORM7				S5P_PMUREG(0x081C)
 
+#define EXYNOS_IROM_DATA2			S5P_PMUREG(0x0988)
+
 #define S5P_ARM_CORE0_LOWPWR			S5P_PMUREG(0x1000)
 #define S5P_DIS_IRQ_CORE0			S5P_PMUREG(0x1004)
 #define S5P_DIS_IRQ_CENTRAL0			S5P_PMUREG(0x1008)
@@ -108,6 +110,10 @@
 #define S5P_ARM_CORE1_CONFIGURATION		S5P_PMUREG(0x2080)
 #define S5P_ARM_CORE1_STATUS			S5P_PMUREG(0x2084)
 
+#define EXYNOS5_ARM_L2_OPTION			S5P_PMUREG(0x2608)
+#define EXYNOS5_USE_RETENTION			BIT(4)
+
+
 #define S5P_PAD_RET_MAUDIO_OPTION		S5P_PMUREG(0x3028)
 #define S5P_PAD_RET_GPIO_OPTION			S5P_PMUREG(0x3108)
 #define S5P_PAD_RET_UART_OPTION			S5P_PMUREG(0x3128)
@@ -121,6 +127,36 @@
 
 #define S5P_CHECK_SLEEP				0x00000BAD
 
+#define S5P_ARM_CORE0_CONFIGURATION		S5P_PMUREG(0x2000)
+#define S5P_ARM_CORE0_STATUS			S5P_PMUREG(0x2004)
+#define S5P_ARM_CORE0_OPTION			S5P_PMUREG(0x2008)
+
+#define EXYNOS_ARM_CORE_OPTION(_nr)		(S5P_ARM_CORE0_OPTION \
+						+ ((_nr) * 0x80))
+#define EXYNOS_ARM_CORE_STATUS(_nr)		(S5P_ARM_CORE0_STATUS \
+						+ ((_nr) * 0x80))
+#define EXYNOS_ARM_CORE_CONFIGURATION(_nr)	\
+			(S5P_ARM_CORE0_CONFIGURATION + ((_nr) * 0x80))
+#define EXYNOS_CORE_LOCAL_PWR_EN		0x3
+
+#define EXYNOS_ARM_COMMON_CONFIGURATION		S5P_PMUREG(0x2500)
+#define EXYNOS_ARM_COMMON_STATUS		S5P_PMUREG(0x2504)
+#define EXYNOS_COMMON_CONFIGURATION(_nr)	\
+			(EXYNOS_ARM_COMMON_CONFIGURATION + ((_nr) * 0x80))
+#define EXYNOS_COMMON_STATUS(_nr)		\
+			(EXYNOS_COMMON_CONFIGURATION(_nr) + 0x4)
+#define EXYNOS_COMMON_OPTION(_nr)		\
+			(EXYNOS_COMMON_CONFIGURATION(_nr) + 0x8)
+
+#define EXYNOS_ARM_L2_CONFIGURATION		S5P_PMUREG(0x2600)
+#define EXYNOS_L2_CONFIGURATION(_nr)		\
+			(EXYNOS_ARM_L2_CONFIGURATION + ((_nr) * 0x80))
+#define EXYNOS_L2_STATUS(_nr)			\
+			(EXYNOS_L2_CONFIGURATION(_nr) + 0x4)
+#define EXYNOS_L2_OPTION(_nr)			\
+			(EXYNOS_L2_CONFIGURATION(_nr) + 0x8)
+#define EXYNOS_L2_COMMON_PWR_EN			0x3
+
 /* Only for EXYNOS4210 */
 #define S5P_CMU_CLKSTOP_LCD1_LOWPWR	S5P_PMUREG(0x1154)
 #define S5P_CMU_RESET_LCD1_LOWPWR	S5P_PMUREG(0x1174)
@@ -313,4 +349,196 @@
 
 #define EXYNOS5_OPTION_USE_RETENTION				(1 << 4)
 
+/* Only for EXYNOS5420 */
+#define EXYNOS5420_ISP_ARM_OPTION				S5P_PMUREG(0x2488)
+#define EXYNOS5420_L2RSTDISABLE_VALUE				(1 << 3)
+
+#define EXYNOS5420_LPI_MASK					S5P_PMUREG(0x0004)
+#define EXYNOS5420_LPI_MASK1					S5P_PMUREG(0x0008)
+#define EXYNOS5420_UFS						(1 << 8)
+#define EXYNOS5420_ATB_KFC					(1 << 13)
+#define EXYNOS5420_ATB_ISP_ARM					(1 << 19)
+#define EXYNOS5420_EMULATION					(1 << 31)
+#define ATB_ISP_ARM						(1 << 12)
+#define ATB_KFC							(1 << 13)
+#define ATB_NOC							(1 << 14)
+
+#define EXYNOS5420_ARM_INTR_SPREAD_ENABLE			S5P_PMUREG(0x0100)
+#define EXYNOS5420_ARM_INTR_SPREAD_USE_STANDBYWFI		S5P_PMUREG(0x0104)
+#define EXYNOS5420_UP_SCHEDULER					S5P_PMUREG(0x0120)
+#define SPREAD_ENABLE						0xF
+#define SPREAD_USE_STANDWFI					0xF
+
+#define EXYNOS5420_BB_CON1					S5P_PMUREG(0x0784)
+#define EXYNOS5420_BB_SEL_EN					(1 << 31)
+#define EXYNOS5420_BB_PMOS_EN					(1 << 7)
+#define EXYNOS5420_BB_1300X					0XF
+
+#define EXYNOS5420_ARM_CORE2_SYS_PWR_REG			S5P_PMUREG(0x1020)
+#define EXYNOS5420_DIS_IRQ_ARM_CORE2_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1024)
+#define EXYNOS5420_DIS_IRQ_ARM_CORE2_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1028)
+#define EXYNOS5420_ARM_CORE3_SYS_PWR_REG			S5P_PMUREG(0x1030)
+#define EXYNOS5420_DIS_IRQ_ARM_CORE3_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1034)
+#define EXYNOS5420_DIS_IRQ_ARM_CORE3_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1038)
+#define EXYNOS5420_KFC_CORE0_SYS_PWR_REG			S5P_PMUREG(0x1040)
+#define EXYNOS5420_DIS_IRQ_KFC_CORE0_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1044)
+#define EXYNOS5420_DIS_IRQ_KFC_CORE0_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1048)
+#define EXYNOS5420_KFC_CORE1_SYS_PWR_REG			S5P_PMUREG(0x1050)
+#define EXYNOS5420_DIS_IRQ_KFC_CORE1_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1054)
+#define EXYNOS5420_DIS_IRQ_KFC_CORE1_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1058)
+#define EXYNOS5420_KFC_CORE2_SYS_PWR_REG			S5P_PMUREG(0x1060)
+#define EXYNOS5420_DIS_IRQ_KFC_CORE2_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1064)
+#define EXYNOS5420_DIS_IRQ_KFC_CORE2_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1068)
+#define EXYNOS5420_KFC_CORE3_SYS_PWR_REG			S5P_PMUREG(0x1070)
+#define EXYNOS5420_DIS_IRQ_KFC_CORE3_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1074)
+#define EXYNOS5420_DIS_IRQ_KFC_CORE3_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1078)
+#define EXYNOS5420_ISP_ARM_SYS_PWR_REG				S5P_PMUREG(0x1090)
+#define EXYNOS5420_DIS_IRQ_ISP_ARM_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1094)
+#define EXYNOS5420_DIS_IRQ_ISP_ARM_CENTRAL_SYS_PWR_REG		S5P_PMUREG(0x1098)
+#define EXYNOS5420_ARM_COMMON_SYS_PWR_REG			S5P_PMUREG(0x10A0)
+#define EXYNOS5420_KFC_COMMON_SYS_PWR_REG			S5P_PMUREG(0x10B0)
+#define EXYNOS5420_KFC_L2_SYS_PWR_REG				S5P_PMUREG(0x10D0)
+#define EXYNOS5420_DPLL_SYSCLK_SYS_PWR_REG			S5P_PMUREG(0x1158)
+#define EXYNOS5420_IPLL_SYSCLK_SYS_PWR_REG			S5P_PMUREG(0x115C)
+#define EXYNOS5420_KPLL_SYSCLK_SYS_PWR_REG			S5P_PMUREG(0x1160)
+#define EXYNOS5420_RPLL_SYSCLK_SYS_PWR_REG                      S5P_PMUREG(0x1174)
+#define EXYNOS5420_SPLL_SYSCLK_SYS_PWR_REG                      S5P_PMUREG(0x1178)
+#define EXYNOS5420_INTRAM_MEM_SYS_PWR_REG                       S5P_PMUREG(0x11B8)
+#define EXYNOS5420_INTROM_MEM_SYS_PWR_REG                       S5P_PMUREG(0x11BC)
+#define EXYNOS5420_ONENANDXL_MEM_SYS_PWR			S5P_PMUREG(0x11C0)
+#define EXYNOS5420_USBDEV_MEM_SYS_PWR				S5P_PMUREG(0x11CC)
+#define EXYNOS5420_USBDEV1_MEM_SYS_PWR				S5P_PMUREG(0x11D0)
+#define EXYNOS5420_SDMMC_MEM_SYS_PWR				S5P_PMUREG(0x11D4)
+#define EXYNOS5420_CSSYS_MEM_SYS_PWR				S5P_PMUREG(0x11D8)
+#define EXYNOS5420_SECSS_MEM_SYS_PWR				S5P_PMUREG(0x11DC)
+#define EXYNOS5420_ROTATOR_MEM_SYS_PWR				S5P_PMUREG(0x11E0)
+#define EXYNOS5420_INTRAM_MEM_SYS_PWR				S5P_PMUREG(0x11E4)
+#define EXYNOS5420_INTROM_MEM_SYS_PWR				S5P_PMUREG(0x11E8)
+#define EXYNOS5420_PAD_RETENTION_JTAG_SYS_PWR_REG		S5P_PMUREG(0x1208)
+#define EXYNOS5420_PAD_RETENTION_DRAM_SYS_PWR_REG		S5P_PMUREG(0x1210)
+#define EXYNOS5420_PAD_RETENTION_UART_SYS_PWR_REG		S5P_PMUREG(0x1214)
+#define EXYNOS5420_PAD_RETENTION_MMC0_SYS_PWR_REG		S5P_PMUREG(0x1218)
+#define EXYNOS5420_PAD_RETENTION_MMC1_SYS_PWR_REG		S5P_PMUREG(0x121C)
+#define EXYNOS5420_PAD_RETENTION_MMC2_SYS_PWR_REG		S5P_PMUREG(0x1220)
+#define EXYNOS5420_PAD_RETENTION_HSI_SYS_PWR_REG		S5P_PMUREG(0x1224)
+#define EXYNOS5420_PAD_RETENTION_EBIA_SYS_PWR_REG		S5P_PMUREG(0x1228)
+#define EXYNOS5420_PAD_RETENTION_EBIB_SYS_PWR_REG		S5P_PMUREG(0x122C)
+#define EXYNOS5420_PAD_RETENTION_SPI_SYS_PWR_REG		S5P_PMUREG(0x1230)
+#define EXYNOS5420_PAD_RETENTION_DRAM_COREBLK_SYS_PWR_REG	S5P_PMUREG(0x1234)
+#define EXYNOS5420_DISP1_SYS_PWR_REG				S5P_PMUREG(0x1410)
+#define EXYNOS5420_MAU_SYS_PWR_REG				S5P_PMUREG(0x1414)
+#define EXYNOS5420_G2D_SYS_PWR_REG				S5P_PMUREG(0x1418)
+#define EXYNOS5420_MSC_SYS_PWR_REG				S5P_PMUREG(0x141C)
+#define EXYNOS5420_FSYS_SYS_PWR_REG				S5P_PMUREG(0x1420)
+#define EXYNOS5420_FSYS2_SYS_PWR_REG				S5P_PMUREG(0x1424)
+#define EXYNOS5420_PSGEN_SYS_PWR_REG				S5P_PMUREG(0x1428)
+#define EXYNOS5420_PERIC_SYS_PWR_REG				S5P_PMUREG(0x142C)
+#define EXYNOS5420_WCORE_SYS_PWR_REG				S5P_PMUREG(0x1430)
+#define EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG		S5P_PMUREG(0x1490)
+#define EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG			S5P_PMUREG(0x1494)
+#define EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG			S5P_PMUREG(0x1498)
+#define EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG			S5P_PMUREG(0x149C)
+#define EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG			S5P_PMUREG(0x14A0)
+#define EXYNOS5420_CMU_CLKSTOP_FSYS2_SYS_PWR_REG		S5P_PMUREG(0x14A4)
+#define EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG		S5P_PMUREG(0x14A8)
+#define EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG		S5P_PMUREG(0x14AC)
+#define EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG		S5P_PMUREG(0x14B0)
+#define EXYNOS5420_CMU_SYSCLK_TOPPWR_SYS_PWR_REG		S5P_PMUREG(0x14BC)
+#define EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG			S5P_PMUREG(0x14D0)
+#define EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG			S5P_PMUREG(0x14D4)
+#define EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG			S5P_PMUREG(0x14D8)
+#define EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG			S5P_PMUREG(0x14DC)
+#define EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG			S5P_PMUREG(0x14E0)
+#define EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG			S5P_PMUREG(0x14E4)
+#define EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG			S5P_PMUREG(0x14E8)
+#define EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG			S5P_PMUREG(0x14EC)
+#define EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG			S5P_PMUREG(0x14F0)
+#define EXYNOS5420_CMU_SYSCLK_SYSMEM_TOPPWR_SYS_PWR_REG		S5P_PMUREG(0x14F4)
+#define EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG			S5P_PMUREG(0x1570)
+#define EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG			S5P_PMUREG(0x1574)
+#define EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG			S5P_PMUREG(0x1578)
+#define EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG			S5P_PMUREG(0x157C)
+#define EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG			S5P_PMUREG(0x1590)
+#define EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG			S5P_PMUREG(0x1594)
+#define EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG			S5P_PMUREG(0x1598)
+#define EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG			S5P_PMUREG(0x159C)
+#define EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG			S5P_PMUREG(0x15A0)
+#define EXYNOS5420_SFR_AXI_CGDIS1				S5P_PMUREG(0x15E4)
+#define EXYNOS_ARM_CORE2_CONFIGURATION				S5P_PMUREG(0x2100)
+#define EXYNOS5420_ARM_CORE2_OPTION				S5P_PMUREG(0x2108)
+#define EXYNOS_ARM_CORE3_CONFIGURATION				S5P_PMUREG(0x2180)
+#define EXYNOS5420_ARM_CORE3_OPTION				S5P_PMUREG(0x2188)
+#define EXYNOS5420_ARM_COMMON_STATUS				S5P_PMUREG(0x2504)
+#define EXYNOS5420_ARM_COMMON_OPTION				S5P_PMUREG(0x2508)
+#define EXYNOS5420_KFC_COMMON_STATUS				S5P_PMUREG(0x2584)
+#define EXYNOS5420_KFC_COMMON_OPTION				S5P_PMUREG(0x2588)
+#define EXYNOS5420_LOGIC_RESET_DURATION3			S5P_PMUREG(0x2D1C)
+
+#define EXYNOS5420_PAD_RET_GPIO_OPTION				S5P_PMUREG(0x30C8)
+#define EXYNOS5420_PAD_RET_UART_OPTION				S5P_PMUREG(0x30E8)
+#define EXYNOS5420_PAD_RET_MMCA_OPTION				S5P_PMUREG(0x3108)
+#define EXYNOS5420_PAD_RET_MMCB_OPTION				S5P_PMUREG(0x3128)
+#define EXYNOS5420_PAD_RET_MMCC_OPTION				S5P_PMUREG(0x3148)
+#define EXYNOS5420_PAD_RET_HSI_OPTION				S5P_PMUREG(0x3168)
+#define EXYNOS5420_PAD_RET_SPI_OPTION				S5P_PMUREG(0x31C8)
+#define EXYNOS5420_PAD_RET_DRAM_COREBLK_OPTION			S5P_PMUREG(0x31E8)
+#define EXYNOS_PAD_RET_DRAM_OPTION				S5P_PMUREG(0x3008)
+#define EXYNOS_PAD_RET_MAUDIO_OPTION				S5P_PMUREG(0x3028)
+#define EXYNOS_PAD_RET_JTAG_OPTION				S5P_PMUREG(0x3048)
+#define EXYNOS_PAD_RET_GPIO_OPTION				S5P_PMUREG(0x3108)
+#define EXYNOS_PAD_RET_UART_OPTION				S5P_PMUREG(0x3128)
+#define EXYNOS_PAD_RET_MMCA_OPTION				S5P_PMUREG(0x3148)
+#define EXYNOS_PAD_RET_MMCB_OPTION				S5P_PMUREG(0x3168)
+#define EXYNOS_PAD_RET_EBIA_OPTION				S5P_PMUREG(0x3188)
+#define EXYNOS_PAD_RET_EBIB_OPTION				S5P_PMUREG(0x31A8)
+
+#define EXYNOS_PS_HOLD_CONTROL					S5P_PMUREG(0x330C)
+#define EXYNOS5_XXTI_DURATION3					S5P_PMUREG(0x343C)
+
+/* For SYS_PWR_REG */
+#define EXYNOS_SYS_PWR_CFG					(1 << 0)
+
+#define EXYNOS5420_MFC_CONFIGURATION				S5P_PMUREG(0x4060)
+#define EXYNOS5420_MFC_STATUS					S5P_PMUREG(0x4064)
+#define EXYNOS5420_MFC_OPTION					S5P_PMUREG(0x4068)
+#define EXYNOS5420_G3D_CONFIGURATION				S5P_PMUREG(0x4080)
+#define EXYNOS5420_G3D_STATUS					S5P_PMUREG(0x4084)
+#define EXYNOS5420_G3D_OPTION					S5P_PMUREG(0x4088)
+#define EXYNOS5420_DISP0_CONFIGURATION				S5P_PMUREG(0x40A0)
+#define EXYNOS5420_DISP0_STATUS					S5P_PMUREG(0x40A4)
+#define EXYNOS5420_DISP0_OPTION					S5P_PMUREG(0x40A8)
+#define EXYNOS5420_DISP1_CONFIGURATION				S5P_PMUREG(0x40C0)
+#define EXYNOS5420_DISP1_STATUS					S5P_PMUREG(0x40C4)
+#define EXYNOS5420_DISP1_OPTION					S5P_PMUREG(0x40C8)
+#define EXYNOS5420_MAU_CONFIGURATION				S5P_PMUREG(0x40E0)
+#define EXYNOS5420_MAU_STATUS					S5P_PMUREG(0x40E4)
+#define EXYNOS5420_MAU_OPTION					S5P_PMUREG(0x40E8)
+#define EXYNOS5420_FSYS2_OPTION					S5P_PMUREG(0x4168)
+#define EXYNOS5420_PSGEN_OPTION					S5P_PMUREG(0x4188)
+
+/* For EXYNOS_CENTRAL_SEQ_OPTION */
+#define EXYNOS5_USE_STANDBYWFI_ARM_CORE0			(1 << 16)
+#define EXYNOS5_USE_STANDBYWFI_ARM_CORE1			(1 << 17)
+#define EXYNOS5_USE_STANDBYWFE_ARM_CORE0			(1 << 24)
+#define EXYNOS5_USE_STANDBYWFE_ARM_CORE1			(1 << 25)
+
+#define EXYNOS5420_ARM_USE_STANDBY_WFI0				(1 << 4)
+#define EXYNOS5420_ARM_USE_STANDBY_WFI1				(1 << 5)
+#define EXYNOS5420_ARM_USE_STANDBY_WFI2				(1 << 6)
+#define EXYNOS5420_ARM_USE_STANDBY_WFI3				(1 << 7)
+#define EXYNOS5420_KFC_USE_STANDBY_WFI0				(1 << 8)
+#define EXYNOS5420_KFC_USE_STANDBY_WFI1				(1 << 9)
+#define EXYNOS5420_KFC_USE_STANDBY_WFI2				(1 << 10)
+#define EXYNOS5420_KFC_USE_STANDBY_WFI3				(1 << 11)
+#define EXYNOS5420_ARM_USE_STANDBY_WFE0				(1 << 16)
+#define EXYNOS5420_ARM_USE_STANDBY_WFE1				(1 << 17)
+#define EXYNOS5420_ARM_USE_STANDBY_WFE2				(1 << 18)
+#define EXYNOS5420_ARM_USE_STANDBY_WFE3				(1 << 19)
+#define EXYNOS5420_KFC_USE_STANDBY_WFE0				(1 << 20)
+#define EXYNOS5420_KFC_USE_STANDBY_WFE1				(1 << 21)
+#define EXYNOS5420_KFC_USE_STANDBY_WFE2				(1 << 22)
+#define EXYNOS5420_KFC_USE_STANDBY_WFE3				(1 << 23)
+
+#define DUR_WAIT_RESET				0xF
+
 #endif /* __ASM_ARCH_REGS_PMU_H */
-- 
1.7.9.5

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

* [PATCH V2 2/3] ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420
  2014-03-27  6:13 [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
  2014-03-27  6:13 ` [PATCH V2 1/3] ARM: EXYNOS5: Add PMU settings " Vikas Sajjan
@ 2014-03-27  6:13 ` Vikas Sajjan
  2014-04-15 19:03   ` Tomasz Figa
  2014-03-27  6:13 ` [PATCH V2 3/3] ARM: dts: Add node for GPIO keys on SMDK5420 Vikas Sajjan
  2014-04-11 13:17 ` [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
  3 siblings, 1 reply; 12+ messages in thread
From: Vikas Sajjan @ 2014-03-27  6:13 UTC (permalink / raw)
  To: linux-arm-kernel

Adds Suspend-to-RAM support for EXYNOS5420

Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
---
 arch/arm/mach-exynos/pm.c                    |  148 ++++++++++++++++++++++----
 arch/arm/mach-exynos/regs-pmu.h              |    4 +-
 arch/arm/plat-samsung/include/plat/map-s5p.h |    2 +
 drivers/clk/samsung/clk-exynos5420.c         |   32 ++++++
 4 files changed, 167 insertions(+), 19 deletions(-)

diff --git a/arch/arm/mach-exynos/pm.c b/arch/arm/mach-exynos/pm.c
index 15af0ce..aa3c2c8 100644
--- a/arch/arm/mach-exynos/pm.c
+++ b/arch/arm/mach-exynos/pm.c
@@ -59,6 +59,16 @@ static struct sleep_save exynos_core_save[] = {
 	SAVE_ITEM(S5P_SROM_BC3),
 };
 
+static struct sleep_save exynos5420_cpustate_save[] = {
+	SAVE_ITEM(EXYNOS5420_VA_CPU_STATE),
+};
+
+static struct sleep_save exynos5420_reg_save[] = {
+	SAVE_ITEM(EXYNOS5_SYS_DISP1_BLK_CFG),
+	SAVE_ITEM(S5P_PMU_SPARE3),
+};
+
+
 /*
  * GIC wake-up support
  */
@@ -81,7 +91,7 @@ static int exynos_irq_set_wake(struct irq_data *data, unsigned int state)
 {
 	const struct exynos_wkup_irq *wkup_irq;
 
-	if (soc_is_exynos5250())
+	if (soc_is_exynos5250() || soc_is_exynos5420())
 		wkup_irq = exynos5250_wkup_irq;
 	else
 		wkup_irq = exynos4_wkup_irq;
@@ -109,7 +119,15 @@ static int exynos_cpu_suspend(unsigned long arg)
 	outer_flush_all();
 #endif
 
-	if (soc_is_exynos5250())
+	/*
+	 * Clear IRAM register for cpu state so that primary CPU does
+	 * not enter low power start in U-Boot.
+	 * This is specific to exynos5420 SoC only.
+	 */
+	if (soc_is_exynos5420())
+		__raw_writel(0x0, EXYNOS5420_VA_CPU_STATE);
+
+	if (soc_is_exynos5250() || soc_is_exynos5420())
 		flush_cache_all();
 
 	/* issue the standby signal into the pm unit. */
@@ -135,6 +153,20 @@ static void exynos_pm_prepare(void)
 		tmp = __raw_readl(EXYNOS5_JPEG_MEM_OPTION);
 		tmp &= ~EXYNOS5_OPTION_USE_RETENTION;
 		__raw_writel(tmp, EXYNOS5_JPEG_MEM_OPTION);
+	} else if (soc_is_exynos5420()) {
+
+		s3c_pm_do_save(exynos5420_reg_save,
+			ARRAY_SIZE(exynos5420_reg_save));
+
+		/*
+		 * The cpu state needs to be saved and restored so that the
+		 * secondary CPUs will enter low power start. Though the U-Boot
+		 * is setting the cpu state with low power flag, the kernel
+		 * needs to restore it back in case, the primary cpu fails to
+		 * suspend for any reason
+		 */
+		s3c_pm_do_save(exynos5420_cpustate_save,
+			ARRAY_SIZE(exynos5420_cpustate_save));
 	}
 
 	/* Set value of power down register for sleep mode */
@@ -145,11 +177,34 @@ static void exynos_pm_prepare(void)
 	/* ensure at least INFORM0 has the resume address */
 
 	__raw_writel(virt_to_phys(exynos_cpu_resume), S5P_INFORM0);
+
+	if (soc_is_exynos5420()) {
+
+		tmp = __raw_readl(EXYNOS5_ARM_L2_OPTION);
+		tmp &= ~EXYNOS5_USE_RETENTION;
+		__raw_writel(tmp, EXYNOS5_ARM_L2_OPTION);
+
+		tmp = __raw_readl(EXYNOS5420_SFR_AXI_CGDIS1);
+		tmp |= EXYNOS5420_UFS;
+		__raw_writel(tmp, EXYNOS5420_SFR_AXI_CGDIS1);
+
+		tmp = __raw_readl(EXYNOS5420_ARM_COMMON_OPTION);
+		tmp &= ~EXYNOS5420_L2RSTDISABLE_VALUE;
+		__raw_writel(tmp, EXYNOS5420_ARM_COMMON_OPTION);
+		tmp = __raw_readl(EXYNOS5420_FSYS2_OPTION);
+		tmp |= EXYNOS5420_EMULATION;
+		__raw_writel(tmp, EXYNOS5420_FSYS2_OPTION);
+		tmp = __raw_readl(EXYNOS5420_PSGEN_OPTION);
+		tmp |= EXYNOS5420_EMULATION;
+		__raw_writel(tmp, EXYNOS5420_PSGEN_OPTION);
+	}
+
 }
 
 static int exynos_pm_suspend(void)
 {
 	unsigned long tmp;
+	unsigned long cluster_id;
 
 	/* Setting Central Sequence Register for power down mode */
 
@@ -159,10 +214,20 @@ static int exynos_pm_suspend(void)
 
 	/* Setting SEQ_OPTION register */
 
-	tmp = (S5P_USE_STANDBY_WFI0 | S5P_USE_STANDBY_WFE0);
-	__raw_writel(tmp, S5P_CENTRAL_SEQ_OPTION);
+	if (soc_is_exynos5420()) {
+		cluster_id = (read_cpuid(CPUID_MPIDR) >> 8) & 0xf;
+		if (!cluster_id)
+			__raw_writel(EXYNOS5420_ARM_USE_STANDBY_WFI0,
+				     S5P_CENTRAL_SEQ_OPTION);
+		else
+			__raw_writel(EXYNOS5420_KFC_USE_STANDBY_WFI0,
+				     S5P_CENTRAL_SEQ_OPTION);
+	} else {
+		tmp = (S5P_USE_STANDBY_WFI0 | S5P_USE_STANDBY_WFE0);
+		__raw_writel(tmp, S5P_CENTRAL_SEQ_OPTION);
+	}
 
-	if (!soc_is_exynos5250()) {
+	if (!(soc_is_exynos5250() || soc_is_exynos5420())) {
 		/* Save Power control register */
 		asm ("mrc p15, 0, %0, c15, c0, 0"
 		     : "=r" (tmp) : : "cc");
@@ -181,6 +246,15 @@ static void exynos_pm_resume(void)
 {
 	unsigned long tmp;
 
+	if (soc_is_exynos5420()) {
+		/* Restore the IRAM register cpu state */
+		s3c_pm_do_restore(exynos5420_cpustate_save,
+			ARRAY_SIZE(exynos5420_cpustate_save));
+
+		__raw_writel(EXYNOS5420_USE_STANDBY_WFI_ALL,
+			S5P_CENTRAL_SEQ_OPTION);
+	}
+
 	/*
 	 * If PMU failed while entering sleep mode, WFI will be
 	 * ignored by PMU and then exiting cpu_do_idle().
@@ -196,7 +270,8 @@ static void exynos_pm_resume(void)
 		/* No need to perform below restore code */
 		goto early_wakeup;
 	}
-	if (!soc_is_exynos5250()) {
+
+	if (!(soc_is_exynos5250() || soc_is_exynos5420())) {
 		/* Restore Power control register */
 		tmp = save_arm_register[0];
 		asm volatile ("mcr p15, 0, %0, c15, c0, 0"
@@ -212,25 +287,57 @@ static void exynos_pm_resume(void)
 
 	/* For release retention */
 
-	__raw_writel((1 << 28), S5P_PAD_RET_MAUDIO_OPTION);
-	__raw_writel((1 << 28), S5P_PAD_RET_GPIO_OPTION);
-	__raw_writel((1 << 28), S5P_PAD_RET_UART_OPTION);
-	__raw_writel((1 << 28), S5P_PAD_RET_MMCA_OPTION);
-	__raw_writel((1 << 28), S5P_PAD_RET_MMCB_OPTION);
-	__raw_writel((1 << 28), S5P_PAD_RET_EBIA_OPTION);
-	__raw_writel((1 << 28), S5P_PAD_RET_EBIB_OPTION);
+	if (soc_is_exynos5250()) {
+		__raw_writel((1 << 28), S5P_PAD_RET_MAUDIO_OPTION);
+		__raw_writel((1 << 28), S5P_PAD_RET_GPIO_OPTION);
+		__raw_writel((1 << 28), S5P_PAD_RET_UART_OPTION);
+		__raw_writel((1 << 28), S5P_PAD_RET_MMCA_OPTION);
+		__raw_writel((1 << 28), S5P_PAD_RET_MMCB_OPTION);
+		__raw_writel((1 << 28), S5P_PAD_RET_EBIA_OPTION);
+		__raw_writel((1 << 28), S5P_PAD_RET_EBIB_OPTION);
+	} else if (soc_is_exynos5420()) {
+		__raw_writel(1 << 28, EXYNOS_PAD_RET_DRAM_OPTION);
+		__raw_writel(1 << 28, EXYNOS_PAD_RET_MAUDIO_OPTION);
+		__raw_writel(1 << 28, EXYNOS_PAD_RET_JTAG_OPTION);
+		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_GPIO_OPTION);
+		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_UART_OPTION);
+		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCA_OPTION);
+		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCB_OPTION);
+		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCC_OPTION);
+		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_HSI_OPTION);
+		__raw_writel(1 << 28, EXYNOS_PAD_RET_EBIA_OPTION);
+		__raw_writel(1 << 28, EXYNOS_PAD_RET_EBIB_OPTION);
+		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_SPI_OPTION);
+		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_DRAM_COREBLK_OPTION);
+	}
 
 	if (soc_is_exynos5250())
 		s3c_pm_do_restore(exynos5_sys_save,
 			ARRAY_SIZE(exynos5_sys_save));
+	else if (soc_is_exynos5420())
+		s3c_pm_do_restore(exynos5420_reg_save,
+			ARRAY_SIZE(exynos5420_reg_save));
 
 	s3c_pm_do_restore_core(exynos_core_save, ARRAY_SIZE(exynos_core_save));
 
-	if (IS_ENABLED(CONFIG_SMP) && !soc_is_exynos5250())
+	if (IS_ENABLED(CONFIG_SMP) &&
+			!soc_is_exynos5250() && !soc_is_exynos5420())
 		scu_enable(S5P_VA_SCU);
 
 early_wakeup:
 
+	if (soc_is_exynos5420()) {
+		tmp = __raw_readl(EXYNOS5420_SFR_AXI_CGDIS1);
+		tmp &= ~EXYNOS5420_UFS;
+		__raw_writel(tmp, EXYNOS5420_SFR_AXI_CGDIS1);
+		tmp = __raw_readl(EXYNOS5420_FSYS2_OPTION);
+		tmp &= ~EXYNOS5420_EMULATION;
+		__raw_writel(tmp, EXYNOS5420_FSYS2_OPTION);
+		tmp = __raw_readl(EXYNOS5420_PSGEN_OPTION);
+		tmp &= ~EXYNOS5420_EMULATION;
+		__raw_writel(tmp, EXYNOS5420_PSGEN_OPTION);
+	}
+
 	/* Clear SLEEP mode set in INFORM1 */
 	__raw_writel(0x0, S5P_INFORM1);
 
@@ -312,10 +419,15 @@ void __init exynos_pm_init(void)
 	gic_arch_extn.irq_set_wake = exynos_irq_set_wake;
 
 	/* All wakeup disable */
-	tmp = __raw_readl(S5P_WAKEUP_MASK);
-	tmp |= ((0xFF << 8) | (0x1F << 1));
-	__raw_writel(tmp, S5P_WAKEUP_MASK);
-
+	if (soc_is_exynos5420()) {
+		tmp = __raw_readl(S5P_WAKEUP_MASK);
+		tmp |= ((0x7F << 7) | (0x1F << 1));
+		__raw_writel(tmp, S5P_WAKEUP_MASK);
+	} else {
+		tmp = __raw_readl(S5P_WAKEUP_MASK);
+		tmp |= ((0xFF << 8) | (0x1F << 1));
+		__raw_writel(tmp, S5P_WAKEUP_MASK);
+	}
 	register_syscore_ops(&exynos_pm_syscore_ops);
 	suspend_set_ops(&exynos_suspend_ops);
 }
diff --git a/arch/arm/mach-exynos/regs-pmu.h b/arch/arm/mach-exynos/regs-pmu.h
index cfbfc575..53eac2b 100644
--- a/arch/arm/mach-exynos/regs-pmu.h
+++ b/arch/arm/mach-exynos/regs-pmu.h
@@ -39,6 +39,7 @@
 #define S5P_INFORM6				S5P_PMUREG(0x0818)
 #define S5P_INFORM7				S5P_PMUREG(0x081C)
 
+#define S5P_PMU_SPARE3				S5P_PMUREG(0x090C)
 #define EXYNOS_IROM_DATA2			S5P_PMUREG(0x0988)
 
 #define S5P_ARM_CORE0_LOWPWR			S5P_PMUREG(0x1000)
@@ -113,7 +114,6 @@
 #define EXYNOS5_ARM_L2_OPTION			S5P_PMUREG(0x2608)
 #define EXYNOS5_USE_RETENTION			BIT(4)
 
-
 #define S5P_PAD_RET_MAUDIO_OPTION		S5P_PMUREG(0x3028)
 #define S5P_PAD_RET_GPIO_OPTION			S5P_PMUREG(0x3108)
 #define S5P_PAD_RET_UART_OPTION			S5P_PMUREG(0x3128)
@@ -216,6 +216,8 @@
 
 /* For EXYNOS5 */
 
+#define EXYNOS5_SYS_DISP1_BLK_CFG			S5P_SYSREG(0x0214)
+
 #define EXYNOS5_SYS_I2C_CFG					S5P_SYSREG(0x0234)
 
 #define EXYNOS5_AUTO_WDTRESET_DISABLE				S5P_PMUREG(0x0408)
diff --git a/arch/arm/plat-samsung/include/plat/map-s5p.h b/arch/arm/plat-samsung/include/plat/map-s5p.h
index c186786..cfb150e 100644
--- a/arch/arm/plat-samsung/include/plat/map-s5p.h
+++ b/arch/arm/plat-samsung/include/plat/map-s5p.h
@@ -57,6 +57,8 @@
 #define S3C_UART_OFFSET		(0x400)
 #endif
 
+#define EXYNOS5420_VA_CPU_STATE	(S5P_VA_SYSRAM + 0x28)
+
 #include <plat/map-s3c.h>
 
 #endif /* __ASM_PLAT_MAP_S5P_H */
diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c
index 60b2681..965ea69 100644
--- a/drivers/clk/samsung/clk-exynos5420.c
+++ b/drivers/clk/samsung/clk-exynos5420.c
@@ -101,6 +101,19 @@
 #define KPLL_CON0		0x28100
 #define SRC_KFC			0x28200
 #define DIV_KFC0		0x28500
+#define SRC_MASK_CPERI		0x04300
+#define SRC_MASK_TOP0		0x10300
+#define SRC_MASK_TOP1		0x10304
+#define SRC_MASK_TOP2		0x10308
+#define SRC_MASK_TOP7		0x1031C
+#define SRC_MASK_MAU		0x10334
+#define SRC_MASK_FSYS		0x10340
+#define SRC_MASK_PERIC0		0x10350
+#define SRC_MASK_PERIC1		0x10354
+#define SRC_MASK_ISP		0x10370
+#define GATE_BUS_DISP1		0x10728
+#define GATE_IP_PERIC		0x10950
+
 
 /* list of PLLs */
 enum exynos5420_plls {
@@ -180,11 +193,30 @@ static unsigned long exynos5420_clk_regs[] __initdata = {
 	DIV_KFC0,
 };
 
+static const struct samsung_clk_reg_dump exynos5420_set_clksrc[] = {
+	{ .offset = SRC_MASK_CPERI,		.value = 0xffffffff, },
+	{ .offset = SRC_MASK_TOP0,		.value = 0x11111111, },
+	{ .offset = SRC_MASK_TOP1,		.value = 0x11101111, },
+	{ .offset = SRC_MASK_TOP2,		.value = 0x11111110, },
+	{ .offset = SRC_MASK_TOP7,		.value = 0x00111100, },
+	{ .offset = SRC_MASK_DISP10,		.value = 0x11111110, },
+	{ .offset = SRC_MASK_MAU,		.value = 0x10000000, },
+	{ .offset = SRC_MASK_FSYS,		.value = 0x11111110, },
+	{ .offset = SRC_MASK_PERIC0,		.value = 0x11111110, },
+	{ .offset = SRC_MASK_PERIC1,		.value = 0x11111100, },
+	{ .offset = SRC_MASK_ISP,		.value = 0x11111000, },
+	{ .offset = GATE_BUS_DISP1,		.value = 0xffffffff, },
+	{ .offset = GATE_IP_PERIC,		.value = 0xffffffff, },
+};
+
+
 static int exynos5420_clk_suspend(void)
 {
 	samsung_clk_save(reg_base, exynos5420_save,
 				ARRAY_SIZE(exynos5420_clk_regs));
 
+	samsung_clk_restore(reg_base, exynos5420_set_clksrc,
+					ARRAY_SIZE(exynos5420_set_clksrc));
 	return 0;
 }
 
-- 
1.7.9.5

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

* [PATCH V2 3/3] ARM: dts: Add node for GPIO keys on SMDK5420
  2014-03-27  6:13 [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
  2014-03-27  6:13 ` [PATCH V2 1/3] ARM: EXYNOS5: Add PMU settings " Vikas Sajjan
  2014-03-27  6:13 ` [PATCH V2 2/3] ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420 Vikas Sajjan
@ 2014-03-27  6:13 ` Vikas Sajjan
  2014-04-15 19:06   ` Tomasz Figa
  2014-04-11 13:17 ` [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
  3 siblings, 1 reply; 12+ messages in thread
From: Vikas Sajjan @ 2014-03-27  6:13 UTC (permalink / raw)
  To: linux-arm-kernel

From: Abhilash Kesavan <a.kesavan@samsung.com>

Add support for the suspend/resume button (SW7) on SMDK5420 using
gpio-keys driver.

Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
---
 arch/arm/boot/dts/exynos5420-smdk5420.dts |   11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts b/arch/arm/boot/dts/exynos5420-smdk5420.dts
index 6910485..5ae0dd4 100644
--- a/arch/arm/boot/dts/exynos5420-smdk5420.dts
+++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts
@@ -158,6 +158,17 @@
 		};
 	};
 
+	gpio-keys {
+		compatible = "gpio-keys";
+
+		wakeup {
+			label = "SW7";
+			gpios = <&gpx0 0 1>;
+			linux,code = <143>;	/* KEY_WAKEUP */
+			gpio-key,wakeup;
+		};
+	};
+
 	hsi2c_4: i2c at 12CA0000 {
 		status = "okay";
 
-- 
1.7.9.5

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

* [PATCH V2 0/3] Add PMU and S2R support for exynos5420
  2014-03-27  6:13 [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
                   ` (2 preceding siblings ...)
  2014-03-27  6:13 ` [PATCH V2 3/3] ARM: dts: Add node for GPIO keys on SMDK5420 Vikas Sajjan
@ 2014-04-11 13:17 ` Vikas Sajjan
  2014-04-13 23:30   ` Tomasz Figa
  3 siblings, 1 reply; 12+ messages in thread
From: Vikas Sajjan @ 2014-04-11 13:17 UTC (permalink / raw)
  To: linux-arm-kernel

 Hi,

 Any comments on this series.

On Thu, Mar 27, 2014 at 11:43 AM, Vikas Sajjan <vikas.sajjan@samsung.com> wrote:
> rebased on kgene tree for-next branch
> https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/log/?h=for-next
>
> changes since v1:
>         - Addressed comments from Tomasz figa.
>         - restructured/consolidated as per Tomasz figa's PM consolidations for exynos
>
> Abhilash Kesavan (1):
>   ARM: dts: Add node for GPIO keys on SMDK5420
>
> Vikas Sajjan (2):
>   ARM: EXYNOS5: Add PMU settings for exynos5420
>   ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420
>
>  arch/arm/boot/dts/exynos5420-smdk5420.dts    |   11 +
>  arch/arm/mach-exynos/common.h                |   10 +
>  arch/arm/mach-exynos/pm.c                    |  148 +++++++++++--
>  arch/arm/mach-exynos/pmu.c                   |  307 ++++++++++++++++++++++++++
>  arch/arm/mach-exynos/regs-pmu.h              |  230 +++++++++++++++++++
>  arch/arm/plat-samsung/include/plat/map-s5p.h |    2 +
>  drivers/clk/samsung/clk-exynos5420.c         |   32 +++
>  7 files changed, 722 insertions(+), 18 deletions(-)
>
> --
> 1.7.9.5
>

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

* [PATCH V2 0/3] Add PMU and S2R support for exynos5420
  2014-04-11 13:17 ` [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
@ 2014-04-13 23:30   ` Tomasz Figa
  0 siblings, 0 replies; 12+ messages in thread
From: Tomasz Figa @ 2014-04-13 23:30 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Vikas,

Will review on Tuesday, when back at work. Same for the other series 
from you.

Best regards,
Tomasz

On 11.04.2014 15:17, Vikas Sajjan wrote:
>   Hi,
>
>   Any comments on this series.
>
> On Thu, Mar 27, 2014 at 11:43 AM, Vikas Sajjan <vikas.sajjan@samsung.com> wrote:
>> rebased on kgene tree for-next branch
>> https://git.kernel.org/cgit/linux/kernel/git/kgene/linux-samsung.git/log/?h=for-next
>>
>> changes since v1:
>>          - Addressed comments from Tomasz figa.
>>          - restructured/consolidated as per Tomasz figa's PM consolidations for exynos
>>
>> Abhilash Kesavan (1):
>>    ARM: dts: Add node for GPIO keys on SMDK5420
>>
>> Vikas Sajjan (2):
>>    ARM: EXYNOS5: Add PMU settings for exynos5420
>>    ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420
>>
>>   arch/arm/boot/dts/exynos5420-smdk5420.dts    |   11 +
>>   arch/arm/mach-exynos/common.h                |   10 +
>>   arch/arm/mach-exynos/pm.c                    |  148 +++++++++++--
>>   arch/arm/mach-exynos/pmu.c                   |  307 ++++++++++++++++++++++++++
>>   arch/arm/mach-exynos/regs-pmu.h              |  230 +++++++++++++++++++
>>   arch/arm/plat-samsung/include/plat/map-s5p.h |    2 +
>>   drivers/clk/samsung/clk-exynos5420.c         |   32 +++
>>   7 files changed, 722 insertions(+), 18 deletions(-)
>>
>> --
>> 1.7.9.5
>>

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

* [PATCH V2 1/3] ARM: EXYNOS5: Add PMU settings for exynos5420
  2014-03-27  6:13 ` [PATCH V2 1/3] ARM: EXYNOS5: Add PMU settings " Vikas Sajjan
@ 2014-04-15 18:57   ` Tomasz Figa
  2014-04-28 12:36     ` Vikas Sajjan
  0 siblings, 1 reply; 12+ messages in thread
From: Tomasz Figa @ 2014-04-15 18:57 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Vikas,

Basically the same comments apply as for the series:

[PATCH v2 0/3] Add initial support of PMU for exynos5260
(https://www.mail-archive.com/linux-samsung-soc at vger.kernel.org/msg27339.html)

In addition to above, please see some comments inline.

On 27.03.2014 07:13, Vikas Sajjan wrote:
> Add intial PMU settings for exynos5420. This is required for
> future S2R and Switching support.
>
> Signed-off-by: Thomas Abraham <thomas.ab@samsung.com>
> Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
> Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
> ---
>   arch/arm/mach-exynos/common.h   |   10 ++
>   arch/arm/mach-exynos/pmu.c      |  307 +++++++++++++++++++++++++++++++++++++++
>   arch/arm/mach-exynos/regs-pmu.h |  228 +++++++++++++++++++++++++++++
>   3 files changed, 545 insertions(+)
>
> diff --git a/arch/arm/mach-exynos/common.h b/arch/arm/mach-exynos/common.h
> index 9ef3f83..347afc2 100644
> --- a/arch/arm/mach-exynos/common.h
> +++ b/arch/arm/mach-exynos/common.h
> @@ -15,6 +15,16 @@
>   #include <linux/reboot.h>
>   #include <linux/of.h>
>
> +#define EXYNOS5420_USE_STANDBY_WFI_ALL	(EXYNOS5420_ARM_USE_STANDBY_WFI0  \
> +					 | EXYNOS5420_ARM_USE_STANDBY_WFI1  \
> +					 | EXYNOS5420_ARM_USE_STANDBY_WFI2  \
> +					 | EXYNOS5420_ARM_USE_STANDBY_WFI3  \
> +					 | EXYNOS5420_KFC_USE_STANDBY_WFI0  \
> +					 | EXYNOS5420_KFC_USE_STANDBY_WFI1  \
> +					 | EXYNOS5420_KFC_USE_STANDBY_WFI2  \
> +					 | EXYNOS5420_KFC_USE_STANDBY_WFI3)
> +
> +
>   void mct_init(void __iomem *base, int irq_g0, int irq_l0, int irq_l1);
>
>   struct map_desc;
> diff --git a/arch/arm/mach-exynos/pmu.c b/arch/arm/mach-exynos/pmu.c
> index 05c7ce1..f69a6ed 100644
> --- a/arch/arm/mach-exynos/pmu.c
> +++ b/arch/arm/mach-exynos/pmu.c
> @@ -12,9 +12,14 @@
>   #include <linux/io.h>
>   #include <linux/kernel.h>
>   #include <linux/bug.h>
> +#include <linux/cpumask.h>
> +#include <linux/delay.h>
> +#include <linux/pm.h>
>
>   #include <plat/cpu.h>
>
> +#include <asm/cputype.h>
> +
>   #include "common.h"
>   #include "regs-pmu.h"
>
> @@ -318,6 +323,212 @@ static const struct exynos_pmu_conf exynos5250_pmu_config[] = {
>   	{ PMU_TABLE_END,},
>   };
>
> +static const struct exynos_pmu_conf exynos5420_pmu_config[] = {
> +	/* { .reg = address, .val = { AFTR, LPA, SLEEP } */
> +	{ EXYNOS5_ARM_CORE0_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_DIS_IRQ_ARM_CORE0_LOCAL_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_DIS_IRQ_ARM_CORE0_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_ARM_CORE1_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_DIS_IRQ_ARM_CORE1_LOCAL_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_DIS_IRQ_ARM_CORE1_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_ARM_CORE2_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_ARM_CORE2_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_ARM_CORE2_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_ARM_CORE3_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_ARM_CORE3_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_ARM_CORE3_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_KFC_CORE0_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_KFC_CORE0_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_KFC_CORE0_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_KFC_CORE1_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_KFC_CORE1_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_KFC_CORE1_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_KFC_CORE2_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_KFC_CORE2_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_KFC_CORE2_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_KFC_CORE3_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_KFC_CORE3_LOCAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_DIS_IRQ_KFC_CORE3_CENTRAL_SYS_PWR_REG,	{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_ISP_ARM_SYS_PWR_REG,				{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_DIS_IRQ_ISP_ARM_LOCAL_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_DIS_IRQ_ISP_ARM_CENTRAL_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_ARM_COMMON_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_KFC_COMMON_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_ARM_L2_SYS_PWR_REG,				{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_KFC_L2_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_ACLKSTOP_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_SCLKSTOP_SYS_PWR_REG,			{ 0x1, 0x0, 0x1} },
> +	{ EXYNOS5_CMU_RESET_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_CMU_ACLKSTOP_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_SCLKSTOP_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x1} },
> +	{ EXYNOS5_CMU_RESET_SYSMEM_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_DRAM_FREQ_DOWN_SYS_PWR_REG,			{ 0x1, 0x0, 0x1} },
> +	{ EXYNOS5_DDRPHY_DLLOFF_SYS_PWR_REG,			{ 0x1, 0x1, 0x1} },
> +	{ EXYNOS5_DDRPHY_DLLLOCK_SYS_PWR_REG,			{ 0x1, 0x0, 0x1} },
> +	{ EXYNOS5_APLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_MPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_VPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_EPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_BPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_CPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_DPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_IPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_KPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_MPLLUSER_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_BPLLUSER_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_RPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_SPLL_SYSCLK_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_TOP_BUS_SYS_PWR_REG,				{ 0x3, 0x0, 0x0} },
> +	{ EXYNOS5_TOP_RETENTION_SYS_PWR_REG,			{ 0x1, 0x1, 0x1} },
> +	{ EXYNOS5_TOP_PWR_SYS_PWR_REG,				{ 0x3, 0x3, 0x0} },
> +	{ EXYNOS5_TOP_BUS_SYSMEM_SYS_PWR_REG,			{ 0x3, 0x0, 0x0} },
> +	{ EXYNOS5_TOP_RETENTION_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x1} },
> +	{ EXYNOS5_TOP_PWR_SYSMEM_SYS_PWR_REG,			{ 0x3, 0x0, 0x0} },
> +	{ EXYNOS5_LOGIC_RESET_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_OSCCLK_GATE_SYS_PWR_REG,			{ 0x1, 0x0, 0x1} },
> +	{ EXYNOS5_LOGIC_RESET_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_OSCCLK_GATE_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_INTRAM_MEM_SYS_PWR_REG,			{ 0x3, 0x0, 0x3} },
> +	{ EXYNOS5420_INTROM_MEM_SYS_PWR_REG,			{ 0x3, 0x0, 0x3} },
> +	{ EXYNOS5_PAD_RETENTION_DRAM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_PAD_RETENTION_MAU_SYS_PWR_REG,		{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_JTAG_SYS_PWR_REG,		{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_DRAM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_UART_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_MMC0_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_MMC1_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_MMC2_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_HSI_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_EBIA_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_EBIB_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_SPI_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5420_PAD_RETENTION_DRAM_COREBLK_SYS_PWR_REG,	{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_PAD_ISOLATION_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_PAD_ISOLATION_SYSMEM_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_PAD_ALV_SEL_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_XUSBXTI_SYS_PWR_REG,				{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_XXTI_SYS_PWR_REG,				{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_EXT_REGULATOR_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_GPIO_MODE_SYS_PWR_REG,			{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_GPIO_MODE_SYSMEM_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_GPIO_MODE_MAU_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_TOP_ASB_RESET_SYS_PWR_REG,			{ 0x1, 0x1, 0x0} },
> +	{ EXYNOS5_TOP_ASB_ISOLATION_SYS_PWR_REG,		{ 0x1, 0x0, 0x0} },
> +	{ EXYNOS5_GSCL_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5_ISP_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5_MFC_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5_G3D_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5420_DISP1_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5420_MAU_SYS_PWR_REG,				{ 0x7, 0x7, 0x0} },
> +	{ EXYNOS5420_G2D_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5420_MSC_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5420_FSYS_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5420_FSYS2_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5420_PSGEN_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5420_PERIC_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5420_WCORE_SYS_PWR_REG,				{ 0x7, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_RESET_ISP_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG,			{ 0x0, 0x0, 0x0} },
> +	{ EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG,		{ 0x0, 0x0, 0x0} },

style: Lines over 80 characters.

> +	{ PMU_TABLE_END,},
> +};
> +
> +void __iomem *exynos5_list_disable_pmu_reg[] = {

static void __iomem * const

> +	EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,
> +	EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,
> +	EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,
> +	EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,
> +	EXYNOS5_CMU_CLKSTOP_DISP1_SYS_PWR_REG,
> +	EXYNOS5_CMU_CLKSTOP_MAU_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_DISP1_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_MAU_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_DISP1_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_MAU_SYS_PWR_REG,
> +};
> +
> +void __iomem *exynos5420_list_disable_pmu_reg[] = {

static void __iomem * const

> +	EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,
> +	EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,
> +	EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,
> +	EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,
> +	EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG,
> +	EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG,
> +	EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG,
> +	EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG,
> +	EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG,
> +	EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG,
> +	EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG,
> +	EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,
> +	EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG,
> +	EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_ISP_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,
> +	EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG,
> +	EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG,
> +};
> +
>   static void __iomem * const exynos5_list_both_cnt_feed[] = {
>   	EXYNOS5_ARM_CORE0_OPTION,
>   	EXYNOS5_ARM_CORE1_OPTION,
> @@ -371,6 +582,21 @@ static void exynos5_init_pmu(void)
>   	}
>   }
>
> +/*
> + * exynos_set_core_flag - set the cluster id to IROM register
> + *			  to ensure that we wake up with the
> + *			  current cluster.
> + */
> +static void exynos_set_core_flag(void)
> +{
> +	int cluster_id = (read_cpuid_mpidr() >> 8) & 0xf;

Please use proper macros to extract cluster ID from mpidr value.

> +
> +	if (cluster_id)
> +		__raw_writel(1, EXYNOS_IROM_DATA2);
> +	else
> +		__raw_writel(0, EXYNOS_IROM_DATA2);

Couldn't __raw_writel(cluster_id, EXYNOS_IROM_DATA2) work here?

> +}
> +
>   void exynos_sys_powerdown_conf(enum sys_powerdown mode)
>   {
>   	unsigned int i;
> @@ -378,6 +604,9 @@ void exynos_sys_powerdown_conf(enum sys_powerdown mode)
>   	if (soc_is_exynos5250())
>   		exynos5_init_pmu();
>
> +	if (soc_is_exynos5420())
> +		exynos_set_core_flag();
> +
>   	for (i = 0; (exynos_pmu_config[i].reg != PMU_TABLE_END) ; i++)
>   		__raw_writel(exynos_pmu_config[i].val[mode],
>   				exynos_pmu_config[i].reg);
> @@ -389,9 +618,27 @@ void exynos_sys_powerdown_conf(enum sys_powerdown mode)
>   	}
>   }
>
> +static void exynos5_power_off(void)
> +{
> +	unsigned int tmp;
> +
> +	pr_info("Power down.\n");
> +	tmp = __raw_readl(EXYNOS_PS_HOLD_CONTROL);
> +	tmp &= ~(1 << 8);

This is wrong as it assumes fixed polarity of PSHOLD pin. Instead, it 
should just invert the setting. A simple XOR should be enough.

> +	__raw_writel(tmp, EXYNOS_PS_HOLD_CONTROL);
> +
> +	/* Wait a little so we don't give a false warning below */
> +	mdelay(100);
> +
> +	pr_err("Power down failed, please power off system manually.\n");
> +	while (1)
> +		;
> +}
> +
>   static int __init exynos_pmu_init(void)
>   {
>   	unsigned int value;
> +	int i;
>
>   	exynos_pmu_config = exynos4210_pmu_config;
>
> @@ -416,6 +663,66 @@ static int __init exynos_pmu_init(void)
>
>   		exynos_pmu_config = exynos5250_pmu_config;
>   		pr_info("EXYNOS5250 PMU Initialize\n");
> +	} else if (soc_is_exynos5420()) {
> +		/*
> +		 * Set the CMU_RESET, CMU_SYSCLK and CMU_CLKSTOP registers
> +		 * for local power blocks to Low initially as per Table 8-4:
> +		 * "System-Level Power-Down Configuration Registers".
> +		 */
> +		for (i = 0;
> +			i < ARRAY_SIZE(exynos5420_list_disable_pmu_reg);
> +			i++)
> +			__raw_writel(0x0, exynos5420_list_disable_pmu_reg[i]);
> +		/* Time taken to stabilized XXTI clock */
> +		__raw_writel(0x005dc, EXYNOS5_XXTI_DURATION3);
> +
> +		/* Enable USE_STANDBY_WFI for all CORE */
> +		__raw_writel(EXYNOS5420_USE_STANDBY_WFI_ALL,
> +				S5P_CENTRAL_SEQ_OPTION);
> +
> +		value = __raw_readl(EXYNOS_L2_OPTION(0));
> +		value &= ~EXYNOS5_USE_RETENTION;
> +		__raw_writel(value, EXYNOS_L2_OPTION(0));
> +
> +		value = __raw_readl(EXYNOS_L2_OPTION(1));
> +		value &= ~EXYNOS5_USE_RETENTION;
> +		__raw_writel(value, EXYNOS_L2_OPTION(1));
> +
> +		/*
> +		 * If L2_COMMON is turned off, clocks related to ATB async
> +		 * bridge are gated. Thus, when ISP power is gated, LPI
> +		 * may get stuck.
> +		 */
> +		value = __raw_readl(EXYNOS5420_LPI_MASK);
> +		value |= EXYNOS5420_ATB_ISP_ARM;
> +		__raw_writel(value, EXYNOS5420_LPI_MASK);
> +		value = __raw_readl(EXYNOS5420_LPI_MASK1);
> +		value |= EXYNOS5420_ATB_KFC;
> +		__raw_writel(value, EXYNOS5420_LPI_MASK1);
> +
> +		/* Prevent issue of new bus request from L2 memory */
> +		value = __raw_readl(EXYNOS5420_ARM_COMMON_OPTION);
> +		value |= EXYNOS5_SKIP_DEACTIVATE_ACEACP_IN_PWDN;
> +		__raw_writel(value, EXYNOS5420_ARM_COMMON_OPTION);
> +
> +		value = __raw_readl(EXYNOS5420_KFC_COMMON_OPTION);
> +		value |= EXYNOS5_SKIP_DEACTIVATE_ACEACP_IN_PWDN;
> +		__raw_writel(value, EXYNOS5420_KFC_COMMON_OPTION);
> +
> +		/*
> +		 * This setting is to reduce suspend/resume time.
> +		 */
> +		__raw_writel(DUR_WAIT_RESET, EXYNOS5420_LOGIC_RESET_DURATION3);
> +
> +		/* Serialized CPU wakeup of Eagle */
> +		__raw_writel(SPREAD_ENABLE, EXYNOS5420_ARM_INTR_SPREAD_ENABLE);
> +		__raw_writel(SPREAD_USE_STANDWFI,
> +			EXYNOS5420_ARM_INTR_SPREAD_USE_STANDBYWFI);
> +		__raw_writel(0x1, EXYNOS5420_UP_SCHEDULER);
> +
> +		exynos_pmu_config = exynos5420_pmu_config;
> +		pm_power_off = exynos5_power_off;
> +		pr_info("EXYNOS5420 PMU Initialized\n");
>   	} else {
>   		pr_info("EXYNOS: PMU not supported\n");
>   	}
> diff --git a/arch/arm/mach-exynos/regs-pmu.h b/arch/arm/mach-exynos/regs-pmu.h
> index 4f6a256..cfbfc575 100644
> --- a/arch/arm/mach-exynos/regs-pmu.h
> +++ b/arch/arm/mach-exynos/regs-pmu.h
> @@ -39,6 +39,8 @@
>   #define S5P_INFORM6				S5P_PMUREG(0x0818)
>   #define S5P_INFORM7				S5P_PMUREG(0x081C)
>
> +#define EXYNOS_IROM_DATA2			S5P_PMUREG(0x0988)

Is this register available on Exynos4 too?

> +
>   #define S5P_ARM_CORE0_LOWPWR			S5P_PMUREG(0x1000)
>   #define S5P_DIS_IRQ_CORE0			S5P_PMUREG(0x1004)
>   #define S5P_DIS_IRQ_CENTRAL0			S5P_PMUREG(0x1008)
> @@ -108,6 +110,10 @@
>   #define S5P_ARM_CORE1_CONFIGURATION		S5P_PMUREG(0x2080)
>   #define S5P_ARM_CORE1_STATUS			S5P_PMUREG(0x2084)
>
> +#define EXYNOS5_ARM_L2_OPTION			S5P_PMUREG(0x2608)
> +#define EXYNOS5_USE_RETENTION			BIT(4)
> +
> +
>   #define S5P_PAD_RET_MAUDIO_OPTION		S5P_PMUREG(0x3028)
>   #define S5P_PAD_RET_GPIO_OPTION			S5P_PMUREG(0x3108)
>   #define S5P_PAD_RET_UART_OPTION			S5P_PMUREG(0x3128)
> @@ -121,6 +127,36 @@
>
>   #define S5P_CHECK_SLEEP				0x00000BAD
>
> +#define S5P_ARM_CORE0_CONFIGURATION		S5P_PMUREG(0x2000)
> +#define S5P_ARM_CORE0_STATUS			S5P_PMUREG(0x2004)
> +#define S5P_ARM_CORE0_OPTION			S5P_PMUREG(0x2008)
> +
> +#define EXYNOS_ARM_CORE_OPTION(_nr)		(S5P_ARM_CORE0_OPTION \
> +						+ ((_nr) * 0x80))
> +#define EXYNOS_ARM_CORE_STATUS(_nr)		(S5P_ARM_CORE0_STATUS \
> +						+ ((_nr) * 0x80))
> +#define EXYNOS_ARM_CORE_CONFIGURATION(_nr)	\
> +			(S5P_ARM_CORE0_CONFIGURATION + ((_nr) * 0x80))
> +#define EXYNOS_CORE_LOCAL_PWR_EN		0x3
> +
> +#define EXYNOS_ARM_COMMON_CONFIGURATION		S5P_PMUREG(0x2500)
> +#define EXYNOS_ARM_COMMON_STATUS		S5P_PMUREG(0x2504)
> +#define EXYNOS_COMMON_CONFIGURATION(_nr)	\
> +			(EXYNOS_ARM_COMMON_CONFIGURATION + ((_nr) * 0x80))
> +#define EXYNOS_COMMON_STATUS(_nr)		\
> +			(EXYNOS_COMMON_CONFIGURATION(_nr) + 0x4)
> +#define EXYNOS_COMMON_OPTION(_nr)		\
> +			(EXYNOS_COMMON_CONFIGURATION(_nr) + 0x8)
> +
> +#define EXYNOS_ARM_L2_CONFIGURATION		S5P_PMUREG(0x2600)
> +#define EXYNOS_L2_CONFIGURATION(_nr)		\
> +			(EXYNOS_ARM_L2_CONFIGURATION + ((_nr) * 0x80))
> +#define EXYNOS_L2_STATUS(_nr)			\
> +			(EXYNOS_L2_CONFIGURATION(_nr) + 0x4)
> +#define EXYNOS_L2_OPTION(_nr)			\
> +			(EXYNOS_L2_CONFIGURATION(_nr) + 0x8)
> +#define EXYNOS_L2_COMMON_PWR_EN			0x3
> +
>   /* Only for EXYNOS4210 */
>   #define S5P_CMU_CLKSTOP_LCD1_LOWPWR	S5P_PMUREG(0x1154)
>   #define S5P_CMU_RESET_LCD1_LOWPWR	S5P_PMUREG(0x1174)
> @@ -313,4 +349,196 @@
>
>   #define EXYNOS5_OPTION_USE_RETENTION				(1 << 4)
>
> +/* Only for EXYNOS5420 */
> +#define EXYNOS5420_ISP_ARM_OPTION				S5P_PMUREG(0x2488)
> +#define EXYNOS5420_L2RSTDISABLE_VALUE				(1 << 3)
> +
> +#define EXYNOS5420_LPI_MASK					S5P_PMUREG(0x0004)
> +#define EXYNOS5420_LPI_MASK1					S5P_PMUREG(0x0008)
> +#define EXYNOS5420_UFS						(1 << 8)
> +#define EXYNOS5420_ATB_KFC					(1 << 13)
> +#define EXYNOS5420_ATB_ISP_ARM					(1 << 19)
> +#define EXYNOS5420_EMULATION					(1 << 31)
> +#define ATB_ISP_ARM						(1 << 12)
> +#define ATB_KFC							(1 << 13)
> +#define ATB_NOC							(1 << 14)
> +
> +#define EXYNOS5420_ARM_INTR_SPREAD_ENABLE			S5P_PMUREG(0x0100)
> +#define EXYNOS5420_ARM_INTR_SPREAD_USE_STANDBYWFI		S5P_PMUREG(0x0104)
> +#define EXYNOS5420_UP_SCHEDULER					S5P_PMUREG(0x0120)
> +#define SPREAD_ENABLE						0xF
> +#define SPREAD_USE_STANDWFI					0xF
> +
> +#define EXYNOS5420_BB_CON1					S5P_PMUREG(0x0784)
> +#define EXYNOS5420_BB_SEL_EN					(1 << 31)
> +#define EXYNOS5420_BB_PMOS_EN					(1 << 7)
> +#define EXYNOS5420_BB_1300X					0XF
> +
> +#define EXYNOS5420_ARM_CORE2_SYS_PWR_REG			S5P_PMUREG(0x1020)
> +#define EXYNOS5420_DIS_IRQ_ARM_CORE2_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1024)
> +#define EXYNOS5420_DIS_IRQ_ARM_CORE2_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1028)
> +#define EXYNOS5420_ARM_CORE3_SYS_PWR_REG			S5P_PMUREG(0x1030)
> +#define EXYNOS5420_DIS_IRQ_ARM_CORE3_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1034)
> +#define EXYNOS5420_DIS_IRQ_ARM_CORE3_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1038)
> +#define EXYNOS5420_KFC_CORE0_SYS_PWR_REG			S5P_PMUREG(0x1040)
> +#define EXYNOS5420_DIS_IRQ_KFC_CORE0_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1044)
> +#define EXYNOS5420_DIS_IRQ_KFC_CORE0_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1048)
> +#define EXYNOS5420_KFC_CORE1_SYS_PWR_REG			S5P_PMUREG(0x1050)
> +#define EXYNOS5420_DIS_IRQ_KFC_CORE1_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1054)
> +#define EXYNOS5420_DIS_IRQ_KFC_CORE1_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1058)
> +#define EXYNOS5420_KFC_CORE2_SYS_PWR_REG			S5P_PMUREG(0x1060)
> +#define EXYNOS5420_DIS_IRQ_KFC_CORE2_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1064)
> +#define EXYNOS5420_DIS_IRQ_KFC_CORE2_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1068)
> +#define EXYNOS5420_KFC_CORE3_SYS_PWR_REG			S5P_PMUREG(0x1070)
> +#define EXYNOS5420_DIS_IRQ_KFC_CORE3_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1074)
> +#define EXYNOS5420_DIS_IRQ_KFC_CORE3_CENTRAL_SYS_PWR_REG	S5P_PMUREG(0x1078)
> +#define EXYNOS5420_ISP_ARM_SYS_PWR_REG				S5P_PMUREG(0x1090)
> +#define EXYNOS5420_DIS_IRQ_ISP_ARM_LOCAL_SYS_PWR_REG		S5P_PMUREG(0x1094)
> +#define EXYNOS5420_DIS_IRQ_ISP_ARM_CENTRAL_SYS_PWR_REG		S5P_PMUREG(0x1098)
> +#define EXYNOS5420_ARM_COMMON_SYS_PWR_REG			S5P_PMUREG(0x10A0)
> +#define EXYNOS5420_KFC_COMMON_SYS_PWR_REG			S5P_PMUREG(0x10B0)
> +#define EXYNOS5420_KFC_L2_SYS_PWR_REG				S5P_PMUREG(0x10D0)
> +#define EXYNOS5420_DPLL_SYSCLK_SYS_PWR_REG			S5P_PMUREG(0x1158)
> +#define EXYNOS5420_IPLL_SYSCLK_SYS_PWR_REG			S5P_PMUREG(0x115C)
> +#define EXYNOS5420_KPLL_SYSCLK_SYS_PWR_REG			S5P_PMUREG(0x1160)
> +#define EXYNOS5420_RPLL_SYSCLK_SYS_PWR_REG                      S5P_PMUREG(0x1174)
> +#define EXYNOS5420_SPLL_SYSCLK_SYS_PWR_REG                      S5P_PMUREG(0x1178)
> +#define EXYNOS5420_INTRAM_MEM_SYS_PWR_REG                       S5P_PMUREG(0x11B8)
> +#define EXYNOS5420_INTROM_MEM_SYS_PWR_REG                       S5P_PMUREG(0x11BC)
> +#define EXYNOS5420_ONENANDXL_MEM_SYS_PWR			S5P_PMUREG(0x11C0)
> +#define EXYNOS5420_USBDEV_MEM_SYS_PWR				S5P_PMUREG(0x11CC)
> +#define EXYNOS5420_USBDEV1_MEM_SYS_PWR				S5P_PMUREG(0x11D0)
> +#define EXYNOS5420_SDMMC_MEM_SYS_PWR				S5P_PMUREG(0x11D4)
> +#define EXYNOS5420_CSSYS_MEM_SYS_PWR				S5P_PMUREG(0x11D8)
> +#define EXYNOS5420_SECSS_MEM_SYS_PWR				S5P_PMUREG(0x11DC)
> +#define EXYNOS5420_ROTATOR_MEM_SYS_PWR				S5P_PMUREG(0x11E0)
> +#define EXYNOS5420_INTRAM_MEM_SYS_PWR				S5P_PMUREG(0x11E4)
> +#define EXYNOS5420_INTROM_MEM_SYS_PWR				S5P_PMUREG(0x11E8)
> +#define EXYNOS5420_PAD_RETENTION_JTAG_SYS_PWR_REG		S5P_PMUREG(0x1208)
> +#define EXYNOS5420_PAD_RETENTION_DRAM_SYS_PWR_REG		S5P_PMUREG(0x1210)
> +#define EXYNOS5420_PAD_RETENTION_UART_SYS_PWR_REG		S5P_PMUREG(0x1214)
> +#define EXYNOS5420_PAD_RETENTION_MMC0_SYS_PWR_REG		S5P_PMUREG(0x1218)
> +#define EXYNOS5420_PAD_RETENTION_MMC1_SYS_PWR_REG		S5P_PMUREG(0x121C)
> +#define EXYNOS5420_PAD_RETENTION_MMC2_SYS_PWR_REG		S5P_PMUREG(0x1220)
> +#define EXYNOS5420_PAD_RETENTION_HSI_SYS_PWR_REG		S5P_PMUREG(0x1224)
> +#define EXYNOS5420_PAD_RETENTION_EBIA_SYS_PWR_REG		S5P_PMUREG(0x1228)
> +#define EXYNOS5420_PAD_RETENTION_EBIB_SYS_PWR_REG		S5P_PMUREG(0x122C)
> +#define EXYNOS5420_PAD_RETENTION_SPI_SYS_PWR_REG		S5P_PMUREG(0x1230)
> +#define EXYNOS5420_PAD_RETENTION_DRAM_COREBLK_SYS_PWR_REG	S5P_PMUREG(0x1234)
> +#define EXYNOS5420_DISP1_SYS_PWR_REG				S5P_PMUREG(0x1410)
> +#define EXYNOS5420_MAU_SYS_PWR_REG				S5P_PMUREG(0x1414)
> +#define EXYNOS5420_G2D_SYS_PWR_REG				S5P_PMUREG(0x1418)
> +#define EXYNOS5420_MSC_SYS_PWR_REG				S5P_PMUREG(0x141C)
> +#define EXYNOS5420_FSYS_SYS_PWR_REG				S5P_PMUREG(0x1420)
> +#define EXYNOS5420_FSYS2_SYS_PWR_REG				S5P_PMUREG(0x1424)
> +#define EXYNOS5420_PSGEN_SYS_PWR_REG				S5P_PMUREG(0x1428)
> +#define EXYNOS5420_PERIC_SYS_PWR_REG				S5P_PMUREG(0x142C)
> +#define EXYNOS5420_WCORE_SYS_PWR_REG				S5P_PMUREG(0x1430)
> +#define EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG		S5P_PMUREG(0x1490)
> +#define EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG			S5P_PMUREG(0x1494)
> +#define EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG			S5P_PMUREG(0x1498)
> +#define EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG			S5P_PMUREG(0x149C)
> +#define EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG			S5P_PMUREG(0x14A0)
> +#define EXYNOS5420_CMU_CLKSTOP_FSYS2_SYS_PWR_REG		S5P_PMUREG(0x14A4)
> +#define EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG		S5P_PMUREG(0x14A8)
> +#define EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG		S5P_PMUREG(0x14AC)
> +#define EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG		S5P_PMUREG(0x14B0)
> +#define EXYNOS5420_CMU_SYSCLK_TOPPWR_SYS_PWR_REG		S5P_PMUREG(0x14BC)
> +#define EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG			S5P_PMUREG(0x14D0)
> +#define EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG			S5P_PMUREG(0x14D4)
> +#define EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG			S5P_PMUREG(0x14D8)
> +#define EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG			S5P_PMUREG(0x14DC)
> +#define EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG			S5P_PMUREG(0x14E0)
> +#define EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG			S5P_PMUREG(0x14E4)
> +#define EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG			S5P_PMUREG(0x14E8)
> +#define EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG			S5P_PMUREG(0x14EC)
> +#define EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG			S5P_PMUREG(0x14F0)
> +#define EXYNOS5420_CMU_SYSCLK_SYSMEM_TOPPWR_SYS_PWR_REG		S5P_PMUREG(0x14F4)
> +#define EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG			S5P_PMUREG(0x1570)
> +#define EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG			S5P_PMUREG(0x1574)
> +#define EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG			S5P_PMUREG(0x1578)
> +#define EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG			S5P_PMUREG(0x157C)
> +#define EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG			S5P_PMUREG(0x1590)
> +#define EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG			S5P_PMUREG(0x1594)
> +#define EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG			S5P_PMUREG(0x1598)
> +#define EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG			S5P_PMUREG(0x159C)
> +#define EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG			S5P_PMUREG(0x15A0)
> +#define EXYNOS5420_SFR_AXI_CGDIS1				S5P_PMUREG(0x15E4)
> +#define EXYNOS_ARM_CORE2_CONFIGURATION				S5P_PMUREG(0x2100)
> +#define EXYNOS5420_ARM_CORE2_OPTION				S5P_PMUREG(0x2108)
> +#define EXYNOS_ARM_CORE3_CONFIGURATION				S5P_PMUREG(0x2180)
> +#define EXYNOS5420_ARM_CORE3_OPTION				S5P_PMUREG(0x2188)
> +#define EXYNOS5420_ARM_COMMON_STATUS				S5P_PMUREG(0x2504)
> +#define EXYNOS5420_ARM_COMMON_OPTION				S5P_PMUREG(0x2508)
> +#define EXYNOS5420_KFC_COMMON_STATUS				S5P_PMUREG(0x2584)
> +#define EXYNOS5420_KFC_COMMON_OPTION				S5P_PMUREG(0x2588)
> +#define EXYNOS5420_LOGIC_RESET_DURATION3			S5P_PMUREG(0x2D1C)
> +
> +#define EXYNOS5420_PAD_RET_GPIO_OPTION				S5P_PMUREG(0x30C8)
> +#define EXYNOS5420_PAD_RET_UART_OPTION				S5P_PMUREG(0x30E8)
> +#define EXYNOS5420_PAD_RET_MMCA_OPTION				S5P_PMUREG(0x3108)
> +#define EXYNOS5420_PAD_RET_MMCB_OPTION				S5P_PMUREG(0x3128)
> +#define EXYNOS5420_PAD_RET_MMCC_OPTION				S5P_PMUREG(0x3148)
> +#define EXYNOS5420_PAD_RET_HSI_OPTION				S5P_PMUREG(0x3168)
> +#define EXYNOS5420_PAD_RET_SPI_OPTION				S5P_PMUREG(0x31C8)
> +#define EXYNOS5420_PAD_RET_DRAM_COREBLK_OPTION			S5P_PMUREG(0x31E8)
> +#define EXYNOS_PAD_RET_DRAM_OPTION				S5P_PMUREG(0x3008)
> +#define EXYNOS_PAD_RET_MAUDIO_OPTION				S5P_PMUREG(0x3028)
> +#define EXYNOS_PAD_RET_JTAG_OPTION				S5P_PMUREG(0x3048)
> +#define EXYNOS_PAD_RET_GPIO_OPTION				S5P_PMUREG(0x3108)
> +#define EXYNOS_PAD_RET_UART_OPTION				S5P_PMUREG(0x3128)
> +#define EXYNOS_PAD_RET_MMCA_OPTION				S5P_PMUREG(0x3148)
> +#define EXYNOS_PAD_RET_MMCB_OPTION				S5P_PMUREG(0x3168)
> +#define EXYNOS_PAD_RET_EBIA_OPTION				S5P_PMUREG(0x3188)
> +#define EXYNOS_PAD_RET_EBIB_OPTION				S5P_PMUREG(0x31A8)
> +
> +#define EXYNOS_PS_HOLD_CONTROL					S5P_PMUREG(0x330C)
> +#define EXYNOS5_XXTI_DURATION3					S5P_PMUREG(0x343C)
> +
> +/* For SYS_PWR_REG */
> +#define EXYNOS_SYS_PWR_CFG					(1 << 0)
> +
> +#define EXYNOS5420_MFC_CONFIGURATION				S5P_PMUREG(0x4060)
> +#define EXYNOS5420_MFC_STATUS					S5P_PMUREG(0x4064)
> +#define EXYNOS5420_MFC_OPTION					S5P_PMUREG(0x4068)
> +#define EXYNOS5420_G3D_CONFIGURATION				S5P_PMUREG(0x4080)
> +#define EXYNOS5420_G3D_STATUS					S5P_PMUREG(0x4084)
> +#define EXYNOS5420_G3D_OPTION					S5P_PMUREG(0x4088)
> +#define EXYNOS5420_DISP0_CONFIGURATION				S5P_PMUREG(0x40A0)
> +#define EXYNOS5420_DISP0_STATUS					S5P_PMUREG(0x40A4)
> +#define EXYNOS5420_DISP0_OPTION					S5P_PMUREG(0x40A8)
> +#define EXYNOS5420_DISP1_CONFIGURATION				S5P_PMUREG(0x40C0)
> +#define EXYNOS5420_DISP1_STATUS					S5P_PMUREG(0x40C4)
> +#define EXYNOS5420_DISP1_OPTION					S5P_PMUREG(0x40C8)
> +#define EXYNOS5420_MAU_CONFIGURATION				S5P_PMUREG(0x40E0)
> +#define EXYNOS5420_MAU_STATUS					S5P_PMUREG(0x40E4)
> +#define EXYNOS5420_MAU_OPTION					S5P_PMUREG(0x40E8)
> +#define EXYNOS5420_FSYS2_OPTION					S5P_PMUREG(0x4168)
> +#define EXYNOS5420_PSGEN_OPTION					S5P_PMUREG(0x4188)

style: A lot of lines over 80 characters.

> +
> +/* For EXYNOS_CENTRAL_SEQ_OPTION */
> +#define EXYNOS5_USE_STANDBYWFI_ARM_CORE0			(1 << 16)
> +#define EXYNOS5_USE_STANDBYWFI_ARM_CORE1			(1 << 17)
> +#define EXYNOS5_USE_STANDBYWFE_ARM_CORE0			(1 << 24)
> +#define EXYNOS5_USE_STANDBYWFE_ARM_CORE1			(1 << 25)
> +
> +#define EXYNOS5420_ARM_USE_STANDBY_WFI0				(1 << 4)
> +#define EXYNOS5420_ARM_USE_STANDBY_WFI1				(1 << 5)
> +#define EXYNOS5420_ARM_USE_STANDBY_WFI2				(1 << 6)
> +#define EXYNOS5420_ARM_USE_STANDBY_WFI3				(1 << 7)
> +#define EXYNOS5420_KFC_USE_STANDBY_WFI0				(1 << 8)
> +#define EXYNOS5420_KFC_USE_STANDBY_WFI1				(1 << 9)
> +#define EXYNOS5420_KFC_USE_STANDBY_WFI2				(1 << 10)
> +#define EXYNOS5420_KFC_USE_STANDBY_WFI3				(1 << 11)
> +#define EXYNOS5420_ARM_USE_STANDBY_WFE0				(1 << 16)
> +#define EXYNOS5420_ARM_USE_STANDBY_WFE1				(1 << 17)
> +#define EXYNOS5420_ARM_USE_STANDBY_WFE2				(1 << 18)
> +#define EXYNOS5420_ARM_USE_STANDBY_WFE3				(1 << 19)
> +#define EXYNOS5420_KFC_USE_STANDBY_WFE0				(1 << 20)
> +#define EXYNOS5420_KFC_USE_STANDBY_WFE1				(1 << 21)
> +#define EXYNOS5420_KFC_USE_STANDBY_WFE2				(1 << 22)
> +#define EXYNOS5420_KFC_USE_STANDBY_WFE3				(1 << 23)

BIT() macro could be used. The same for similar bit definitions above.

Best regards,
Tomasz

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

* [PATCH V2 2/3] ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420
  2014-03-27  6:13 ` [PATCH V2 2/3] ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420 Vikas Sajjan
@ 2014-04-15 19:03   ` Tomasz Figa
  2014-04-15 19:04     ` Tomasz Figa
  2014-04-28 12:39     ` Vikas Sajjan
  0 siblings, 2 replies; 12+ messages in thread
From: Tomasz Figa @ 2014-04-15 19:03 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Vikas,

Basically same comments as for the series for Exynos5260. Also see more 
comments inline.

On 27.03.2014 07:13, Vikas Sajjan wrote:
> Adds Suspend-to-RAM support for EXYNOS5420
>
> Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
> Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
> ---
>   arch/arm/mach-exynos/pm.c                    |  148 ++++++++++++++++++++++----
>   arch/arm/mach-exynos/regs-pmu.h              |    4 +-
>   arch/arm/plat-samsung/include/plat/map-s5p.h |    2 +
>   drivers/clk/samsung/clk-exynos5420.c         |   32 ++++++
>   4 files changed, 167 insertions(+), 19 deletions(-)
>
> diff --git a/arch/arm/mach-exynos/pm.c b/arch/arm/mach-exynos/pm.c
> index 15af0ce..aa3c2c8 100644
> --- a/arch/arm/mach-exynos/pm.c
> +++ b/arch/arm/mach-exynos/pm.c
> @@ -59,6 +59,16 @@ static struct sleep_save exynos_core_save[] = {
>   	SAVE_ITEM(S5P_SROM_BC3),
>   };
>
> +static struct sleep_save exynos5420_cpustate_save[] = {
> +	SAVE_ITEM(EXYNOS5420_VA_CPU_STATE),
> +};
> +
> +static struct sleep_save exynos5420_reg_save[] = {
> +	SAVE_ITEM(EXYNOS5_SYS_DISP1_BLK_CFG),
> +	SAVE_ITEM(S5P_PMU_SPARE3),
> +};
> +
> +
>   /*
>    * GIC wake-up support
>    */
> @@ -81,7 +91,7 @@ static int exynos_irq_set_wake(struct irq_data *data, unsigned int state)
>   {
>   	const struct exynos_wkup_irq *wkup_irq;
>
> -	if (soc_is_exynos5250())
> +	if (soc_is_exynos5250() || soc_is_exynos5420())
>   		wkup_irq = exynos5250_wkup_irq;
>   	else
>   		wkup_irq = exynos4_wkup_irq;
> @@ -109,7 +119,15 @@ static int exynos_cpu_suspend(unsigned long arg)
>   	outer_flush_all();
>   #endif
>
> -	if (soc_is_exynos5250())
> +	/*
> +	 * Clear IRAM register for cpu state so that primary CPU does
> +	 * not enter low power start in U-Boot.
> +	 * This is specific to exynos5420 SoC only.
> +	 */
> +	if (soc_is_exynos5420())
> +		__raw_writel(0x0, EXYNOS5420_VA_CPU_STATE);
> +
> +	if (soc_is_exynos5250() || soc_is_exynos5420())
>   		flush_cache_all();
>
>   	/* issue the standby signal into the pm unit. */
> @@ -135,6 +153,20 @@ static void exynos_pm_prepare(void)
>   		tmp = __raw_readl(EXYNOS5_JPEG_MEM_OPTION);
>   		tmp &= ~EXYNOS5_OPTION_USE_RETENTION;
>   		__raw_writel(tmp, EXYNOS5_JPEG_MEM_OPTION);
> +	} else if (soc_is_exynos5420()) {
> +

nit: Unnecessary blank line.

> +		s3c_pm_do_save(exynos5420_reg_save,
> +			ARRAY_SIZE(exynos5420_reg_save));
> +
> +		/*
> +		 * The cpu state needs to be saved and restored so that the
> +		 * secondary CPUs will enter low power start. Though the U-Boot
> +		 * is setting the cpu state with low power flag, the kernel
> +		 * needs to restore it back in case, the primary cpu fails to
> +		 * suspend for any reason
> +		 */
> +		s3c_pm_do_save(exynos5420_cpustate_save,
> +			ARRAY_SIZE(exynos5420_cpustate_save));
>   	}
>
>   	/* Set value of power down register for sleep mode */
> @@ -145,11 +177,34 @@ static void exynos_pm_prepare(void)
>   	/* ensure at least INFORM0 has the resume address */
>
>   	__raw_writel(virt_to_phys(exynos_cpu_resume), S5P_INFORM0);
> +
> +	if (soc_is_exynos5420()) {
> +
> +		tmp = __raw_readl(EXYNOS5_ARM_L2_OPTION);
> +		tmp &= ~EXYNOS5_USE_RETENTION;
> +		__raw_writel(tmp, EXYNOS5_ARM_L2_OPTION);
> +
> +		tmp = __raw_readl(EXYNOS5420_SFR_AXI_CGDIS1);
> +		tmp |= EXYNOS5420_UFS;
> +		__raw_writel(tmp, EXYNOS5420_SFR_AXI_CGDIS1);
> +
> +		tmp = __raw_readl(EXYNOS5420_ARM_COMMON_OPTION);
> +		tmp &= ~EXYNOS5420_L2RSTDISABLE_VALUE;
> +		__raw_writel(tmp, EXYNOS5420_ARM_COMMON_OPTION);
> +		tmp = __raw_readl(EXYNOS5420_FSYS2_OPTION);
> +		tmp |= EXYNOS5420_EMULATION;
> +		__raw_writel(tmp, EXYNOS5420_FSYS2_OPTION);
> +		tmp = __raw_readl(EXYNOS5420_PSGEN_OPTION);
> +		tmp |= EXYNOS5420_EMULATION;
> +		__raw_writel(tmp, EXYNOS5420_PSGEN_OPTION);
> +	}
> +
>   }
>
>   static int exynos_pm_suspend(void)
>   {
>   	unsigned long tmp;
> +	unsigned long cluster_id;
>
>   	/* Setting Central Sequence Register for power down mode */
>
> @@ -159,10 +214,20 @@ static int exynos_pm_suspend(void)
>
>   	/* Setting SEQ_OPTION register */
>
> -	tmp = (S5P_USE_STANDBY_WFI0 | S5P_USE_STANDBY_WFE0);
> -	__raw_writel(tmp, S5P_CENTRAL_SEQ_OPTION);
> +	if (soc_is_exynos5420()) {
> +		cluster_id = (read_cpuid(CPUID_MPIDR) >> 8) & 0xf;
> +		if (!cluster_id)
> +			__raw_writel(EXYNOS5420_ARM_USE_STANDBY_WFI0,
> +				     S5P_CENTRAL_SEQ_OPTION);
> +		else
> +			__raw_writel(EXYNOS5420_KFC_USE_STANDBY_WFI0,
> +				     S5P_CENTRAL_SEQ_OPTION);
> +	} else {
> +		tmp = (S5P_USE_STANDBY_WFI0 | S5P_USE_STANDBY_WFE0);
> +		__raw_writel(tmp, S5P_CENTRAL_SEQ_OPTION);
> +	}
>
> -	if (!soc_is_exynos5250()) {
> +	if (!(soc_is_exynos5250() || soc_is_exynos5420())) {
>   		/* Save Power control register */
>   		asm ("mrc p15, 0, %0, c15, c0, 0"
>   		     : "=r" (tmp) : : "cc");
> @@ -181,6 +246,15 @@ static void exynos_pm_resume(void)
>   {
>   	unsigned long tmp;
>
> +	if (soc_is_exynos5420()) {
> +		/* Restore the IRAM register cpu state */
> +		s3c_pm_do_restore(exynos5420_cpustate_save,
> +			ARRAY_SIZE(exynos5420_cpustate_save));
> +
> +		__raw_writel(EXYNOS5420_USE_STANDBY_WFI_ALL,
> +			S5P_CENTRAL_SEQ_OPTION);
> +	}
> +
>   	/*
>   	 * If PMU failed while entering sleep mode, WFI will be
>   	 * ignored by PMU and then exiting cpu_do_idle().
> @@ -196,7 +270,8 @@ static void exynos_pm_resume(void)
>   		/* No need to perform below restore code */
>   		goto early_wakeup;
>   	}
> -	if (!soc_is_exynos5250()) {
> +
> +	if (!(soc_is_exynos5250() || soc_is_exynos5420())) {
>   		/* Restore Power control register */
>   		tmp = save_arm_register[0];
>   		asm volatile ("mcr p15, 0, %0, c15, c0, 0"
> @@ -212,25 +287,57 @@ static void exynos_pm_resume(void)
>
>   	/* For release retention */
>
> -	__raw_writel((1 << 28), S5P_PAD_RET_MAUDIO_OPTION);
> -	__raw_writel((1 << 28), S5P_PAD_RET_GPIO_OPTION);
> -	__raw_writel((1 << 28), S5P_PAD_RET_UART_OPTION);
> -	__raw_writel((1 << 28), S5P_PAD_RET_MMCA_OPTION);
> -	__raw_writel((1 << 28), S5P_PAD_RET_MMCB_OPTION);
> -	__raw_writel((1 << 28), S5P_PAD_RET_EBIA_OPTION);
> -	__raw_writel((1 << 28), S5P_PAD_RET_EBIB_OPTION);
> +	if (soc_is_exynos5250()) {
> +		__raw_writel((1 << 28), S5P_PAD_RET_MAUDIO_OPTION);
> +		__raw_writel((1 << 28), S5P_PAD_RET_GPIO_OPTION);
> +		__raw_writel((1 << 28), S5P_PAD_RET_UART_OPTION);
> +		__raw_writel((1 << 28), S5P_PAD_RET_MMCA_OPTION);
> +		__raw_writel((1 << 28), S5P_PAD_RET_MMCB_OPTION);
> +		__raw_writel((1 << 28), S5P_PAD_RET_EBIA_OPTION);
> +		__raw_writel((1 << 28), S5P_PAD_RET_EBIB_OPTION);
> +	} else if (soc_is_exynos5420()) {
> +		__raw_writel(1 << 28, EXYNOS_PAD_RET_DRAM_OPTION);
> +		__raw_writel(1 << 28, EXYNOS_PAD_RET_MAUDIO_OPTION);
> +		__raw_writel(1 << 28, EXYNOS_PAD_RET_JTAG_OPTION);
> +		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_GPIO_OPTION);
> +		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_UART_OPTION);
> +		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCA_OPTION);
> +		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCB_OPTION);
> +		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCC_OPTION);
> +		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_HSI_OPTION);
> +		__raw_writel(1 << 28, EXYNOS_PAD_RET_EBIA_OPTION);
> +		__raw_writel(1 << 28, EXYNOS_PAD_RET_EBIB_OPTION);
> +		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_SPI_OPTION);
> +		__raw_writel(1 << 28, EXYNOS5420_PAD_RET_DRAM_COREBLK_OPTION);
> +	}
>
>   	if (soc_is_exynos5250())
>   		s3c_pm_do_restore(exynos5_sys_save,
>   			ARRAY_SIZE(exynos5_sys_save));
> +	else if (soc_is_exynos5420())
> +		s3c_pm_do_restore(exynos5420_reg_save,
> +			ARRAY_SIZE(exynos5420_reg_save));
>
>   	s3c_pm_do_restore_core(exynos_core_save, ARRAY_SIZE(exynos_core_save));
>
> -	if (IS_ENABLED(CONFIG_SMP) && !soc_is_exynos5250())
> +	if (IS_ENABLED(CONFIG_SMP) &&
> +			!soc_is_exynos5250() && !soc_is_exynos5420())
>   		scu_enable(S5P_VA_SCU);
>
>   early_wakeup:
>
> +	if (soc_is_exynos5420()) {
> +		tmp = __raw_readl(EXYNOS5420_SFR_AXI_CGDIS1);
> +		tmp &= ~EXYNOS5420_UFS;
> +		__raw_writel(tmp, EXYNOS5420_SFR_AXI_CGDIS1);
> +		tmp = __raw_readl(EXYNOS5420_FSYS2_OPTION);
> +		tmp &= ~EXYNOS5420_EMULATION;
> +		__raw_writel(tmp, EXYNOS5420_FSYS2_OPTION);
> +		tmp = __raw_readl(EXYNOS5420_PSGEN_OPTION);
> +		tmp &= ~EXYNOS5420_EMULATION;
> +		__raw_writel(tmp, EXYNOS5420_PSGEN_OPTION);
> +	}
> +
>   	/* Clear SLEEP mode set in INFORM1 */
>   	__raw_writel(0x0, S5P_INFORM1);
>
> @@ -312,10 +419,15 @@ void __init exynos_pm_init(void)
>   	gic_arch_extn.irq_set_wake = exynos_irq_set_wake;
>
>   	/* All wakeup disable */
> -	tmp = __raw_readl(S5P_WAKEUP_MASK);
> -	tmp |= ((0xFF << 8) | (0x1F << 1));
> -	__raw_writel(tmp, S5P_WAKEUP_MASK);
> -
> +	if (soc_is_exynos5420()) {
> +		tmp = __raw_readl(S5P_WAKEUP_MASK);
> +		tmp |= ((0x7F << 7) | (0x1F << 1));
> +		__raw_writel(tmp, S5P_WAKEUP_MASK);
> +	} else {
> +		tmp = __raw_readl(S5P_WAKEUP_MASK);
> +		tmp |= ((0xFF << 8) | (0x1F << 1));
> +		__raw_writel(tmp, S5P_WAKEUP_MASK);
> +	}
>   	register_syscore_ops(&exynos_pm_syscore_ops);
>   	suspend_set_ops(&exynos_suspend_ops);
>   }
> diff --git a/arch/arm/mach-exynos/regs-pmu.h b/arch/arm/mach-exynos/regs-pmu.h
> index cfbfc575..53eac2b 100644
> --- a/arch/arm/mach-exynos/regs-pmu.h
> +++ b/arch/arm/mach-exynos/regs-pmu.h
> @@ -39,6 +39,7 @@
>   #define S5P_INFORM6				S5P_PMUREG(0x0818)
>   #define S5P_INFORM7				S5P_PMUREG(0x081C)
>
> +#define S5P_PMU_SPARE3				S5P_PMUREG(0x090C)
>   #define EXYNOS_IROM_DATA2			S5P_PMUREG(0x0988)
>
>   #define S5P_ARM_CORE0_LOWPWR			S5P_PMUREG(0x1000)
> @@ -113,7 +114,6 @@
>   #define EXYNOS5_ARM_L2_OPTION			S5P_PMUREG(0x2608)
>   #define EXYNOS5_USE_RETENTION			BIT(4)
>
> -
>   #define S5P_PAD_RET_MAUDIO_OPTION		S5P_PMUREG(0x3028)
>   #define S5P_PAD_RET_GPIO_OPTION			S5P_PMUREG(0x3108)
>   #define S5P_PAD_RET_UART_OPTION			S5P_PMUREG(0x3128)
> @@ -216,6 +216,8 @@
>
>   /* For EXYNOS5 */
>
> +#define EXYNOS5_SYS_DISP1_BLK_CFG			S5P_SYSREG(0x0214)
> +
>   #define EXYNOS5_SYS_I2C_CFG					S5P_SYSREG(0x0234)
>
>   #define EXYNOS5_AUTO_WDTRESET_DISABLE				S5P_PMUREG(0x0408)
> diff --git a/arch/arm/plat-samsung/include/plat/map-s5p.h b/arch/arm/plat-samsung/include/plat/map-s5p.h
> index c186786..cfb150e 100644
> --- a/arch/arm/plat-samsung/include/plat/map-s5p.h
> +++ b/arch/arm/plat-samsung/include/plat/map-s5p.h
> @@ -57,6 +57,8 @@
>   #define S3C_UART_OFFSET		(0x400)
>   #endif
>
> +#define EXYNOS5420_VA_CPU_STATE	(S5P_VA_SYSRAM + 0x28)
> +

Please, no more static addresses. This should be defined as an offset to 
SYSRAM in the source file that actually uses it.

>   #include <plat/map-s3c.h>
>
>   #endif /* __ASM_PLAT_MAP_S5P_H */
> diff --git a/drivers/clk/samsung/clk-exynos5420.c b/drivers/clk/samsung/clk-exynos5420.c
> index 60b2681..965ea69 100644
> --- a/drivers/clk/samsung/clk-exynos5420.c
> +++ b/drivers/clk/samsung/clk-exynos5420.c
> @@ -101,6 +101,19 @@
>   #define KPLL_CON0		0x28100
>   #define SRC_KFC			0x28200
>   #define DIV_KFC0		0x28500
> +#define SRC_MASK_CPERI		0x04300

Please keep the registers sorted...

> +#define SRC_MASK_TOP0		0x10300
> +#define SRC_MASK_TOP1		0x10304
> +#define SRC_MASK_TOP2		0x10308
> +#define SRC_MASK_TOP7		0x1031C
> +#define SRC_MASK_MAU		0x10334
> +#define SRC_MASK_FSYS		0x10340
> +#define SRC_MASK_PERIC0		0x10350
> +#define SRC_MASK_PERIC1		0x10354
> +#define SRC_MASK_ISP		0x10370
> +#define GATE_BUS_DISP1		0x10728
> +#define GATE_IP_PERIC		0x10950
> +
>
>   /* list of PLLs */
>   enum exynos5420_plls {
> @@ -180,11 +193,30 @@ static unsigned long exynos5420_clk_regs[] __initdata = {
>   	DIV_KFC0,
>   };
>
> +static const struct samsung_clk_reg_dump exynos5420_set_clksrc[] = {
> +	{ .offset = SRC_MASK_CPERI,		.value = 0xffffffff, },
> +	{ .offset = SRC_MASK_TOP0,		.value = 0x11111111, },
> +	{ .offset = SRC_MASK_TOP1,		.value = 0x11101111, },
> +	{ .offset = SRC_MASK_TOP2,		.value = 0x11111110, },
> +	{ .offset = SRC_MASK_TOP7,		.value = 0x00111100, },
> +	{ .offset = SRC_MASK_DISP10,		.value = 0x11111110, },
> +	{ .offset = SRC_MASK_MAU,		.value = 0x10000000, },
> +	{ .offset = SRC_MASK_FSYS,		.value = 0x11111110, },
> +	{ .offset = SRC_MASK_PERIC0,		.value = 0x11111110, },
> +	{ .offset = SRC_MASK_PERIC1,		.value = 0x11111100, },
> +	{ .offset = SRC_MASK_ISP,		.value = 0x11111000, },
> +	{ .offset = GATE_BUS_DISP1,		.value = 0xffffffff, },
> +	{ .offset = GATE_IP_PERIC,		.value = 0xffffffff, },
> +};
> +
> +
>   static int exynos5420_clk_suspend(void)
>   {
>   	samsung_clk_save(reg_base, exynos5420_save,
>   				ARRAY_SIZE(exynos5420_clk_regs));
>
> +	samsung_clk_restore(reg_base, exynos5420_set_clksrc,
> +					ARRAY_SIZE(exynos5420_set_clksrc));

nit: You could align ARRAY_SIZE to the same indentation level as the 
call to samsung_clk_save() above.

Best regards,
Tomasz

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

* [PATCH V2 2/3] ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420
  2014-04-15 19:03   ` Tomasz Figa
@ 2014-04-15 19:04     ` Tomasz Figa
  2014-04-28 12:39     ` Vikas Sajjan
  1 sibling, 0 replies; 12+ messages in thread
From: Tomasz Figa @ 2014-04-15 19:04 UTC (permalink / raw)
  To: linux-arm-kernel

One more thing. The change to the clock driver should be separated from 
arch changes.

Best regards,
Tomasz

On 15.04.2014 21:03, Tomasz Figa wrote:
> Hi Vikas,
>
> Basically same comments as for the series for Exynos5260. Also see more
> comments inline.
>
> On 27.03.2014 07:13, Vikas Sajjan wrote:
>> Adds Suspend-to-RAM support for EXYNOS5420
>>
>> Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
>> Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
>> ---
>>   arch/arm/mach-exynos/pm.c                    |  148
>> ++++++++++++++++++++++----
>>   arch/arm/mach-exynos/regs-pmu.h              |    4 +-
>>   arch/arm/plat-samsung/include/plat/map-s5p.h |    2 +
>>   drivers/clk/samsung/clk-exynos5420.c         |   32 ++++++
>>   4 files changed, 167 insertions(+), 19 deletions(-)
>>
>> diff --git a/arch/arm/mach-exynos/pm.c b/arch/arm/mach-exynos/pm.c
>> index 15af0ce..aa3c2c8 100644
>> --- a/arch/arm/mach-exynos/pm.c
>> +++ b/arch/arm/mach-exynos/pm.c
>> @@ -59,6 +59,16 @@ static struct sleep_save exynos_core_save[] = {
>>       SAVE_ITEM(S5P_SROM_BC3),
>>   };
>>
>> +static struct sleep_save exynos5420_cpustate_save[] = {
>> +    SAVE_ITEM(EXYNOS5420_VA_CPU_STATE),
>> +};
>> +
>> +static struct sleep_save exynos5420_reg_save[] = {
>> +    SAVE_ITEM(EXYNOS5_SYS_DISP1_BLK_CFG),
>> +    SAVE_ITEM(S5P_PMU_SPARE3),
>> +};
>> +
>> +
>>   /*
>>    * GIC wake-up support
>>    */
>> @@ -81,7 +91,7 @@ static int exynos_irq_set_wake(struct irq_data
>> *data, unsigned int state)
>>   {
>>       const struct exynos_wkup_irq *wkup_irq;
>>
>> -    if (soc_is_exynos5250())
>> +    if (soc_is_exynos5250() || soc_is_exynos5420())
>>           wkup_irq = exynos5250_wkup_irq;
>>       else
>>           wkup_irq = exynos4_wkup_irq;
>> @@ -109,7 +119,15 @@ static int exynos_cpu_suspend(unsigned long arg)
>>       outer_flush_all();
>>   #endif
>>
>> -    if (soc_is_exynos5250())
>> +    /*
>> +     * Clear IRAM register for cpu state so that primary CPU does
>> +     * not enter low power start in U-Boot.
>> +     * This is specific to exynos5420 SoC only.
>> +     */
>> +    if (soc_is_exynos5420())
>> +        __raw_writel(0x0, EXYNOS5420_VA_CPU_STATE);
>> +
>> +    if (soc_is_exynos5250() || soc_is_exynos5420())
>>           flush_cache_all();
>>
>>       /* issue the standby signal into the pm unit. */
>> @@ -135,6 +153,20 @@ static void exynos_pm_prepare(void)
>>           tmp = __raw_readl(EXYNOS5_JPEG_MEM_OPTION);
>>           tmp &= ~EXYNOS5_OPTION_USE_RETENTION;
>>           __raw_writel(tmp, EXYNOS5_JPEG_MEM_OPTION);
>> +    } else if (soc_is_exynos5420()) {
>> +
>
> nit: Unnecessary blank line.
>
>> +        s3c_pm_do_save(exynos5420_reg_save,
>> +            ARRAY_SIZE(exynos5420_reg_save));
>> +
>> +        /*
>> +         * The cpu state needs to be saved and restored so that the
>> +         * secondary CPUs will enter low power start. Though the U-Boot
>> +         * is setting the cpu state with low power flag, the kernel
>> +         * needs to restore it back in case, the primary cpu fails to
>> +         * suspend for any reason
>> +         */
>> +        s3c_pm_do_save(exynos5420_cpustate_save,
>> +            ARRAY_SIZE(exynos5420_cpustate_save));
>>       }
>>
>>       /* Set value of power down register for sleep mode */
>> @@ -145,11 +177,34 @@ static void exynos_pm_prepare(void)
>>       /* ensure at least INFORM0 has the resume address */
>>
>>       __raw_writel(virt_to_phys(exynos_cpu_resume), S5P_INFORM0);
>> +
>> +    if (soc_is_exynos5420()) {
>> +
>> +        tmp = __raw_readl(EXYNOS5_ARM_L2_OPTION);
>> +        tmp &= ~EXYNOS5_USE_RETENTION;
>> +        __raw_writel(tmp, EXYNOS5_ARM_L2_OPTION);
>> +
>> +        tmp = __raw_readl(EXYNOS5420_SFR_AXI_CGDIS1);
>> +        tmp |= EXYNOS5420_UFS;
>> +        __raw_writel(tmp, EXYNOS5420_SFR_AXI_CGDIS1);
>> +
>> +        tmp = __raw_readl(EXYNOS5420_ARM_COMMON_OPTION);
>> +        tmp &= ~EXYNOS5420_L2RSTDISABLE_VALUE;
>> +        __raw_writel(tmp, EXYNOS5420_ARM_COMMON_OPTION);
>> +        tmp = __raw_readl(EXYNOS5420_FSYS2_OPTION);
>> +        tmp |= EXYNOS5420_EMULATION;
>> +        __raw_writel(tmp, EXYNOS5420_FSYS2_OPTION);
>> +        tmp = __raw_readl(EXYNOS5420_PSGEN_OPTION);
>> +        tmp |= EXYNOS5420_EMULATION;
>> +        __raw_writel(tmp, EXYNOS5420_PSGEN_OPTION);
>> +    }
>> +
>>   }
>>
>>   static int exynos_pm_suspend(void)
>>   {
>>       unsigned long tmp;
>> +    unsigned long cluster_id;
>>
>>       /* Setting Central Sequence Register for power down mode */
>>
>> @@ -159,10 +214,20 @@ static int exynos_pm_suspend(void)
>>
>>       /* Setting SEQ_OPTION register */
>>
>> -    tmp = (S5P_USE_STANDBY_WFI0 | S5P_USE_STANDBY_WFE0);
>> -    __raw_writel(tmp, S5P_CENTRAL_SEQ_OPTION);
>> +    if (soc_is_exynos5420()) {
>> +        cluster_id = (read_cpuid(CPUID_MPIDR) >> 8) & 0xf;
>> +        if (!cluster_id)
>> +            __raw_writel(EXYNOS5420_ARM_USE_STANDBY_WFI0,
>> +                     S5P_CENTRAL_SEQ_OPTION);
>> +        else
>> +            __raw_writel(EXYNOS5420_KFC_USE_STANDBY_WFI0,
>> +                     S5P_CENTRAL_SEQ_OPTION);
>> +    } else {
>> +        tmp = (S5P_USE_STANDBY_WFI0 | S5P_USE_STANDBY_WFE0);
>> +        __raw_writel(tmp, S5P_CENTRAL_SEQ_OPTION);
>> +    }
>>
>> -    if (!soc_is_exynos5250()) {
>> +    if (!(soc_is_exynos5250() || soc_is_exynos5420())) {
>>           /* Save Power control register */
>>           asm ("mrc p15, 0, %0, c15, c0, 0"
>>                : "=r" (tmp) : : "cc");
>> @@ -181,6 +246,15 @@ static void exynos_pm_resume(void)
>>   {
>>       unsigned long tmp;
>>
>> +    if (soc_is_exynos5420()) {
>> +        /* Restore the IRAM register cpu state */
>> +        s3c_pm_do_restore(exynos5420_cpustate_save,
>> +            ARRAY_SIZE(exynos5420_cpustate_save));
>> +
>> +        __raw_writel(EXYNOS5420_USE_STANDBY_WFI_ALL,
>> +            S5P_CENTRAL_SEQ_OPTION);
>> +    }
>> +
>>       /*
>>        * If PMU failed while entering sleep mode, WFI will be
>>        * ignored by PMU and then exiting cpu_do_idle().
>> @@ -196,7 +270,8 @@ static void exynos_pm_resume(void)
>>           /* No need to perform below restore code */
>>           goto early_wakeup;
>>       }
>> -    if (!soc_is_exynos5250()) {
>> +
>> +    if (!(soc_is_exynos5250() || soc_is_exynos5420())) {
>>           /* Restore Power control register */
>>           tmp = save_arm_register[0];
>>           asm volatile ("mcr p15, 0, %0, c15, c0, 0"
>> @@ -212,25 +287,57 @@ static void exynos_pm_resume(void)
>>
>>       /* For release retention */
>>
>> -    __raw_writel((1 << 28), S5P_PAD_RET_MAUDIO_OPTION);
>> -    __raw_writel((1 << 28), S5P_PAD_RET_GPIO_OPTION);
>> -    __raw_writel((1 << 28), S5P_PAD_RET_UART_OPTION);
>> -    __raw_writel((1 << 28), S5P_PAD_RET_MMCA_OPTION);
>> -    __raw_writel((1 << 28), S5P_PAD_RET_MMCB_OPTION);
>> -    __raw_writel((1 << 28), S5P_PAD_RET_EBIA_OPTION);
>> -    __raw_writel((1 << 28), S5P_PAD_RET_EBIB_OPTION);
>> +    if (soc_is_exynos5250()) {
>> +        __raw_writel((1 << 28), S5P_PAD_RET_MAUDIO_OPTION);
>> +        __raw_writel((1 << 28), S5P_PAD_RET_GPIO_OPTION);
>> +        __raw_writel((1 << 28), S5P_PAD_RET_UART_OPTION);
>> +        __raw_writel((1 << 28), S5P_PAD_RET_MMCA_OPTION);
>> +        __raw_writel((1 << 28), S5P_PAD_RET_MMCB_OPTION);
>> +        __raw_writel((1 << 28), S5P_PAD_RET_EBIA_OPTION);
>> +        __raw_writel((1 << 28), S5P_PAD_RET_EBIB_OPTION);
>> +    } else if (soc_is_exynos5420()) {
>> +        __raw_writel(1 << 28, EXYNOS_PAD_RET_DRAM_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS_PAD_RET_MAUDIO_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS_PAD_RET_JTAG_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS5420_PAD_RET_GPIO_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS5420_PAD_RET_UART_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCA_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCB_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCC_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS5420_PAD_RET_HSI_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS_PAD_RET_EBIA_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS_PAD_RET_EBIB_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS5420_PAD_RET_SPI_OPTION);
>> +        __raw_writel(1 << 28, EXYNOS5420_PAD_RET_DRAM_COREBLK_OPTION);
>> +    }
>>
>>       if (soc_is_exynos5250())
>>           s3c_pm_do_restore(exynos5_sys_save,
>>               ARRAY_SIZE(exynos5_sys_save));
>> +    else if (soc_is_exynos5420())
>> +        s3c_pm_do_restore(exynos5420_reg_save,
>> +            ARRAY_SIZE(exynos5420_reg_save));
>>
>>       s3c_pm_do_restore_core(exynos_core_save,
>> ARRAY_SIZE(exynos_core_save));
>>
>> -    if (IS_ENABLED(CONFIG_SMP) && !soc_is_exynos5250())
>> +    if (IS_ENABLED(CONFIG_SMP) &&
>> +            !soc_is_exynos5250() && !soc_is_exynos5420())
>>           scu_enable(S5P_VA_SCU);
>>
>>   early_wakeup:
>>
>> +    if (soc_is_exynos5420()) {
>> +        tmp = __raw_readl(EXYNOS5420_SFR_AXI_CGDIS1);
>> +        tmp &= ~EXYNOS5420_UFS;
>> +        __raw_writel(tmp, EXYNOS5420_SFR_AXI_CGDIS1);
>> +        tmp = __raw_readl(EXYNOS5420_FSYS2_OPTION);
>> +        tmp &= ~EXYNOS5420_EMULATION;
>> +        __raw_writel(tmp, EXYNOS5420_FSYS2_OPTION);
>> +        tmp = __raw_readl(EXYNOS5420_PSGEN_OPTION);
>> +        tmp &= ~EXYNOS5420_EMULATION;
>> +        __raw_writel(tmp, EXYNOS5420_PSGEN_OPTION);
>> +    }
>> +
>>       /* Clear SLEEP mode set in INFORM1 */
>>       __raw_writel(0x0, S5P_INFORM1);
>>
>> @@ -312,10 +419,15 @@ void __init exynos_pm_init(void)
>>       gic_arch_extn.irq_set_wake = exynos_irq_set_wake;
>>
>>       /* All wakeup disable */
>> -    tmp = __raw_readl(S5P_WAKEUP_MASK);
>> -    tmp |= ((0xFF << 8) | (0x1F << 1));
>> -    __raw_writel(tmp, S5P_WAKEUP_MASK);
>> -
>> +    if (soc_is_exynos5420()) {
>> +        tmp = __raw_readl(S5P_WAKEUP_MASK);
>> +        tmp |= ((0x7F << 7) | (0x1F << 1));
>> +        __raw_writel(tmp, S5P_WAKEUP_MASK);
>> +    } else {
>> +        tmp = __raw_readl(S5P_WAKEUP_MASK);
>> +        tmp |= ((0xFF << 8) | (0x1F << 1));
>> +        __raw_writel(tmp, S5P_WAKEUP_MASK);
>> +    }
>>       register_syscore_ops(&exynos_pm_syscore_ops);
>>       suspend_set_ops(&exynos_suspend_ops);
>>   }
>> diff --git a/arch/arm/mach-exynos/regs-pmu.h
>> b/arch/arm/mach-exynos/regs-pmu.h
>> index cfbfc575..53eac2b 100644
>> --- a/arch/arm/mach-exynos/regs-pmu.h
>> +++ b/arch/arm/mach-exynos/regs-pmu.h
>> @@ -39,6 +39,7 @@
>>   #define S5P_INFORM6                S5P_PMUREG(0x0818)
>>   #define S5P_INFORM7                S5P_PMUREG(0x081C)
>>
>> +#define S5P_PMU_SPARE3                S5P_PMUREG(0x090C)
>>   #define EXYNOS_IROM_DATA2            S5P_PMUREG(0x0988)
>>
>>   #define S5P_ARM_CORE0_LOWPWR            S5P_PMUREG(0x1000)
>> @@ -113,7 +114,6 @@
>>   #define EXYNOS5_ARM_L2_OPTION            S5P_PMUREG(0x2608)
>>   #define EXYNOS5_USE_RETENTION            BIT(4)
>>
>> -
>>   #define S5P_PAD_RET_MAUDIO_OPTION        S5P_PMUREG(0x3028)
>>   #define S5P_PAD_RET_GPIO_OPTION            S5P_PMUREG(0x3108)
>>   #define S5P_PAD_RET_UART_OPTION            S5P_PMUREG(0x3128)
>> @@ -216,6 +216,8 @@
>>
>>   /* For EXYNOS5 */
>>
>> +#define EXYNOS5_SYS_DISP1_BLK_CFG            S5P_SYSREG(0x0214)
>> +
>>   #define EXYNOS5_SYS_I2C_CFG                    S5P_SYSREG(0x0234)
>>
>>   #define EXYNOS5_AUTO_WDTRESET_DISABLE                S5P_PMUREG(0x0408)
>> diff --git a/arch/arm/plat-samsung/include/plat/map-s5p.h
>> b/arch/arm/plat-samsung/include/plat/map-s5p.h
>> index c186786..cfb150e 100644
>> --- a/arch/arm/plat-samsung/include/plat/map-s5p.h
>> +++ b/arch/arm/plat-samsung/include/plat/map-s5p.h
>> @@ -57,6 +57,8 @@
>>   #define S3C_UART_OFFSET        (0x400)
>>   #endif
>>
>> +#define EXYNOS5420_VA_CPU_STATE    (S5P_VA_SYSRAM + 0x28)
>> +
>
> Please, no more static addresses. This should be defined as an offset to
> SYSRAM in the source file that actually uses it.
>
>>   #include <plat/map-s3c.h>
>>
>>   #endif /* __ASM_PLAT_MAP_S5P_H */
>> diff --git a/drivers/clk/samsung/clk-exynos5420.c
>> b/drivers/clk/samsung/clk-exynos5420.c
>> index 60b2681..965ea69 100644
>> --- a/drivers/clk/samsung/clk-exynos5420.c
>> +++ b/drivers/clk/samsung/clk-exynos5420.c
>> @@ -101,6 +101,19 @@
>>   #define KPLL_CON0        0x28100
>>   #define SRC_KFC            0x28200
>>   #define DIV_KFC0        0x28500
>> +#define SRC_MASK_CPERI        0x04300
>
> Please keep the registers sorted...
>
>> +#define SRC_MASK_TOP0        0x10300
>> +#define SRC_MASK_TOP1        0x10304
>> +#define SRC_MASK_TOP2        0x10308
>> +#define SRC_MASK_TOP7        0x1031C
>> +#define SRC_MASK_MAU        0x10334
>> +#define SRC_MASK_FSYS        0x10340
>> +#define SRC_MASK_PERIC0        0x10350
>> +#define SRC_MASK_PERIC1        0x10354
>> +#define SRC_MASK_ISP        0x10370
>> +#define GATE_BUS_DISP1        0x10728
>> +#define GATE_IP_PERIC        0x10950
>> +
>>
>>   /* list of PLLs */
>>   enum exynos5420_plls {
>> @@ -180,11 +193,30 @@ static unsigned long exynos5420_clk_regs[]
>> __initdata = {
>>       DIV_KFC0,
>>   };
>>
>> +static const struct samsung_clk_reg_dump exynos5420_set_clksrc[] = {
>> +    { .offset = SRC_MASK_CPERI,        .value = 0xffffffff, },
>> +    { .offset = SRC_MASK_TOP0,        .value = 0x11111111, },
>> +    { .offset = SRC_MASK_TOP1,        .value = 0x11101111, },
>> +    { .offset = SRC_MASK_TOP2,        .value = 0x11111110, },
>> +    { .offset = SRC_MASK_TOP7,        .value = 0x00111100, },
>> +    { .offset = SRC_MASK_DISP10,        .value = 0x11111110, },
>> +    { .offset = SRC_MASK_MAU,        .value = 0x10000000, },
>> +    { .offset = SRC_MASK_FSYS,        .value = 0x11111110, },
>> +    { .offset = SRC_MASK_PERIC0,        .value = 0x11111110, },
>> +    { .offset = SRC_MASK_PERIC1,        .value = 0x11111100, },
>> +    { .offset = SRC_MASK_ISP,        .value = 0x11111000, },
>> +    { .offset = GATE_BUS_DISP1,        .value = 0xffffffff, },
>> +    { .offset = GATE_IP_PERIC,        .value = 0xffffffff, },
>> +};
>> +
>> +
>>   static int exynos5420_clk_suspend(void)
>>   {
>>       samsung_clk_save(reg_base, exynos5420_save,
>>                   ARRAY_SIZE(exynos5420_clk_regs));
>>
>> +    samsung_clk_restore(reg_base, exynos5420_set_clksrc,
>> +                    ARRAY_SIZE(exynos5420_set_clksrc));
>
> nit: You could align ARRAY_SIZE to the same indentation level as the
> call to samsung_clk_save() above.
>
> Best regards,
> Tomasz

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

* [PATCH V2 3/3] ARM: dts: Add node for GPIO keys on SMDK5420
  2014-03-27  6:13 ` [PATCH V2 3/3] ARM: dts: Add node for GPIO keys on SMDK5420 Vikas Sajjan
@ 2014-04-15 19:06   ` Tomasz Figa
  0 siblings, 0 replies; 12+ messages in thread
From: Tomasz Figa @ 2014-04-15 19:06 UTC (permalink / raw)
  To: linux-arm-kernel

Hi,

On 27.03.2014 07:13, Vikas Sajjan wrote:
> From: Abhilash Kesavan <a.kesavan@samsung.com>
>
> Add support for the suspend/resume button (SW7) on SMDK5420 using
> gpio-keys driver.
>
> Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
> Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
> ---
>   arch/arm/boot/dts/exynos5420-smdk5420.dts |   11 +++++++++++
>   1 file changed, 11 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5420-smdk5420.dts b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> index 6910485..5ae0dd4 100644
> --- a/arch/arm/boot/dts/exynos5420-smdk5420.dts
> +++ b/arch/arm/boot/dts/exynos5420-smdk5420.dts
> @@ -158,6 +158,17 @@
>   		};
>   	};
>
> +	gpio-keys {
> +		compatible = "gpio-keys";
> +
> +		wakeup {

Is wakeup the label of this key on the PCB? The "label" property below 
would suggest otherwise. This should probably be called key-sw7 or 
something like this.

Moreover, is this the only GPIO key on the board? If there are also 
SW0-SW6, you should add them too.

> +			label = "SW7";
> +			gpios = <&gpx0 0 1>;
> +			linux,code = <143>;	/* KEY_WAKEUP */

Why not use KEY_WAKEUP defined in include/dt-bindings/input/input.h 
directly?

Best regards,
Tomasz

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

* [PATCH V2 1/3] ARM: EXYNOS5: Add PMU settings for exynos5420
  2014-04-15 18:57   ` Tomasz Figa
@ 2014-04-28 12:36     ` Vikas Sajjan
  0 siblings, 0 replies; 12+ messages in thread
From: Vikas Sajjan @ 2014-04-28 12:36 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Tomasz,

On Wed, Apr 16, 2014 at 12:27 AM, Tomasz Figa <tomasz.figa@gmail.com> wrote:
> Hi Vikas,
>
> Basically the same comments apply as for the series:
>
> [PATCH v2 0/3] Add initial support of PMU for exynos5260
> (https://www.mail-archive.com/linux-samsung-soc at vger.kernel.org/msg27339.html)
>
> In addition to above, please see some comments inline.
>
>
> On 27.03.2014 07:13, Vikas Sajjan wrote:
>>
>> Add intial PMU settings for exynos5420. This is required for
>> future S2R and Switching support.
>>
>> Signed-off-by: Thomas Abraham <thomas.ab@samsung.com>
>> Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
>> Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
>> ---
>>   arch/arm/mach-exynos/common.h   |   10 ++
>>   arch/arm/mach-exynos/pmu.c      |  307
>> +++++++++++++++++++++++++++++++++++++++
>>   arch/arm/mach-exynos/regs-pmu.h |  228 +++++++++++++++++++++++++++++
>>   3 files changed, 545 insertions(+)
>>
>> diff --git a/arch/arm/mach-exynos/common.h b/arch/arm/mach-exynos/common.h
>> index 9ef3f83..347afc2 100644
>> --- a/arch/arm/mach-exynos/common.h
>> +++ b/arch/arm/mach-exynos/common.h
>> @@ -15,6 +15,16 @@
>>   #include <linux/reboot.h>
>>   #include <linux/of.h>
>>
>> +#define EXYNOS5420_USE_STANDBY_WFI_ALL (EXYNOS5420_ARM_USE_STANDBY_WFI0
>> \
>> +                                        | EXYNOS5420_ARM_USE_STANDBY_WFI1
>> \
>> +                                        | EXYNOS5420_ARM_USE_STANDBY_WFI2
>> \
>> +                                        | EXYNOS5420_ARM_USE_STANDBY_WFI3
>> \
>> +                                        | EXYNOS5420_KFC_USE_STANDBY_WFI0
>> \
>> +                                        | EXYNOS5420_KFC_USE_STANDBY_WFI1
>> \
>> +                                        | EXYNOS5420_KFC_USE_STANDBY_WFI2
>> \
>> +                                        |
>> EXYNOS5420_KFC_USE_STANDBY_WFI3)
>> +
>> +
>>   void mct_init(void __iomem *base, int irq_g0, int irq_l0, int irq_l1);
>>
>>   struct map_desc;
>> diff --git a/arch/arm/mach-exynos/pmu.c b/arch/arm/mach-exynos/pmu.c
>> index 05c7ce1..f69a6ed 100644
>> --- a/arch/arm/mach-exynos/pmu.c
>> +++ b/arch/arm/mach-exynos/pmu.c
>> @@ -12,9 +12,14 @@
>>   #include <linux/io.h>
>>   #include <linux/kernel.h>
>>   #include <linux/bug.h>
>> +#include <linux/cpumask.h>
>> +#include <linux/delay.h>
>> +#include <linux/pm.h>
>>
>>   #include <plat/cpu.h>
>>
>> +#include <asm/cputype.h>
>> +
>>   #include "common.h"
>>   #include "regs-pmu.h"
>>
>> @@ -318,6 +323,212 @@ static const struct exynos_pmu_conf
>> exynos5250_pmu_config[] = {
>>         { PMU_TABLE_END,},
>>   };
>>
>> +static const struct exynos_pmu_conf exynos5420_pmu_config[] = {
>> +       /* { .reg = address, .val = { AFTR, LPA, SLEEP } */
>> +       { EXYNOS5_ARM_CORE0_SYS_PWR_REG,                        { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_DIS_IRQ_ARM_CORE0_LOCAL_SYS_PWR_REG,          { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_DIS_IRQ_ARM_CORE0_CENTRAL_SYS_PWR_REG,        { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_ARM_CORE1_SYS_PWR_REG,                        { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_DIS_IRQ_ARM_CORE1_LOCAL_SYS_PWR_REG,          { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_DIS_IRQ_ARM_CORE1_CENTRAL_SYS_PWR_REG,        { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_ARM_CORE2_SYS_PWR_REG,                     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_ARM_CORE2_LOCAL_SYS_PWR_REG,       { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_ARM_CORE2_CENTRAL_SYS_PWR_REG,     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_ARM_CORE3_SYS_PWR_REG,                     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_ARM_CORE3_LOCAL_SYS_PWR_REG,       { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_ARM_CORE3_CENTRAL_SYS_PWR_REG,     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_KFC_CORE0_SYS_PWR_REG,                     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_KFC_CORE0_LOCAL_SYS_PWR_REG,       { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_KFC_CORE0_CENTRAL_SYS_PWR_REG,     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_KFC_CORE1_SYS_PWR_REG,                     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_KFC_CORE1_LOCAL_SYS_PWR_REG,       { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_KFC_CORE1_CENTRAL_SYS_PWR_REG,     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_KFC_CORE2_SYS_PWR_REG,                     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_KFC_CORE2_LOCAL_SYS_PWR_REG,       { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_KFC_CORE2_CENTRAL_SYS_PWR_REG,     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_KFC_CORE3_SYS_PWR_REG,                     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_KFC_CORE3_LOCAL_SYS_PWR_REG,       { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DIS_IRQ_KFC_CORE3_CENTRAL_SYS_PWR_REG,     { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_ISP_ARM_SYS_PWR_REG,                          { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_DIS_IRQ_ISP_ARM_LOCAL_SYS_PWR_REG,            { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_DIS_IRQ_ISP_ARM_CENTRAL_SYS_PWR_REG,          { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_ARM_COMMON_SYS_PWR_REG,                    { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_KFC_COMMON_SYS_PWR_REG,                    { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_ARM_L2_SYS_PWR_REG,                           { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_KFC_L2_SYS_PWR_REG,                        { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_ACLKSTOP_SYS_PWR_REG,                     { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_SCLKSTOP_SYS_PWR_REG,                     { 0x1,
>> 0x0, 0x1} },
>> +       { EXYNOS5_CMU_RESET_SYS_PWR_REG,                        { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_CMU_ACLKSTOP_SYSMEM_SYS_PWR_REG,              { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_SCLKSTOP_SYSMEM_SYS_PWR_REG,              { 0x1,
>> 0x0, 0x1} },
>> +       { EXYNOS5_CMU_RESET_SYSMEM_SYS_PWR_REG,                 { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_DRAM_FREQ_DOWN_SYS_PWR_REG,                   { 0x1,
>> 0x0, 0x1} },
>> +       { EXYNOS5_DDRPHY_DLLOFF_SYS_PWR_REG,                    { 0x1,
>> 0x1, 0x1} },
>> +       { EXYNOS5_DDRPHY_DLLLOCK_SYS_PWR_REG,                   { 0x1,
>> 0x0, 0x1} },
>> +       { EXYNOS5_APLL_SYSCLK_SYS_PWR_REG,                      { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_MPLL_SYSCLK_SYS_PWR_REG,                      { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_VPLL_SYSCLK_SYS_PWR_REG,                      { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_EPLL_SYSCLK_SYS_PWR_REG,                      { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_BPLL_SYSCLK_SYS_PWR_REG,                      { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CPLL_SYSCLK_SYS_PWR_REG,                      { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DPLL_SYSCLK_SYS_PWR_REG,                   { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_IPLL_SYSCLK_SYS_PWR_REG,                   { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_KPLL_SYSCLK_SYS_PWR_REG,                   { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_MPLLUSER_SYSCLK_SYS_PWR_REG,                  { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_BPLLUSER_SYSCLK_SYS_PWR_REG,                  { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_RPLL_SYSCLK_SYS_PWR_REG,                   { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_SPLL_SYSCLK_SYS_PWR_REG,                   { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_TOP_BUS_SYS_PWR_REG,                          { 0x3,
>> 0x0, 0x0} },
>> +       { EXYNOS5_TOP_RETENTION_SYS_PWR_REG,                    { 0x1,
>> 0x1, 0x1} },
>> +       { EXYNOS5_TOP_PWR_SYS_PWR_REG,                          { 0x3,
>> 0x3, 0x0} },
>> +       { EXYNOS5_TOP_BUS_SYSMEM_SYS_PWR_REG,                   { 0x3,
>> 0x0, 0x0} },
>> +       { EXYNOS5_TOP_RETENTION_SYSMEM_SYS_PWR_REG,             { 0x1,
>> 0x0, 0x1} },
>> +       { EXYNOS5_TOP_PWR_SYSMEM_SYS_PWR_REG,                   { 0x3,
>> 0x0, 0x0} },
>> +       { EXYNOS5_LOGIC_RESET_SYS_PWR_REG,                      { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_OSCCLK_GATE_SYS_PWR_REG,                      { 0x1,
>> 0x0, 0x1} },
>> +       { EXYNOS5_LOGIC_RESET_SYSMEM_SYS_PWR_REG,               { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_OSCCLK_GATE_SYSMEM_SYS_PWR_REG,               { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_INTRAM_MEM_SYS_PWR_REG,                    { 0x3,
>> 0x0, 0x3} },
>> +       { EXYNOS5420_INTROM_MEM_SYS_PWR_REG,                    { 0x3,
>> 0x0, 0x3} },
>> +       { EXYNOS5_PAD_RETENTION_DRAM_SYS_PWR_REG,               { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_PAD_RETENTION_MAU_SYS_PWR_REG,                { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_JTAG_SYS_PWR_REG,            { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_DRAM_SYS_PWR_REG,            { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_UART_SYS_PWR_REG,            { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_MMC0_SYS_PWR_REG,            { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_MMC1_SYS_PWR_REG,            { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_MMC2_SYS_PWR_REG,            { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_HSI_SYS_PWR_REG,             { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_EBIA_SYS_PWR_REG,            { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_EBIB_SYS_PWR_REG,            { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_SPI_SYS_PWR_REG,             { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PAD_RETENTION_DRAM_COREBLK_SYS_PWR_REG,    { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_PAD_ISOLATION_SYS_PWR_REG,                    { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_PAD_ISOLATION_SYSMEM_SYS_PWR_REG,             { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_PAD_ALV_SEL_SYS_PWR_REG,                      { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_XUSBXTI_SYS_PWR_REG,                          { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_XXTI_SYS_PWR_REG,                             { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_EXT_REGULATOR_SYS_PWR_REG,                    { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_GPIO_MODE_SYS_PWR_REG,                        { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_GPIO_MODE_SYSMEM_SYS_PWR_REG,                 { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_GPIO_MODE_MAU_SYS_PWR_REG,                    { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_TOP_ASB_RESET_SYS_PWR_REG,                    { 0x1,
>> 0x1, 0x0} },
>> +       { EXYNOS5_TOP_ASB_ISOLATION_SYS_PWR_REG,                { 0x1,
>> 0x0, 0x0} },
>> +       { EXYNOS5_GSCL_SYS_PWR_REG,                             { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5_ISP_SYS_PWR_REG,                              { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5_MFC_SYS_PWR_REG,                              { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5_G3D_SYS_PWR_REG,                              { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_DISP1_SYS_PWR_REG,                         { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_MAU_SYS_PWR_REG,                           { 0x7,
>> 0x7, 0x0} },
>> +       { EXYNOS5420_G2D_SYS_PWR_REG,                           { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_MSC_SYS_PWR_REG,                           { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_FSYS_SYS_PWR_REG,                          { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_FSYS2_SYS_PWR_REG,                         { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PSGEN_SYS_PWR_REG,                         { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_PERIC_SYS_PWR_REG,                         { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_WCORE_SYS_PWR_REG,                         { 0x7,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,                 { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,                  { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,                  { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,                  { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG,             { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG,              { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG,             { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG,             { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG,             { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,                  { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,                   { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,                   { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,                   { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG,              { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG,                { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG,                { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG,                { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG,              { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG,              { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG,              { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG,              { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,                   { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_RESET_ISP_SYS_PWR_REG,                    { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,                    { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,                    { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG,               { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG,                 { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG,                 { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG,                 { 0x0,
>> 0x0, 0x0} },
>> +       { EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG,                { 0x0,
>> 0x0, 0x0} },
>
>
> style: Lines over 80 characters.
>

Yeah rgiht it reaches 83rd column , but wanted to maintain the readability.

>
>> +       { PMU_TABLE_END,},
>> +};
>> +
>> +void __iomem *exynos5_list_disable_pmu_reg[] = {
>
>
> static void __iomem * const
>
>> +       EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,
>> +       EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,
>> +       EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,
>> +       EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,
>> +       EXYNOS5_CMU_CLKSTOP_DISP1_SYS_PWR_REG,
>> +       EXYNOS5_CMU_CLKSTOP_MAU_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_DISP1_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_MAU_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_DISP1_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_MAU_SYS_PWR_REG,
>> +};
>> +
>> +void __iomem *exynos5420_list_disable_pmu_reg[] = {
>
>
> static void __iomem * const
>
>> +       EXYNOS5_CMU_CLKSTOP_GSCL_SYS_PWR_REG,
>> +       EXYNOS5_CMU_CLKSTOP_ISP_SYS_PWR_REG,
>> +       EXYNOS5_CMU_CLKSTOP_MFC_SYS_PWR_REG,
>> +       EXYNOS5_CMU_CLKSTOP_G3D_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_GSCL_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_ISP_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_MFC_SYS_PWR_REG,
>> +       EXYNOS5_CMU_SYSCLK_G3D_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_GSCL_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_ISP_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_MFC_SYS_PWR_REG,
>> +       EXYNOS5_CMU_RESET_G3D_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG,
>> +       EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG,
>> +};
>> +
>>   static void __iomem * const exynos5_list_both_cnt_feed[] = {
>>         EXYNOS5_ARM_CORE0_OPTION,
>>         EXYNOS5_ARM_CORE1_OPTION,
>> @@ -371,6 +582,21 @@ static void exynos5_init_pmu(void)
>>         }
>>   }
>>
>> +/*
>> + * exynos_set_core_flag - set the cluster id to IROM register
>> + *                       to ensure that we wake up with the
>> + *                       current cluster.
>> + */
>> +static void exynos_set_core_flag(void)
>> +{
>> +       int cluster_id = (read_cpuid_mpidr() >> 8) & 0xf;
>
>
> Please use proper macros to extract cluster ID from mpidr value.
>
>
>> +
>> +       if (cluster_id)
>> +               __raw_writel(1, EXYNOS_IROM_DATA2);
>> +       else
>> +               __raw_writel(0, EXYNOS_IROM_DATA2);
>
>
> Couldn't __raw_writel(cluster_id, EXYNOS_IROM_DATA2) work here?
>
>
>> +}
>> +
>>   void exynos_sys_powerdown_conf(enum sys_powerdown mode)
>>   {
>>         unsigned int i;
>> @@ -378,6 +604,9 @@ void exynos_sys_powerdown_conf(enum sys_powerdown
>> mode)
>>         if (soc_is_exynos5250())
>>                 exynos5_init_pmu();
>>
>> +       if (soc_is_exynos5420())
>> +               exynos_set_core_flag();
>> +
>>         for (i = 0; (exynos_pmu_config[i].reg != PMU_TABLE_END) ; i++)
>>                 __raw_writel(exynos_pmu_config[i].val[mode],
>>                                 exynos_pmu_config[i].reg);
>> @@ -389,9 +618,27 @@ void exynos_sys_powerdown_conf(enum sys_powerdown
>> mode)
>>         }
>>   }
>>
>> +static void exynos5_power_off(void)
>> +{
>> +       unsigned int tmp;
>> +
>> +       pr_info("Power down.\n");
>> +       tmp = __raw_readl(EXYNOS_PS_HOLD_CONTROL);
>> +       tmp &= ~(1 << 8);
>
>
> This is wrong as it assumes fixed polarity of PSHOLD pin. Instead, it should
> just invert the setting. A simple XOR should be enough.

OK, Will change.

>
>
>> +       __raw_writel(tmp, EXYNOS_PS_HOLD_CONTROL);
>> +
>> +       /* Wait a little so we don't give a false warning below */
>> +       mdelay(100);
>> +
>> +       pr_err("Power down failed, please power off system manually.\n");
>> +       while (1)
>> +               ;
>> +}
>> +
>>   static int __init exynos_pmu_init(void)
>>   {
>>         unsigned int value;
>> +       int i;
>>
>>         exynos_pmu_config = exynos4210_pmu_config;
>>
>> @@ -416,6 +663,66 @@ static int __init exynos_pmu_init(void)
>>
>>                 exynos_pmu_config = exynos5250_pmu_config;
>>                 pr_info("EXYNOS5250 PMU Initialize\n");
>> +       } else if (soc_is_exynos5420()) {
>> +               /*
>> +                * Set the CMU_RESET, CMU_SYSCLK and CMU_CLKSTOP registers
>> +                * for local power blocks to Low initially as per Table
>> 8-4:
>> +                * "System-Level Power-Down Configuration Registers".
>> +                */
>> +               for (i = 0;
>> +                       i < ARRAY_SIZE(exynos5420_list_disable_pmu_reg);
>> +                       i++)
>> +                       __raw_writel(0x0,
>> exynos5420_list_disable_pmu_reg[i]);
>> +               /* Time taken to stabilized XXTI clock */
>> +               __raw_writel(0x005dc, EXYNOS5_XXTI_DURATION3);
>> +
>> +               /* Enable USE_STANDBY_WFI for all CORE */
>> +               __raw_writel(EXYNOS5420_USE_STANDBY_WFI_ALL,
>> +                               S5P_CENTRAL_SEQ_OPTION);
>> +
>> +               value = __raw_readl(EXYNOS_L2_OPTION(0));
>> +               value &= ~EXYNOS5_USE_RETENTION;
>> +               __raw_writel(value, EXYNOS_L2_OPTION(0));
>> +
>> +               value = __raw_readl(EXYNOS_L2_OPTION(1));
>> +               value &= ~EXYNOS5_USE_RETENTION;
>> +               __raw_writel(value, EXYNOS_L2_OPTION(1));
>> +
>> +               /*
>> +                * If L2_COMMON is turned off, clocks related to ATB async
>> +                * bridge are gated. Thus, when ISP power is gated, LPI
>> +                * may get stuck.
>> +                */
>> +               value = __raw_readl(EXYNOS5420_LPI_MASK);
>> +               value |= EXYNOS5420_ATB_ISP_ARM;
>> +               __raw_writel(value, EXYNOS5420_LPI_MASK);
>> +               value = __raw_readl(EXYNOS5420_LPI_MASK1);
>> +               value |= EXYNOS5420_ATB_KFC;
>> +               __raw_writel(value, EXYNOS5420_LPI_MASK1);
>> +
>> +               /* Prevent issue of new bus request from L2 memory */
>> +               value = __raw_readl(EXYNOS5420_ARM_COMMON_OPTION);
>> +               value |= EXYNOS5_SKIP_DEACTIVATE_ACEACP_IN_PWDN;
>> +               __raw_writel(value, EXYNOS5420_ARM_COMMON_OPTION);
>> +
>> +               value = __raw_readl(EXYNOS5420_KFC_COMMON_OPTION);
>> +               value |= EXYNOS5_SKIP_DEACTIVATE_ACEACP_IN_PWDN;
>> +               __raw_writel(value, EXYNOS5420_KFC_COMMON_OPTION);
>> +
>> +               /*
>> +                * This setting is to reduce suspend/resume time.
>> +                */
>> +               __raw_writel(DUR_WAIT_RESET,
>> EXYNOS5420_LOGIC_RESET_DURATION3);
>> +
>> +               /* Serialized CPU wakeup of Eagle */
>> +               __raw_writel(SPREAD_ENABLE,
>> EXYNOS5420_ARM_INTR_SPREAD_ENABLE);
>> +               __raw_writel(SPREAD_USE_STANDWFI,
>> +                       EXYNOS5420_ARM_INTR_SPREAD_USE_STANDBYWFI);
>> +               __raw_writel(0x1, EXYNOS5420_UP_SCHEDULER);
>> +
>> +               exynos_pmu_config = exynos5420_pmu_config;
>> +               pm_power_off = exynos5_power_off;
>> +               pr_info("EXYNOS5420 PMU Initialized\n");
>>         } else {
>>                 pr_info("EXYNOS: PMU not supported\n");
>>         }
>> diff --git a/arch/arm/mach-exynos/regs-pmu.h
>> b/arch/arm/mach-exynos/regs-pmu.h
>> index 4f6a256..cfbfc575 100644
>> --- a/arch/arm/mach-exynos/regs-pmu.h
>> +++ b/arch/arm/mach-exynos/regs-pmu.h
>> @@ -39,6 +39,8 @@
>>   #define S5P_INFORM6                           S5P_PMUREG(0x0818)
>>   #define S5P_INFORM7                           S5P_PMUREG(0x081C)
>>
>> +#define EXYNOS_IROM_DATA2                      S5P_PMUREG(0x0988)
>
>
> Is this register available on Exynos4 too?

Yeah, it is available in Exynos4 also.

>
>
>> +
>>   #define S5P_ARM_CORE0_LOWPWR                  S5P_PMUREG(0x1000)
>>   #define S5P_DIS_IRQ_CORE0                     S5P_PMUREG(0x1004)
>>   #define S5P_DIS_IRQ_CENTRAL0                  S5P_PMUREG(0x1008)
>> @@ -108,6 +110,10 @@
>>   #define S5P_ARM_CORE1_CONFIGURATION           S5P_PMUREG(0x2080)
>>   #define S5P_ARM_CORE1_STATUS                  S5P_PMUREG(0x2084)
>>
>> +#define EXYNOS5_ARM_L2_OPTION                  S5P_PMUREG(0x2608)
>> +#define EXYNOS5_USE_RETENTION                  BIT(4)
>> +
>> +
>>   #define S5P_PAD_RET_MAUDIO_OPTION             S5P_PMUREG(0x3028)
>>   #define S5P_PAD_RET_GPIO_OPTION                       S5P_PMUREG(0x3108)
>>   #define S5P_PAD_RET_UART_OPTION                       S5P_PMUREG(0x3128)
>> @@ -121,6 +127,36 @@
>>
>>   #define S5P_CHECK_SLEEP                               0x00000BAD
>>
>> +#define S5P_ARM_CORE0_CONFIGURATION            S5P_PMUREG(0x2000)
>> +#define S5P_ARM_CORE0_STATUS                   S5P_PMUREG(0x2004)
>> +#define S5P_ARM_CORE0_OPTION                   S5P_PMUREG(0x2008)
>> +
>> +#define EXYNOS_ARM_CORE_OPTION(_nr)            (S5P_ARM_CORE0_OPTION \
>> +                                               + ((_nr) * 0x80))
>> +#define EXYNOS_ARM_CORE_STATUS(_nr)            (S5P_ARM_CORE0_STATUS \
>> +                                               + ((_nr) * 0x80))
>> +#define EXYNOS_ARM_CORE_CONFIGURATION(_nr)     \
>> +                       (S5P_ARM_CORE0_CONFIGURATION + ((_nr) * 0x80))
>> +#define EXYNOS_CORE_LOCAL_PWR_EN               0x3
>> +
>> +#define EXYNOS_ARM_COMMON_CONFIGURATION                S5P_PMUREG(0x2500)
>> +#define EXYNOS_ARM_COMMON_STATUS               S5P_PMUREG(0x2504)
>> +#define EXYNOS_COMMON_CONFIGURATION(_nr)       \
>> +                       (EXYNOS_ARM_COMMON_CONFIGURATION + ((_nr) * 0x80))
>> +#define EXYNOS_COMMON_STATUS(_nr)              \
>> +                       (EXYNOS_COMMON_CONFIGURATION(_nr) + 0x4)
>> +#define EXYNOS_COMMON_OPTION(_nr)              \
>> +                       (EXYNOS_COMMON_CONFIGURATION(_nr) + 0x8)
>> +
>> +#define EXYNOS_ARM_L2_CONFIGURATION            S5P_PMUREG(0x2600)
>> +#define EXYNOS_L2_CONFIGURATION(_nr)           \
>> +                       (EXYNOS_ARM_L2_CONFIGURATION + ((_nr) * 0x80))
>> +#define EXYNOS_L2_STATUS(_nr)                  \
>> +                       (EXYNOS_L2_CONFIGURATION(_nr) + 0x4)
>> +#define EXYNOS_L2_OPTION(_nr)                  \
>> +                       (EXYNOS_L2_CONFIGURATION(_nr) + 0x8)
>> +#define EXYNOS_L2_COMMON_PWR_EN                        0x3
>> +
>>   /* Only for EXYNOS4210 */
>>   #define S5P_CMU_CLKSTOP_LCD1_LOWPWR   S5P_PMUREG(0x1154)
>>   #define S5P_CMU_RESET_LCD1_LOWPWR     S5P_PMUREG(0x1174)
>> @@ -313,4 +349,196 @@
>>
>>   #define EXYNOS5_OPTION_USE_RETENTION                          (1 << 4)
>>
>> +/* Only for EXYNOS5420 */
>> +#define EXYNOS5420_ISP_ARM_OPTION
>> S5P_PMUREG(0x2488)
>> +#define EXYNOS5420_L2RSTDISABLE_VALUE                          (1 << 3)
>> +
>> +#define EXYNOS5420_LPI_MASK
>> S5P_PMUREG(0x0004)
>> +#define EXYNOS5420_LPI_MASK1
>> S5P_PMUREG(0x0008)
>> +#define EXYNOS5420_UFS                                         (1 << 8)
>> +#define EXYNOS5420_ATB_KFC                                     (1 << 13)
>> +#define EXYNOS5420_ATB_ISP_ARM                                 (1 << 19)
>> +#define EXYNOS5420_EMULATION                                   (1 << 31)
>> +#define ATB_ISP_ARM                                            (1 << 12)
>> +#define ATB_KFC                                                        (1
>> << 13)
>> +#define ATB_NOC                                                        (1
>> << 14)
>> +
>> +#define EXYNOS5420_ARM_INTR_SPREAD_ENABLE
>> S5P_PMUREG(0x0100)
>> +#define EXYNOS5420_ARM_INTR_SPREAD_USE_STANDBYWFI
>> S5P_PMUREG(0x0104)
>> +#define EXYNOS5420_UP_SCHEDULER
>> S5P_PMUREG(0x0120)
>> +#define SPREAD_ENABLE                                          0xF
>> +#define SPREAD_USE_STANDWFI                                    0xF
>> +
>> +#define EXYNOS5420_BB_CON1
>> S5P_PMUREG(0x0784)
>> +#define EXYNOS5420_BB_SEL_EN                                   (1 << 31)
>> +#define EXYNOS5420_BB_PMOS_EN                                  (1 << 7)
>> +#define EXYNOS5420_BB_1300X                                    0XF
>> +
>> +#define EXYNOS5420_ARM_CORE2_SYS_PWR_REG
>> S5P_PMUREG(0x1020)
>> +#define EXYNOS5420_DIS_IRQ_ARM_CORE2_LOCAL_SYS_PWR_REG
>> S5P_PMUREG(0x1024)
>> +#define EXYNOS5420_DIS_IRQ_ARM_CORE2_CENTRAL_SYS_PWR_REG
>> S5P_PMUREG(0x1028)
>> +#define EXYNOS5420_ARM_CORE3_SYS_PWR_REG
>> S5P_PMUREG(0x1030)
>> +#define EXYNOS5420_DIS_IRQ_ARM_CORE3_LOCAL_SYS_PWR_REG
>> S5P_PMUREG(0x1034)
>> +#define EXYNOS5420_DIS_IRQ_ARM_CORE3_CENTRAL_SYS_PWR_REG
>> S5P_PMUREG(0x1038)
>> +#define EXYNOS5420_KFC_CORE0_SYS_PWR_REG
>> S5P_PMUREG(0x1040)
>> +#define EXYNOS5420_DIS_IRQ_KFC_CORE0_LOCAL_SYS_PWR_REG
>> S5P_PMUREG(0x1044)
>> +#define EXYNOS5420_DIS_IRQ_KFC_CORE0_CENTRAL_SYS_PWR_REG
>> S5P_PMUREG(0x1048)
>> +#define EXYNOS5420_KFC_CORE1_SYS_PWR_REG
>> S5P_PMUREG(0x1050)
>> +#define EXYNOS5420_DIS_IRQ_KFC_CORE1_LOCAL_SYS_PWR_REG
>> S5P_PMUREG(0x1054)
>> +#define EXYNOS5420_DIS_IRQ_KFC_CORE1_CENTRAL_SYS_PWR_REG
>> S5P_PMUREG(0x1058)
>> +#define EXYNOS5420_KFC_CORE2_SYS_PWR_REG
>> S5P_PMUREG(0x1060)
>> +#define EXYNOS5420_DIS_IRQ_KFC_CORE2_LOCAL_SYS_PWR_REG
>> S5P_PMUREG(0x1064)
>> +#define EXYNOS5420_DIS_IRQ_KFC_CORE2_CENTRAL_SYS_PWR_REG
>> S5P_PMUREG(0x1068)
>> +#define EXYNOS5420_KFC_CORE3_SYS_PWR_REG
>> S5P_PMUREG(0x1070)
>> +#define EXYNOS5420_DIS_IRQ_KFC_CORE3_LOCAL_SYS_PWR_REG
>> S5P_PMUREG(0x1074)
>> +#define EXYNOS5420_DIS_IRQ_KFC_CORE3_CENTRAL_SYS_PWR_REG
>> S5P_PMUREG(0x1078)
>> +#define EXYNOS5420_ISP_ARM_SYS_PWR_REG
>> S5P_PMUREG(0x1090)
>> +#define EXYNOS5420_DIS_IRQ_ISP_ARM_LOCAL_SYS_PWR_REG
>> S5P_PMUREG(0x1094)
>> +#define EXYNOS5420_DIS_IRQ_ISP_ARM_CENTRAL_SYS_PWR_REG
>> S5P_PMUREG(0x1098)
>> +#define EXYNOS5420_ARM_COMMON_SYS_PWR_REG
>> S5P_PMUREG(0x10A0)
>> +#define EXYNOS5420_KFC_COMMON_SYS_PWR_REG
>> S5P_PMUREG(0x10B0)
>> +#define EXYNOS5420_KFC_L2_SYS_PWR_REG
>> S5P_PMUREG(0x10D0)
>> +#define EXYNOS5420_DPLL_SYSCLK_SYS_PWR_REG
>> S5P_PMUREG(0x1158)
>> +#define EXYNOS5420_IPLL_SYSCLK_SYS_PWR_REG
>> S5P_PMUREG(0x115C)
>> +#define EXYNOS5420_KPLL_SYSCLK_SYS_PWR_REG
>> S5P_PMUREG(0x1160)
>> +#define EXYNOS5420_RPLL_SYSCLK_SYS_PWR_REG
>> S5P_PMUREG(0x1174)
>> +#define EXYNOS5420_SPLL_SYSCLK_SYS_PWR_REG
>> S5P_PMUREG(0x1178)
>> +#define EXYNOS5420_INTRAM_MEM_SYS_PWR_REG
>> S5P_PMUREG(0x11B8)
>> +#define EXYNOS5420_INTROM_MEM_SYS_PWR_REG
>> S5P_PMUREG(0x11BC)
>> +#define EXYNOS5420_ONENANDXL_MEM_SYS_PWR
>> S5P_PMUREG(0x11C0)
>> +#define EXYNOS5420_USBDEV_MEM_SYS_PWR
>> S5P_PMUREG(0x11CC)
>> +#define EXYNOS5420_USBDEV1_MEM_SYS_PWR
>> S5P_PMUREG(0x11D0)
>> +#define EXYNOS5420_SDMMC_MEM_SYS_PWR
>> S5P_PMUREG(0x11D4)
>> +#define EXYNOS5420_CSSYS_MEM_SYS_PWR
>> S5P_PMUREG(0x11D8)
>> +#define EXYNOS5420_SECSS_MEM_SYS_PWR
>> S5P_PMUREG(0x11DC)
>> +#define EXYNOS5420_ROTATOR_MEM_SYS_PWR
>> S5P_PMUREG(0x11E0)
>> +#define EXYNOS5420_INTRAM_MEM_SYS_PWR
>> S5P_PMUREG(0x11E4)
>> +#define EXYNOS5420_INTROM_MEM_SYS_PWR
>> S5P_PMUREG(0x11E8)
>> +#define EXYNOS5420_PAD_RETENTION_JTAG_SYS_PWR_REG
>> S5P_PMUREG(0x1208)
>> +#define EXYNOS5420_PAD_RETENTION_DRAM_SYS_PWR_REG
>> S5P_PMUREG(0x1210)
>> +#define EXYNOS5420_PAD_RETENTION_UART_SYS_PWR_REG
>> S5P_PMUREG(0x1214)
>> +#define EXYNOS5420_PAD_RETENTION_MMC0_SYS_PWR_REG
>> S5P_PMUREG(0x1218)
>> +#define EXYNOS5420_PAD_RETENTION_MMC1_SYS_PWR_REG
>> S5P_PMUREG(0x121C)
>> +#define EXYNOS5420_PAD_RETENTION_MMC2_SYS_PWR_REG
>> S5P_PMUREG(0x1220)
>> +#define EXYNOS5420_PAD_RETENTION_HSI_SYS_PWR_REG
>> S5P_PMUREG(0x1224)
>> +#define EXYNOS5420_PAD_RETENTION_EBIA_SYS_PWR_REG
>> S5P_PMUREG(0x1228)
>> +#define EXYNOS5420_PAD_RETENTION_EBIB_SYS_PWR_REG
>> S5P_PMUREG(0x122C)
>> +#define EXYNOS5420_PAD_RETENTION_SPI_SYS_PWR_REG
>> S5P_PMUREG(0x1230)
>> +#define EXYNOS5420_PAD_RETENTION_DRAM_COREBLK_SYS_PWR_REG
>> S5P_PMUREG(0x1234)
>> +#define EXYNOS5420_DISP1_SYS_PWR_REG
>> S5P_PMUREG(0x1410)
>> +#define EXYNOS5420_MAU_SYS_PWR_REG
>> S5P_PMUREG(0x1414)
>> +#define EXYNOS5420_G2D_SYS_PWR_REG
>> S5P_PMUREG(0x1418)
>> +#define EXYNOS5420_MSC_SYS_PWR_REG
>> S5P_PMUREG(0x141C)
>> +#define EXYNOS5420_FSYS_SYS_PWR_REG
>> S5P_PMUREG(0x1420)
>> +#define EXYNOS5420_FSYS2_SYS_PWR_REG
>> S5P_PMUREG(0x1424)
>> +#define EXYNOS5420_PSGEN_SYS_PWR_REG
>> S5P_PMUREG(0x1428)
>> +#define EXYNOS5420_PERIC_SYS_PWR_REG
>> S5P_PMUREG(0x142C)
>> +#define EXYNOS5420_WCORE_SYS_PWR_REG
>> S5P_PMUREG(0x1430)
>> +#define EXYNOS5420_CMU_CLKSTOP_DISP1_SYS_PWR_REG
>> S5P_PMUREG(0x1490)
>> +#define EXYNOS5420_CMU_CLKSTOP_MAU_SYS_PWR_REG
>> S5P_PMUREG(0x1494)
>> +#define EXYNOS5420_CMU_CLKSTOP_G2D_SYS_PWR_REG
>> S5P_PMUREG(0x1498)
>> +#define EXYNOS5420_CMU_CLKSTOP_MSC_SYS_PWR_REG
>> S5P_PMUREG(0x149C)
>> +#define EXYNOS5420_CMU_CLKSTOP_FSYS_SYS_PWR_REG
>> S5P_PMUREG(0x14A0)
>> +#define EXYNOS5420_CMU_CLKSTOP_FSYS2_SYS_PWR_REG
>> S5P_PMUREG(0x14A4)
>> +#define EXYNOS5420_CMU_CLKSTOP_PSGEN_SYS_PWR_REG
>> S5P_PMUREG(0x14A8)
>> +#define EXYNOS5420_CMU_CLKSTOP_PERIC_SYS_PWR_REG
>> S5P_PMUREG(0x14AC)
>> +#define EXYNOS5420_CMU_CLKSTOP_WCORE_SYS_PWR_REG
>> S5P_PMUREG(0x14B0)
>> +#define EXYNOS5420_CMU_SYSCLK_TOPPWR_SYS_PWR_REG
>> S5P_PMUREG(0x14BC)
>> +#define EXYNOS5420_CMU_SYSCLK_DISP1_SYS_PWR_REG
>> S5P_PMUREG(0x14D0)
>> +#define EXYNOS5420_CMU_SYSCLK_MAU_SYS_PWR_REG
>> S5P_PMUREG(0x14D4)
>> +#define EXYNOS5420_CMU_SYSCLK_G2D_SYS_PWR_REG
>> S5P_PMUREG(0x14D8)
>> +#define EXYNOS5420_CMU_SYSCLK_MSC_SYS_PWR_REG
>> S5P_PMUREG(0x14DC)
>> +#define EXYNOS5420_CMU_SYSCLK_FSYS_SYS_PWR_REG
>> S5P_PMUREG(0x14E0)
>> +#define EXYNOS5420_CMU_SYSCLK_FSYS2_SYS_PWR_REG
>> S5P_PMUREG(0x14E4)
>> +#define EXYNOS5420_CMU_SYSCLK_PSGEN_SYS_PWR_REG
>> S5P_PMUREG(0x14E8)
>> +#define EXYNOS5420_CMU_SYSCLK_PERIC_SYS_PWR_REG
>> S5P_PMUREG(0x14EC)
>> +#define EXYNOS5420_CMU_SYSCLK_WCORE_SYS_PWR_REG
>> S5P_PMUREG(0x14F0)
>> +#define EXYNOS5420_CMU_SYSCLK_SYSMEM_TOPPWR_SYS_PWR_REG
>> S5P_PMUREG(0x14F4)
>> +#define EXYNOS5420_CMU_RESET_FSYS2_SYS_PWR_REG
>> S5P_PMUREG(0x1570)
>> +#define EXYNOS5420_CMU_RESET_PSGEN_SYS_PWR_REG
>> S5P_PMUREG(0x1574)
>> +#define EXYNOS5420_CMU_RESET_PERIC_SYS_PWR_REG
>> S5P_PMUREG(0x1578)
>> +#define EXYNOS5420_CMU_RESET_WCORE_SYS_PWR_REG
>> S5P_PMUREG(0x157C)
>> +#define EXYNOS5420_CMU_RESET_DISP1_SYS_PWR_REG
>> S5P_PMUREG(0x1590)
>> +#define EXYNOS5420_CMU_RESET_MAU_SYS_PWR_REG
>> S5P_PMUREG(0x1594)
>> +#define EXYNOS5420_CMU_RESET_G2D_SYS_PWR_REG
>> S5P_PMUREG(0x1598)
>> +#define EXYNOS5420_CMU_RESET_MSC_SYS_PWR_REG
>> S5P_PMUREG(0x159C)
>> +#define EXYNOS5420_CMU_RESET_FSYS_SYS_PWR_REG
>> S5P_PMUREG(0x15A0)
>> +#define EXYNOS5420_SFR_AXI_CGDIS1
>> S5P_PMUREG(0x15E4)
>> +#define EXYNOS_ARM_CORE2_CONFIGURATION
>> S5P_PMUREG(0x2100)
>> +#define EXYNOS5420_ARM_CORE2_OPTION
>> S5P_PMUREG(0x2108)
>> +#define EXYNOS_ARM_CORE3_CONFIGURATION
>> S5P_PMUREG(0x2180)
>> +#define EXYNOS5420_ARM_CORE3_OPTION
>> S5P_PMUREG(0x2188)
>> +#define EXYNOS5420_ARM_COMMON_STATUS
>> S5P_PMUREG(0x2504)
>> +#define EXYNOS5420_ARM_COMMON_OPTION
>> S5P_PMUREG(0x2508)
>> +#define EXYNOS5420_KFC_COMMON_STATUS
>> S5P_PMUREG(0x2584)
>> +#define EXYNOS5420_KFC_COMMON_OPTION
>> S5P_PMUREG(0x2588)
>> +#define EXYNOS5420_LOGIC_RESET_DURATION3
>> S5P_PMUREG(0x2D1C)
>> +
>> +#define EXYNOS5420_PAD_RET_GPIO_OPTION
>> S5P_PMUREG(0x30C8)
>> +#define EXYNOS5420_PAD_RET_UART_OPTION
>> S5P_PMUREG(0x30E8)
>> +#define EXYNOS5420_PAD_RET_MMCA_OPTION
>> S5P_PMUREG(0x3108)
>> +#define EXYNOS5420_PAD_RET_MMCB_OPTION
>> S5P_PMUREG(0x3128)
>> +#define EXYNOS5420_PAD_RET_MMCC_OPTION
>> S5P_PMUREG(0x3148)
>> +#define EXYNOS5420_PAD_RET_HSI_OPTION
>> S5P_PMUREG(0x3168)
>> +#define EXYNOS5420_PAD_RET_SPI_OPTION
>> S5P_PMUREG(0x31C8)
>> +#define EXYNOS5420_PAD_RET_DRAM_COREBLK_OPTION
>> S5P_PMUREG(0x31E8)
>> +#define EXYNOS_PAD_RET_DRAM_OPTION
>> S5P_PMUREG(0x3008)
>> +#define EXYNOS_PAD_RET_MAUDIO_OPTION
>> S5P_PMUREG(0x3028)
>> +#define EXYNOS_PAD_RET_JTAG_OPTION
>> S5P_PMUREG(0x3048)
>> +#define EXYNOS_PAD_RET_GPIO_OPTION
>> S5P_PMUREG(0x3108)
>> +#define EXYNOS_PAD_RET_UART_OPTION
>> S5P_PMUREG(0x3128)
>> +#define EXYNOS_PAD_RET_MMCA_OPTION
>> S5P_PMUREG(0x3148)
>> +#define EXYNOS_PAD_RET_MMCB_OPTION
>> S5P_PMUREG(0x3168)
>> +#define EXYNOS_PAD_RET_EBIA_OPTION
>> S5P_PMUREG(0x3188)
>> +#define EXYNOS_PAD_RET_EBIB_OPTION
>> S5P_PMUREG(0x31A8)
>> +
>> +#define EXYNOS_PS_HOLD_CONTROL
>> S5P_PMUREG(0x330C)
>> +#define EXYNOS5_XXTI_DURATION3
>> S5P_PMUREG(0x343C)
>> +
>> +/* For SYS_PWR_REG */
>> +#define EXYNOS_SYS_PWR_CFG                                     (1 << 0)
>> +
>> +#define EXYNOS5420_MFC_CONFIGURATION
>> S5P_PMUREG(0x4060)
>> +#define EXYNOS5420_MFC_STATUS
>> S5P_PMUREG(0x4064)
>> +#define EXYNOS5420_MFC_OPTION
>> S5P_PMUREG(0x4068)
>> +#define EXYNOS5420_G3D_CONFIGURATION
>> S5P_PMUREG(0x4080)
>> +#define EXYNOS5420_G3D_STATUS
>> S5P_PMUREG(0x4084)
>> +#define EXYNOS5420_G3D_OPTION
>> S5P_PMUREG(0x4088)
>> +#define EXYNOS5420_DISP0_CONFIGURATION
>> S5P_PMUREG(0x40A0)
>> +#define EXYNOS5420_DISP0_STATUS
>> S5P_PMUREG(0x40A4)
>> +#define EXYNOS5420_DISP0_OPTION
>> S5P_PMUREG(0x40A8)
>> +#define EXYNOS5420_DISP1_CONFIGURATION
>> S5P_PMUREG(0x40C0)
>> +#define EXYNOS5420_DISP1_STATUS
>> S5P_PMUREG(0x40C4)
>> +#define EXYNOS5420_DISP1_OPTION
>> S5P_PMUREG(0x40C8)
>> +#define EXYNOS5420_MAU_CONFIGURATION
>> S5P_PMUREG(0x40E0)
>> +#define EXYNOS5420_MAU_STATUS
>> S5P_PMUREG(0x40E4)
>> +#define EXYNOS5420_MAU_OPTION
>> S5P_PMUREG(0x40E8)
>> +#define EXYNOS5420_FSYS2_OPTION
>> S5P_PMUREG(0x4168)
>> +#define EXYNOS5420_PSGEN_OPTION
>> S5P_PMUREG(0x4188)
>
>
> style: A lot of lines over 80 characters.
>
>
>> +
>> +/* For EXYNOS_CENTRAL_SEQ_OPTION */
>> +#define EXYNOS5_USE_STANDBYWFI_ARM_CORE0                       (1 << 16)
>> +#define EXYNOS5_USE_STANDBYWFI_ARM_CORE1                       (1 << 17)
>> +#define EXYNOS5_USE_STANDBYWFE_ARM_CORE0                       (1 << 24)
>> +#define EXYNOS5_USE_STANDBYWFE_ARM_CORE1                       (1 << 25)
>> +
>> +#define EXYNOS5420_ARM_USE_STANDBY_WFI0                                (1
>> << 4)
>> +#define EXYNOS5420_ARM_USE_STANDBY_WFI1                                (1
>> << 5)
>> +#define EXYNOS5420_ARM_USE_STANDBY_WFI2                                (1
>> << 6)
>> +#define EXYNOS5420_ARM_USE_STANDBY_WFI3                                (1
>> << 7)
>> +#define EXYNOS5420_KFC_USE_STANDBY_WFI0                                (1
>> << 8)
>> +#define EXYNOS5420_KFC_USE_STANDBY_WFI1                                (1
>> << 9)
>> +#define EXYNOS5420_KFC_USE_STANDBY_WFI2                                (1
>> << 10)
>> +#define EXYNOS5420_KFC_USE_STANDBY_WFI3                                (1
>> << 11)
>> +#define EXYNOS5420_ARM_USE_STANDBY_WFE0                                (1
>> << 16)
>> +#define EXYNOS5420_ARM_USE_STANDBY_WFE1                                (1
>> << 17)
>> +#define EXYNOS5420_ARM_USE_STANDBY_WFE2                                (1
>> << 18)
>> +#define EXYNOS5420_ARM_USE_STANDBY_WFE3                                (1
>> << 19)
>> +#define EXYNOS5420_KFC_USE_STANDBY_WFE0                                (1
>> << 20)
>> +#define EXYNOS5420_KFC_USE_STANDBY_WFE1                                (1
>> << 21)
>> +#define EXYNOS5420_KFC_USE_STANDBY_WFE2                                (1
>> << 22)
>> +#define EXYNOS5420_KFC_USE_STANDBY_WFE3                                (1
>> << 23)
>
>
> BIT() macro could be used. The same for similar bit definitions above.

Sure, will modify.

>
> Best regards,
> Tomasz

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

* [PATCH V2 2/3] ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420
  2014-04-15 19:03   ` Tomasz Figa
  2014-04-15 19:04     ` Tomasz Figa
@ 2014-04-28 12:39     ` Vikas Sajjan
  1 sibling, 0 replies; 12+ messages in thread
From: Vikas Sajjan @ 2014-04-28 12:39 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Tomasz,


On Wed, Apr 16, 2014 at 12:33 AM, Tomasz Figa <tomasz.figa@gmail.com> wrote:
> Hi Vikas,
>
> Basically same comments as for the series for Exynos5260. Also see more
> comments inline.
>
>
> On 27.03.2014 07:13, Vikas Sajjan wrote:
>>
>> Adds Suspend-to-RAM support for EXYNOS5420
>>
>> Signed-off-by: Abhilash Kesavan <a.kesavan@samsung.com>
>> Signed-off-by: Vikas Sajjan <vikas.sajjan@samsung.com>
>> ---
>>   arch/arm/mach-exynos/pm.c                    |  148
>> ++++++++++++++++++++++----
>>   arch/arm/mach-exynos/regs-pmu.h              |    4 +-
>>   arch/arm/plat-samsung/include/plat/map-s5p.h |    2 +
>>   drivers/clk/samsung/clk-exynos5420.c         |   32 ++++++
>>   4 files changed, 167 insertions(+), 19 deletions(-)
>>
>> diff --git a/arch/arm/mach-exynos/pm.c b/arch/arm/mach-exynos/pm.c
>> index 15af0ce..aa3c2c8 100644
>> --- a/arch/arm/mach-exynos/pm.c
>> +++ b/arch/arm/mach-exynos/pm.c
>> @@ -59,6 +59,16 @@ static struct sleep_save exynos_core_save[] = {
>>         SAVE_ITEM(S5P_SROM_BC3),
>>   };
>>
>> +static struct sleep_save exynos5420_cpustate_save[] = {
>> +       SAVE_ITEM(EXYNOS5420_VA_CPU_STATE),
>> +};
>> +
>> +static struct sleep_save exynos5420_reg_save[] = {
>> +       SAVE_ITEM(EXYNOS5_SYS_DISP1_BLK_CFG),
>> +       SAVE_ITEM(S5P_PMU_SPARE3),
>> +};
>> +
>> +
>>   /*
>>    * GIC wake-up support
>>    */
>> @@ -81,7 +91,7 @@ static int exynos_irq_set_wake(struct irq_data *data,
>> unsigned int state)
>>   {
>>         const struct exynos_wkup_irq *wkup_irq;
>>
>> -       if (soc_is_exynos5250())
>> +       if (soc_is_exynos5250() || soc_is_exynos5420())
>>                 wkup_irq = exynos5250_wkup_irq;
>>         else
>>                 wkup_irq = exynos4_wkup_irq;
>> @@ -109,7 +119,15 @@ static int exynos_cpu_suspend(unsigned long arg)
>>         outer_flush_all();
>>   #endif
>>
>> -       if (soc_is_exynos5250())
>> +       /*
>> +        * Clear IRAM register for cpu state so that primary CPU does
>> +        * not enter low power start in U-Boot.
>> +        * This is specific to exynos5420 SoC only.
>> +        */
>> +       if (soc_is_exynos5420())
>> +               __raw_writel(0x0, EXYNOS5420_VA_CPU_STATE);
>> +
>> +       if (soc_is_exynos5250() || soc_is_exynos5420())
>>                 flush_cache_all();
>>
>>         /* issue the standby signal into the pm unit. */
>> @@ -135,6 +153,20 @@ static void exynos_pm_prepare(void)
>>                 tmp = __raw_readl(EXYNOS5_JPEG_MEM_OPTION);
>>                 tmp &= ~EXYNOS5_OPTION_USE_RETENTION;
>>                 __raw_writel(tmp, EXYNOS5_JPEG_MEM_OPTION);
>> +       } else if (soc_is_exynos5420()) {
>> +
>
>
> nit: Unnecessary blank line.

OK.

>
>
>> +               s3c_pm_do_save(exynos5420_reg_save,
>> +                       ARRAY_SIZE(exynos5420_reg_save));
>> +
>> +               /*
>> +                * The cpu state needs to be saved and restored so that
>> the
>> +                * secondary CPUs will enter low power start. Though the
>> U-Boot
>> +                * is setting the cpu state with low power flag, the
>> kernel
>> +                * needs to restore it back in case, the primary cpu fails
>> to
>> +                * suspend for any reason
>> +                */
>> +               s3c_pm_do_save(exynos5420_cpustate_save,
>> +                       ARRAY_SIZE(exynos5420_cpustate_save));
>>         }
>>
>>         /* Set value of power down register for sleep mode */
>> @@ -145,11 +177,34 @@ static void exynos_pm_prepare(void)
>>         /* ensure at least INFORM0 has the resume address */
>>
>>         __raw_writel(virt_to_phys(exynos_cpu_resume), S5P_INFORM0);
>> +
>> +       if (soc_is_exynos5420()) {
>> +
>> +               tmp = __raw_readl(EXYNOS5_ARM_L2_OPTION);
>> +               tmp &= ~EXYNOS5_USE_RETENTION;
>> +               __raw_writel(tmp, EXYNOS5_ARM_L2_OPTION);
>> +
>> +               tmp = __raw_readl(EXYNOS5420_SFR_AXI_CGDIS1);
>> +               tmp |= EXYNOS5420_UFS;
>> +               __raw_writel(tmp, EXYNOS5420_SFR_AXI_CGDIS1);
>> +
>> +               tmp = __raw_readl(EXYNOS5420_ARM_COMMON_OPTION);
>> +               tmp &= ~EXYNOS5420_L2RSTDISABLE_VALUE;
>> +               __raw_writel(tmp, EXYNOS5420_ARM_COMMON_OPTION);
>> +               tmp = __raw_readl(EXYNOS5420_FSYS2_OPTION);
>> +               tmp |= EXYNOS5420_EMULATION;
>> +               __raw_writel(tmp, EXYNOS5420_FSYS2_OPTION);
>> +               tmp = __raw_readl(EXYNOS5420_PSGEN_OPTION);
>> +               tmp |= EXYNOS5420_EMULATION;
>> +               __raw_writel(tmp, EXYNOS5420_PSGEN_OPTION);
>> +       }
>> +
>>   }
>>
>>   static int exynos_pm_suspend(void)
>>   {
>>         unsigned long tmp;
>> +       unsigned long cluster_id;
>>
>>         /* Setting Central Sequence Register for power down mode */
>>
>> @@ -159,10 +214,20 @@ static int exynos_pm_suspend(void)
>>
>>         /* Setting SEQ_OPTION register */
>>
>> -       tmp = (S5P_USE_STANDBY_WFI0 | S5P_USE_STANDBY_WFE0);
>> -       __raw_writel(tmp, S5P_CENTRAL_SEQ_OPTION);
>> +       if (soc_is_exynos5420()) {
>> +               cluster_id = (read_cpuid(CPUID_MPIDR) >> 8) & 0xf;
>> +               if (!cluster_id)
>> +                       __raw_writel(EXYNOS5420_ARM_USE_STANDBY_WFI0,
>> +                                    S5P_CENTRAL_SEQ_OPTION);
>> +               else
>> +                       __raw_writel(EXYNOS5420_KFC_USE_STANDBY_WFI0,
>> +                                    S5P_CENTRAL_SEQ_OPTION);
>> +       } else {
>> +               tmp = (S5P_USE_STANDBY_WFI0 | S5P_USE_STANDBY_WFE0);
>> +               __raw_writel(tmp, S5P_CENTRAL_SEQ_OPTION);
>> +       }
>>
>> -       if (!soc_is_exynos5250()) {
>> +       if (!(soc_is_exynos5250() || soc_is_exynos5420())) {
>>                 /* Save Power control register */
>>                 asm ("mrc p15, 0, %0, c15, c0, 0"
>>                      : "=r" (tmp) : : "cc");
>> @@ -181,6 +246,15 @@ static void exynos_pm_resume(void)
>>   {
>>         unsigned long tmp;
>>
>> +       if (soc_is_exynos5420()) {
>> +               /* Restore the IRAM register cpu state */
>> +               s3c_pm_do_restore(exynos5420_cpustate_save,
>> +                       ARRAY_SIZE(exynos5420_cpustate_save));
>> +
>> +               __raw_writel(EXYNOS5420_USE_STANDBY_WFI_ALL,
>> +                       S5P_CENTRAL_SEQ_OPTION);
>> +       }
>> +
>>         /*
>>          * If PMU failed while entering sleep mode, WFI will be
>>          * ignored by PMU and then exiting cpu_do_idle().
>> @@ -196,7 +270,8 @@ static void exynos_pm_resume(void)
>>                 /* No need to perform below restore code */
>>                 goto early_wakeup;
>>         }
>> -       if (!soc_is_exynos5250()) {
>> +
>> +       if (!(soc_is_exynos5250() || soc_is_exynos5420())) {
>>                 /* Restore Power control register */
>>                 tmp = save_arm_register[0];
>>                 asm volatile ("mcr p15, 0, %0, c15, c0, 0"
>> @@ -212,25 +287,57 @@ static void exynos_pm_resume(void)
>>
>>         /* For release retention */
>>
>> -       __raw_writel((1 << 28), S5P_PAD_RET_MAUDIO_OPTION);
>> -       __raw_writel((1 << 28), S5P_PAD_RET_GPIO_OPTION);
>> -       __raw_writel((1 << 28), S5P_PAD_RET_UART_OPTION);
>> -       __raw_writel((1 << 28), S5P_PAD_RET_MMCA_OPTION);
>> -       __raw_writel((1 << 28), S5P_PAD_RET_MMCB_OPTION);
>> -       __raw_writel((1 << 28), S5P_PAD_RET_EBIA_OPTION);
>> -       __raw_writel((1 << 28), S5P_PAD_RET_EBIB_OPTION);
>> +       if (soc_is_exynos5250()) {
>> +               __raw_writel((1 << 28), S5P_PAD_RET_MAUDIO_OPTION);
>> +               __raw_writel((1 << 28), S5P_PAD_RET_GPIO_OPTION);
>> +               __raw_writel((1 << 28), S5P_PAD_RET_UART_OPTION);
>> +               __raw_writel((1 << 28), S5P_PAD_RET_MMCA_OPTION);
>> +               __raw_writel((1 << 28), S5P_PAD_RET_MMCB_OPTION);
>> +               __raw_writel((1 << 28), S5P_PAD_RET_EBIA_OPTION);
>> +               __raw_writel((1 << 28), S5P_PAD_RET_EBIB_OPTION);
>> +       } else if (soc_is_exynos5420()) {
>> +               __raw_writel(1 << 28, EXYNOS_PAD_RET_DRAM_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS_PAD_RET_MAUDIO_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS_PAD_RET_JTAG_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS5420_PAD_RET_GPIO_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS5420_PAD_RET_UART_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCA_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCB_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS5420_PAD_RET_MMCC_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS5420_PAD_RET_HSI_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS_PAD_RET_EBIA_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS_PAD_RET_EBIB_OPTION);
>> +               __raw_writel(1 << 28, EXYNOS5420_PAD_RET_SPI_OPTION);
>> +               __raw_writel(1 << 28,
>> EXYNOS5420_PAD_RET_DRAM_COREBLK_OPTION);
>> +       }
>>
>>         if (soc_is_exynos5250())
>>                 s3c_pm_do_restore(exynos5_sys_save,
>>                         ARRAY_SIZE(exynos5_sys_save));
>> +       else if (soc_is_exynos5420())
>> +               s3c_pm_do_restore(exynos5420_reg_save,
>> +                       ARRAY_SIZE(exynos5420_reg_save));
>>
>>         s3c_pm_do_restore_core(exynos_core_save,
>> ARRAY_SIZE(exynos_core_save));
>>
>> -       if (IS_ENABLED(CONFIG_SMP) && !soc_is_exynos5250())
>> +       if (IS_ENABLED(CONFIG_SMP) &&
>> +                       !soc_is_exynos5250() && !soc_is_exynos5420())
>>                 scu_enable(S5P_VA_SCU);
>>
>>   early_wakeup:
>>
>> +       if (soc_is_exynos5420()) {
>> +               tmp = __raw_readl(EXYNOS5420_SFR_AXI_CGDIS1);
>> +               tmp &= ~EXYNOS5420_UFS;
>> +               __raw_writel(tmp, EXYNOS5420_SFR_AXI_CGDIS1);
>> +               tmp = __raw_readl(EXYNOS5420_FSYS2_OPTION);
>> +               tmp &= ~EXYNOS5420_EMULATION;
>> +               __raw_writel(tmp, EXYNOS5420_FSYS2_OPTION);
>> +               tmp = __raw_readl(EXYNOS5420_PSGEN_OPTION);
>> +               tmp &= ~EXYNOS5420_EMULATION;
>> +               __raw_writel(tmp, EXYNOS5420_PSGEN_OPTION);
>> +       }
>> +
>>         /* Clear SLEEP mode set in INFORM1 */
>>         __raw_writel(0x0, S5P_INFORM1);
>>
>> @@ -312,10 +419,15 @@ void __init exynos_pm_init(void)
>>         gic_arch_extn.irq_set_wake = exynos_irq_set_wake;
>>
>>         /* All wakeup disable */
>> -       tmp = __raw_readl(S5P_WAKEUP_MASK);
>> -       tmp |= ((0xFF << 8) | (0x1F << 1));
>> -       __raw_writel(tmp, S5P_WAKEUP_MASK);
>> -
>> +       if (soc_is_exynos5420()) {
>> +               tmp = __raw_readl(S5P_WAKEUP_MASK);
>> +               tmp |= ((0x7F << 7) | (0x1F << 1));
>> +               __raw_writel(tmp, S5P_WAKEUP_MASK);
>> +       } else {
>> +               tmp = __raw_readl(S5P_WAKEUP_MASK);
>> +               tmp |= ((0xFF << 8) | (0x1F << 1));
>> +               __raw_writel(tmp, S5P_WAKEUP_MASK);
>> +       }
>>         register_syscore_ops(&exynos_pm_syscore_ops);
>>         suspend_set_ops(&exynos_suspend_ops);
>>   }
>> diff --git a/arch/arm/mach-exynos/regs-pmu.h
>> b/arch/arm/mach-exynos/regs-pmu.h
>> index cfbfc575..53eac2b 100644
>> --- a/arch/arm/mach-exynos/regs-pmu.h
>> +++ b/arch/arm/mach-exynos/regs-pmu.h
>> @@ -39,6 +39,7 @@
>>   #define S5P_INFORM6                           S5P_PMUREG(0x0818)
>>   #define S5P_INFORM7                           S5P_PMUREG(0x081C)
>>
>> +#define S5P_PMU_SPARE3                         S5P_PMUREG(0x090C)
>>   #define EXYNOS_IROM_DATA2                     S5P_PMUREG(0x0988)
>>
>>   #define S5P_ARM_CORE0_LOWPWR                  S5P_PMUREG(0x1000)
>> @@ -113,7 +114,6 @@
>>   #define EXYNOS5_ARM_L2_OPTION                 S5P_PMUREG(0x2608)
>>   #define EXYNOS5_USE_RETENTION                 BIT(4)
>>
>> -
>>   #define S5P_PAD_RET_MAUDIO_OPTION             S5P_PMUREG(0x3028)
>>   #define S5P_PAD_RET_GPIO_OPTION                       S5P_PMUREG(0x3108)
>>   #define S5P_PAD_RET_UART_OPTION                       S5P_PMUREG(0x3128)
>> @@ -216,6 +216,8 @@
>>
>>   /* For EXYNOS5 */
>>
>> +#define EXYNOS5_SYS_DISP1_BLK_CFG                      S5P_SYSREG(0x0214)
>> +
>>   #define EXYNOS5_SYS_I2C_CFG
>> S5P_SYSREG(0x0234)
>>
>>   #define EXYNOS5_AUTO_WDTRESET_DISABLE
>> S5P_PMUREG(0x0408)
>> diff --git a/arch/arm/plat-samsung/include/plat/map-s5p.h
>> b/arch/arm/plat-samsung/include/plat/map-s5p.h
>> index c186786..cfb150e 100644
>> --- a/arch/arm/plat-samsung/include/plat/map-s5p.h
>> +++ b/arch/arm/plat-samsung/include/plat/map-s5p.h
>> @@ -57,6 +57,8 @@
>>   #define S3C_UART_OFFSET               (0x400)
>>   #endif
>>
>> +#define EXYNOS5420_VA_CPU_STATE        (S5P_VA_SYSRAM + 0x28)
>> +
>
>
> Please, no more static addresses. This should be defined as an offset to
> SYSRAM in the source file that actually uses it.

OK.

>
>
>>   #include <plat/map-s3c.h>
>>
>>   #endif /* __ASM_PLAT_MAP_S5P_H */
>> diff --git a/drivers/clk/samsung/clk-exynos5420.c
>> b/drivers/clk/samsung/clk-exynos5420.c
>> index 60b2681..965ea69 100644
>> --- a/drivers/clk/samsung/clk-exynos5420.c
>> +++ b/drivers/clk/samsung/clk-exynos5420.c
>> @@ -101,6 +101,19 @@
>>   #define KPLL_CON0             0x28100
>>   #define SRC_KFC                       0x28200
>>   #define DIV_KFC0              0x28500
>> +#define SRC_MASK_CPERI         0x04300
>
>
> Please keep the registers sorted...

OK.

>
>
>> +#define SRC_MASK_TOP0          0x10300
>> +#define SRC_MASK_TOP1          0x10304
>> +#define SRC_MASK_TOP2          0x10308
>> +#define SRC_MASK_TOP7          0x1031C
>> +#define SRC_MASK_MAU           0x10334
>> +#define SRC_MASK_FSYS          0x10340
>> +#define SRC_MASK_PERIC0                0x10350
>> +#define SRC_MASK_PERIC1                0x10354
>> +#define SRC_MASK_ISP           0x10370
>> +#define GATE_BUS_DISP1         0x10728
>> +#define GATE_IP_PERIC          0x10950
>> +
>>
>>   /* list of PLLs */
>>   enum exynos5420_plls {
>> @@ -180,11 +193,30 @@ static unsigned long exynos5420_clk_regs[]
>> __initdata = {
>>         DIV_KFC0,
>>   };
>>
>> +static const struct samsung_clk_reg_dump exynos5420_set_clksrc[] = {
>> +       { .offset = SRC_MASK_CPERI,             .value = 0xffffffff, },
>> +       { .offset = SRC_MASK_TOP0,              .value = 0x11111111, },
>> +       { .offset = SRC_MASK_TOP1,              .value = 0x11101111, },
>> +       { .offset = SRC_MASK_TOP2,              .value = 0x11111110, },
>> +       { .offset = SRC_MASK_TOP7,              .value = 0x00111100, },
>> +       { .offset = SRC_MASK_DISP10,            .value = 0x11111110, },
>> +       { .offset = SRC_MASK_MAU,               .value = 0x10000000, },
>> +       { .offset = SRC_MASK_FSYS,              .value = 0x11111110, },
>> +       { .offset = SRC_MASK_PERIC0,            .value = 0x11111110, },
>> +       { .offset = SRC_MASK_PERIC1,            .value = 0x11111100, },
>> +       { .offset = SRC_MASK_ISP,               .value = 0x11111000, },
>> +       { .offset = GATE_BUS_DISP1,             .value = 0xffffffff, },
>> +       { .offset = GATE_IP_PERIC,              .value = 0xffffffff, },
>> +};
>> +
>> +
>>   static int exynos5420_clk_suspend(void)
>>   {
>>         samsung_clk_save(reg_base, exynos5420_save,
>>                                 ARRAY_SIZE(exynos5420_clk_regs));
>>
>> +       samsung_clk_restore(reg_base, exynos5420_set_clksrc,
>> +
>> ARRAY_SIZE(exynos5420_set_clksrc));
>
>
> nit: You could align ARRAY_SIZE to the same indentation level as the call to
> samsung_clk_save() above.

OK.

>
> Best regards,
> Tomasz

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

end of thread, other threads:[~2014-04-28 12:39 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-27  6:13 [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
2014-03-27  6:13 ` [PATCH V2 1/3] ARM: EXYNOS5: Add PMU settings " Vikas Sajjan
2014-04-15 18:57   ` Tomasz Figa
2014-04-28 12:36     ` Vikas Sajjan
2014-03-27  6:13 ` [PATCH V2 2/3] ARM: EXYNOS5: Support Suspend-to-RAM on EXYNOS5420 Vikas Sajjan
2014-04-15 19:03   ` Tomasz Figa
2014-04-15 19:04     ` Tomasz Figa
2014-04-28 12:39     ` Vikas Sajjan
2014-03-27  6:13 ` [PATCH V2 3/3] ARM: dts: Add node for GPIO keys on SMDK5420 Vikas Sajjan
2014-04-15 19:06   ` Tomasz Figa
2014-04-11 13:17 ` [PATCH V2 0/3] Add PMU and S2R support for exynos5420 Vikas Sajjan
2014-04-13 23:30   ` Tomasz Figa

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