* [PATCH 00/12] AT91 pm cleanup for 3.20
@ 2015-01-20 8:16 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
Hi Nicolas,
This patch set is based on Alexandre's patch set of AT91 cleanup for 3.20 #2.
It also includes the patches from Peter Rosin and Sylvain Rochet.
Removes CONFIG_AT91_SLOW_CLOCK config item to simply the PM config,
The suspend to standby mode uses the same sram function as the suspend to memory mode.
Remove some unused code.
Thank Alexandre for many advices.
Best Regards,
Wenyou Yang
Peter Rosin (1):
pm: at91: Workaround DDRSDRC self-refresh bug with LPDDR1 memories.
Sylvain Rochet (1):
pm: at91: pm_slowclock: improve reliability of suspend/resume
Wenyou Yang (10):
pm: at91: pm_slowclock: remove the unused code related with
SLOWDOWN_MASTER_CLOCK
pm: at91: move the copying the sram function to the sram
initializationi phase
ARM: at91: move select SRAM to ARCH_AT91
pm: at91: remove the config item CONFIG_AT91_SLOW_CLOCK
pm: at91: the standby mode uses the same sram function as the suspend
to memory mode
pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
pm: at91: rename function name:
at91_slow_clock()-->at91_pm_suspend_sram_fn
pm: at91: remove the at91_xxx_standby() function definitions in the
pm.h
pm: at91: remove the struct ramc_ids .data at91_xxx_standby members
pm: at91: amend the pm_suspend entry for at91_cpuidle_device
arch/arm/mach-at91/Kconfig | 15 +-
arch/arm/mach-at91/Makefile | 2 +-
arch/arm/mach-at91/pm.c | 156 ++++++++---------
arch/arm/mach-at91/pm.h | 112 +------------
arch/arm/mach-at91/pm_slowclock.S | 322 ------------------------------------
arch/arm/mach-at91/pm_suspend.S | 278 +++++++++++++++++++++++++++++++
include/soc/at91/at91sam9_ddrsdr.h | 2 +-
7 files changed, 356 insertions(+), 531 deletions(-)
delete mode 100644 arch/arm/mach-at91/pm_slowclock.S
create mode 100644 arch/arm/mach-at91/pm_suspend.S
--
1.7.9.5
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 00/12] AT91 pm cleanup for 3.20
@ 2015-01-20 8:16 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: linux-arm-kernel
Hi Nicolas,
This patch set is based on Alexandre's patch set of AT91 cleanup for 3.20 #2.
It also includes the patches from Peter Rosin and Sylvain Rochet.
Removes CONFIG_AT91_SLOW_CLOCK config item to simply the PM config,
The suspend to standby mode uses the same sram function as the suspend to memory mode.
Remove some unused code.
Thank Alexandre for many advices.
Best Regards,
Wenyou Yang
Peter Rosin (1):
pm: at91: Workaround DDRSDRC self-refresh bug with LPDDR1 memories.
Sylvain Rochet (1):
pm: at91: pm_slowclock: improve reliability of suspend/resume
Wenyou Yang (10):
pm: at91: pm_slowclock: remove the unused code related with
SLOWDOWN_MASTER_CLOCK
pm: at91: move the copying the sram function to the sram
initializationi phase
ARM: at91: move select SRAM to ARCH_AT91
pm: at91: remove the config item CONFIG_AT91_SLOW_CLOCK
pm: at91: the standby mode uses the same sram function as the suspend
to memory mode
pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
pm: at91: rename function name:
at91_slow_clock()-->at91_pm_suspend_sram_fn
pm: at91: remove the at91_xxx_standby() function definitions in the
pm.h
pm: at91: remove the struct ramc_ids .data at91_xxx_standby members
pm: at91: amend the pm_suspend entry for at91_cpuidle_device
arch/arm/mach-at91/Kconfig | 15 +-
arch/arm/mach-at91/Makefile | 2 +-
arch/arm/mach-at91/pm.c | 156 ++++++++---------
arch/arm/mach-at91/pm.h | 112 +------------
arch/arm/mach-at91/pm_slowclock.S | 322 ------------------------------------
arch/arm/mach-at91/pm_suspend.S | 278 +++++++++++++++++++++++++++++++
include/soc/at91/at91sam9_ddrsdr.h | 2 +-
7 files changed, 356 insertions(+), 531 deletions(-)
delete mode 100644 arch/arm/mach-at91/pm_slowclock.S
create mode 100644 arch/arm/mach-at91/pm_suspend.S
--
1.7.9.5
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 01/12] pm: at91: pm_slowclock: improve reliability of suspend/resume
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:16 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
From: Sylvain Rochet <sylvain.rochet@finsecur.com>
Assume USB PLL and PLL B are already stopped before entering sleep mode,
print a warning if this isn't the case.
Removed timeout on XTAL, PLL lock and Master Clock Ready, hang if
something went wrong instead of continuing in unknown condition. There
is not much we can do if a PLL lock never ends, we are running in SRAM
and we will not be able to connect back the sdram or ddram in order to
be able to fire up a message or just panic.
As a bonus, not decounting the timeout register in slow clock mode
reduce cumulated suspend time and resume time from ~17ms to ~15ms.
Signed-off-by: Sylvain Rochet <sylvain.rochet@finsecur.com>
---
arch/arm/mach-at91/pm.c | 12 +++++++
arch/arm/mach-at91/pm_slowclock.S | 62 ++-----------------------------------
2 files changed, 15 insertions(+), 59 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index a4eb290..0990486 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -106,6 +106,18 @@ static int at91_pm_verify_clocks(void)
}
}
+ /* Drivers should have previously suspended USB PLL */
+ if (at91_pmc_read(AT91_CKGR_UCKR) & AT91_PMC_UPLLEN) {
+ pr_err("AT91: PM - Suspend-to-RAM with USB PLL running\n");
+ return 0;
+ }
+
+ /* Drivers should have previously suspended PLL B */
+ if (at91_pmc_read(AT91_PMC_SR) & AT91_PMC_LOCKB) {
+ pr_err("AT91: PM - Suspend-to-RAM with PLL B running\n");
+ return 0;
+ }
+
return 1;
}
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
index 160ba0d..3d497c7 100644
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ b/arch/arm/mach-at91/pm_slowclock.S
@@ -24,11 +24,6 @@
*/
#undef SLOWDOWN_MASTER_CLOCK
-#define MCKRDY_TIMEOUT 1000
-#define MOSCRDY_TIMEOUT 1000
-#define PLLALOCK_TIMEOUT 1000
-#define PLLBLOCK_TIMEOUT 1000
-
pmc .req r0
sdramc .req r1
ramc1 .req r2
@@ -40,56 +35,27 @@ tmp2 .req r5
* Wait until master clock is ready (after switching master clock source)
*/
.macro wait_mckrdy
- mov tmp2, #MCKRDY_TIMEOUT
-1: sub tmp2, tmp2, #1
- cmp tmp2, #0
- beq 2f
- ldr tmp1, [pmc, #AT91_PMC_SR]
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
tst tmp1, #AT91_PMC_MCKRDY
beq 1b
-2:
.endm
/*
* Wait until master oscillator has stabilized.
*/
.macro wait_moscrdy
- mov tmp2, #MOSCRDY_TIMEOUT
-1: sub tmp2, tmp2, #1
- cmp tmp2, #0
- beq 2f
- ldr tmp1, [pmc, #AT91_PMC_SR]
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
tst tmp1, #AT91_PMC_MOSCS
beq 1b
-2:
.endm
/*
* Wait until PLLA has locked.
*/
.macro wait_pllalock
- mov tmp2, #PLLALOCK_TIMEOUT
-1: sub tmp2, tmp2, #1
- cmp tmp2, #0
- beq 2f
- ldr tmp1, [pmc, #AT91_PMC_SR]
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
tst tmp1, #AT91_PMC_LOCKA
beq 1b
-2:
- .endm
-
-/*
- * Wait until PLLB has locked.
- */
- .macro wait_pllblock
- mov tmp2, #PLLBLOCK_TIMEOUT
-1: sub tmp2, tmp2, #1
- cmp tmp2, #0
- beq 2f
- ldr tmp1, [pmc, #AT91_PMC_SR]
- tst tmp1, #AT91_PMC_LOCKB
- beq 1b
-2:
.endm
.text
@@ -197,13 +163,6 @@ sdr_sr_done:
orr tmp1, tmp1, #(1 << 29) /* bit 29 always set */
str tmp1, [pmc, #AT91_CKGR_PLLAR]
- /* Save PLLB setting and disable it */
- ldr tmp1, [pmc, #AT91_CKGR_PLLBR]
- str tmp1, .saved_pllbr
-
- mov tmp1, #AT91_PMC_PLLCOUNT
- str tmp1, [pmc, #AT91_CKGR_PLLBR]
-
/* Turn off the main oscillator */
ldr tmp1, [pmc, #AT91_CKGR_MOR]
bic tmp1, tmp1, #AT91_PMC_MOSCEN
@@ -219,18 +178,6 @@ sdr_sr_done:
wait_moscrdy
- /* Restore PLLB setting */
- ldr tmp1, .saved_pllbr
- str tmp1, [pmc, #AT91_CKGR_PLLBR]
-
- tst tmp1, #(AT91_PMC_MUL & 0xff0000)
- bne 1f
- tst tmp1, #(AT91_PMC_MUL & ~0xff0000)
- beq 2f
-1:
- wait_pllblock
-2:
-
/* Restore PLLA setting */
ldr tmp1, .saved_pllar
str tmp1, [pmc, #AT91_CKGR_PLLAR]
@@ -309,9 +256,6 @@ ram_restored:
.saved_pllar:
.word 0
-.saved_pllbr:
- .word 0
-
.saved_sam9_lpr:
.word 0
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 01/12] pm: at91: pm_slowclock: improve reliability of suspend/resume
@ 2015-01-20 8:16 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: linux-arm-kernel
From: Sylvain Rochet <sylvain.rochet@finsecur.com>
Assume USB PLL and PLL B are already stopped before entering sleep mode,
print a warning if this isn't the case.
Removed timeout on XTAL, PLL lock and Master Clock Ready, hang if
something went wrong instead of continuing in unknown condition. There
is not much we can do if a PLL lock never ends, we are running in SRAM
and we will not be able to connect back the sdram or ddram in order to
be able to fire up a message or just panic.
As a bonus, not decounting the timeout register in slow clock mode
reduce cumulated suspend time and resume time from ~17ms to ~15ms.
Signed-off-by: Sylvain Rochet <sylvain.rochet@finsecur.com>
---
arch/arm/mach-at91/pm.c | 12 +++++++
arch/arm/mach-at91/pm_slowclock.S | 62 ++-----------------------------------
2 files changed, 15 insertions(+), 59 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index a4eb290..0990486 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -106,6 +106,18 @@ static int at91_pm_verify_clocks(void)
}
}
+ /* Drivers should have previously suspended USB PLL */
+ if (at91_pmc_read(AT91_CKGR_UCKR) & AT91_PMC_UPLLEN) {
+ pr_err("AT91: PM - Suspend-to-RAM with USB PLL running\n");
+ return 0;
+ }
+
+ /* Drivers should have previously suspended PLL B */
+ if (at91_pmc_read(AT91_PMC_SR) & AT91_PMC_LOCKB) {
+ pr_err("AT91: PM - Suspend-to-RAM with PLL B running\n");
+ return 0;
+ }
+
return 1;
}
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
index 160ba0d..3d497c7 100644
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ b/arch/arm/mach-at91/pm_slowclock.S
@@ -24,11 +24,6 @@
*/
#undef SLOWDOWN_MASTER_CLOCK
-#define MCKRDY_TIMEOUT 1000
-#define MOSCRDY_TIMEOUT 1000
-#define PLLALOCK_TIMEOUT 1000
-#define PLLBLOCK_TIMEOUT 1000
-
pmc .req r0
sdramc .req r1
ramc1 .req r2
@@ -40,56 +35,27 @@ tmp2 .req r5
* Wait until master clock is ready (after switching master clock source)
*/
.macro wait_mckrdy
- mov tmp2, #MCKRDY_TIMEOUT
-1: sub tmp2, tmp2, #1
- cmp tmp2, #0
- beq 2f
- ldr tmp1, [pmc, #AT91_PMC_SR]
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
tst tmp1, #AT91_PMC_MCKRDY
beq 1b
-2:
.endm
/*
* Wait until master oscillator has stabilized.
*/
.macro wait_moscrdy
- mov tmp2, #MOSCRDY_TIMEOUT
-1: sub tmp2, tmp2, #1
- cmp tmp2, #0
- beq 2f
- ldr tmp1, [pmc, #AT91_PMC_SR]
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
tst tmp1, #AT91_PMC_MOSCS
beq 1b
-2:
.endm
/*
* Wait until PLLA has locked.
*/
.macro wait_pllalock
- mov tmp2, #PLLALOCK_TIMEOUT
-1: sub tmp2, tmp2, #1
- cmp tmp2, #0
- beq 2f
- ldr tmp1, [pmc, #AT91_PMC_SR]
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
tst tmp1, #AT91_PMC_LOCKA
beq 1b
-2:
- .endm
-
-/*
- * Wait until PLLB has locked.
- */
- .macro wait_pllblock
- mov tmp2, #PLLBLOCK_TIMEOUT
-1: sub tmp2, tmp2, #1
- cmp tmp2, #0
- beq 2f
- ldr tmp1, [pmc, #AT91_PMC_SR]
- tst tmp1, #AT91_PMC_LOCKB
- beq 1b
-2:
.endm
.text
@@ -197,13 +163,6 @@ sdr_sr_done:
orr tmp1, tmp1, #(1 << 29) /* bit 29 always set */
str tmp1, [pmc, #AT91_CKGR_PLLAR]
- /* Save PLLB setting and disable it */
- ldr tmp1, [pmc, #AT91_CKGR_PLLBR]
- str tmp1, .saved_pllbr
-
- mov tmp1, #AT91_PMC_PLLCOUNT
- str tmp1, [pmc, #AT91_CKGR_PLLBR]
-
/* Turn off the main oscillator */
ldr tmp1, [pmc, #AT91_CKGR_MOR]
bic tmp1, tmp1, #AT91_PMC_MOSCEN
@@ -219,18 +178,6 @@ sdr_sr_done:
wait_moscrdy
- /* Restore PLLB setting */
- ldr tmp1, .saved_pllbr
- str tmp1, [pmc, #AT91_CKGR_PLLBR]
-
- tst tmp1, #(AT91_PMC_MUL & 0xff0000)
- bne 1f
- tst tmp1, #(AT91_PMC_MUL & ~0xff0000)
- beq 2f
-1:
- wait_pllblock
-2:
-
/* Restore PLLA setting */
ldr tmp1, .saved_pllar
str tmp1, [pmc, #AT91_CKGR_PLLAR]
@@ -309,9 +256,6 @@ ram_restored:
.saved_pllar:
.word 0
-.saved_pllbr:
- .word 0
-
.saved_sam9_lpr:
.word 0
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 02/12] pm: at91: Workaround DDRSDRC self-refresh bug with LPDDR1 memories.
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:16 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
From: Peter Rosin <peda@axentia.se>
The DDRSDR controller fails miserably to put LPDDR1 memories in
self-refresh. Force the controller to think it has DDR2 memories
during the self-refresh period, as the DDR2 self-refresh spec is
equivalent to LPDDR1, and is correctly implemented in the
controller.
Assume that the second controller has the same fault, but that is
untested.
Signed-off-by: Peter Rosin <peda@axentia.se>
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
---
arch/arm/mach-at91/pm_slowclock.S | 43 +++++++++++++++++++++++++++++++-----
include/soc/at91/at91sam9_ddrsdr.h | 2 +-
2 files changed, 39 insertions(+), 6 deletions(-)
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
index 3d497c7..5859f42 100644
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ b/arch/arm/mach-at91/pm_slowclock.S
@@ -99,6 +99,16 @@ ddr_sr_enable:
cmp memctrl, #AT91_MEMCTRL_DDRSDR
bne sdr_sr_enable
+ /* LPDDR1 --> force DDR2 mode during self-refresh */
+ ldr tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ str tmp1, .saved_sam9_mdr
+ bic tmp1, tmp1, #~AT91_DDRSDRC_MD
+ cmp tmp1, #AT91_DDRSDRC_MD_LOW_POWER_DDR
+ ldreq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ biceq tmp1, tmp1, #AT91_DDRSDRC_MD
+ orreq tmp1, tmp1, #AT91_DDRSDRC_MD_DDR2
+ streq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+
/* prepare for DDRAM self-refresh mode */
ldr tmp1, [sdramc, #AT91_DDRSDRC_LPR]
str tmp1, .saved_sam9_lpr
@@ -107,14 +117,26 @@ ddr_sr_enable:
/* figure out if we use the second ram controller */
cmp ramc1, #0
- ldrne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
- strne tmp2, .saved_sam9_lpr1
- bicne tmp2, #AT91_DDRSDRC_LPCB
- orrne tmp2, #AT91_DDRSDRC_LPCB_SELF_REFRESH
+ beq ddr_no_2nd_ctrl
+
+ ldr tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ str tmp2, .saved_sam9_mdr1
+ bic tmp2, tmp2, #~AT91_DDRSDRC_MD
+ cmp tmp2, #AT91_DDRSDRC_MD_LOW_POWER_DDR
+ ldreq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ biceq tmp2, tmp2, #AT91_DDRSDRC_MD
+ orreq tmp2, tmp2, #AT91_DDRSDRC_MD_DDR2
+ streq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+
+ ldr tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+ str tmp2, .saved_sam9_lpr1
+ bic tmp2, #AT91_DDRSDRC_LPCB
+ orr tmp2, #AT91_DDRSDRC_LPCB_SELF_REFRESH
/* Enable DDRAM self-refresh mode */
+ str tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+ddr_no_2nd_ctrl:
str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
- strne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
b sdr_sr_done
@@ -226,12 +248,17 @@ sdr_sr_done:
*/
cmp memctrl, #AT91_MEMCTRL_DDRSDR
bne sdr_en_restore
+ /* Restore MDR in case of LPDDR1 */
+ ldr tmp1, .saved_sam9_mdr
+ str tmp1, [sdramc, #AT91_DDRSDRC_MDR]
/* Restore LPR on AT91 with DDRAM */
ldr tmp1, .saved_sam9_lpr
str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
/* if we use the second ram controller */
cmp ramc1, #0
+ ldrne tmp2, .saved_sam9_mdr1
+ strne tmp2, [ramc1, #AT91_DDRSDRC_MDR]
ldrne tmp2, .saved_sam9_lpr1
strne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
@@ -262,5 +289,11 @@ ram_restored:
.saved_sam9_lpr1:
.word 0
+.saved_sam9_mdr:
+ .word 0
+
+.saved_sam9_mdr1:
+ .word 0
+
ENTRY(at91_slow_clock_sz)
.word .-at91_slow_clock
diff --git a/include/soc/at91/at91sam9_ddrsdr.h b/include/soc/at91/at91sam9_ddrsdr.h
index 0210797..dc10c52 100644
--- a/include/soc/at91/at91sam9_ddrsdr.h
+++ b/include/soc/at91/at91sam9_ddrsdr.h
@@ -92,7 +92,7 @@
#define AT91_DDRSDRC_UPD_MR (3 << 20) /* Update load mode register and extended mode register */
#define AT91_DDRSDRC_MDR 0x20 /* Memory Device Register */
-#define AT91_DDRSDRC_MD (3 << 0) /* Memory Device Type */
+#define AT91_DDRSDRC_MD (7 << 0) /* Memory Device Type */
#define AT91_DDRSDRC_MD_SDR 0
#define AT91_DDRSDRC_MD_LOW_POWER_SDR 1
#define AT91_DDRSDRC_MD_LOW_POWER_DDR 3
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 02/12] pm: at91: Workaround DDRSDRC self-refresh bug with LPDDR1 memories.
@ 2015-01-20 8:16 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: linux-arm-kernel
From: Peter Rosin <peda@axentia.se>
The DDRSDR controller fails miserably to put LPDDR1 memories in
self-refresh. Force the controller to think it has DDR2 memories
during the self-refresh period, as the DDR2 self-refresh spec is
equivalent to LPDDR1, and is correctly implemented in the
controller.
Assume that the second controller has the same fault, but that is
untested.
Signed-off-by: Peter Rosin <peda@axentia.se>
Acked-by: Nicolas Ferre <nicolas.ferre@atmel.com>
---
arch/arm/mach-at91/pm_slowclock.S | 43 +++++++++++++++++++++++++++++++-----
include/soc/at91/at91sam9_ddrsdr.h | 2 +-
2 files changed, 39 insertions(+), 6 deletions(-)
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
index 3d497c7..5859f42 100644
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ b/arch/arm/mach-at91/pm_slowclock.S
@@ -99,6 +99,16 @@ ddr_sr_enable:
cmp memctrl, #AT91_MEMCTRL_DDRSDR
bne sdr_sr_enable
+ /* LPDDR1 --> force DDR2 mode during self-refresh */
+ ldr tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ str tmp1, .saved_sam9_mdr
+ bic tmp1, tmp1, #~AT91_DDRSDRC_MD
+ cmp tmp1, #AT91_DDRSDRC_MD_LOW_POWER_DDR
+ ldreq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ biceq tmp1, tmp1, #AT91_DDRSDRC_MD
+ orreq tmp1, tmp1, #AT91_DDRSDRC_MD_DDR2
+ streq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+
/* prepare for DDRAM self-refresh mode */
ldr tmp1, [sdramc, #AT91_DDRSDRC_LPR]
str tmp1, .saved_sam9_lpr
@@ -107,14 +117,26 @@ ddr_sr_enable:
/* figure out if we use the second ram controller */
cmp ramc1, #0
- ldrne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
- strne tmp2, .saved_sam9_lpr1
- bicne tmp2, #AT91_DDRSDRC_LPCB
- orrne tmp2, #AT91_DDRSDRC_LPCB_SELF_REFRESH
+ beq ddr_no_2nd_ctrl
+
+ ldr tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ str tmp2, .saved_sam9_mdr1
+ bic tmp2, tmp2, #~AT91_DDRSDRC_MD
+ cmp tmp2, #AT91_DDRSDRC_MD_LOW_POWER_DDR
+ ldreq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ biceq tmp2, tmp2, #AT91_DDRSDRC_MD
+ orreq tmp2, tmp2, #AT91_DDRSDRC_MD_DDR2
+ streq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+
+ ldr tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+ str tmp2, .saved_sam9_lpr1
+ bic tmp2, #AT91_DDRSDRC_LPCB
+ orr tmp2, #AT91_DDRSDRC_LPCB_SELF_REFRESH
/* Enable DDRAM self-refresh mode */
+ str tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+ddr_no_2nd_ctrl:
str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
- strne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
b sdr_sr_done
@@ -226,12 +248,17 @@ sdr_sr_done:
*/
cmp memctrl, #AT91_MEMCTRL_DDRSDR
bne sdr_en_restore
+ /* Restore MDR in case of LPDDR1 */
+ ldr tmp1, .saved_sam9_mdr
+ str tmp1, [sdramc, #AT91_DDRSDRC_MDR]
/* Restore LPR on AT91 with DDRAM */
ldr tmp1, .saved_sam9_lpr
str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
/* if we use the second ram controller */
cmp ramc1, #0
+ ldrne tmp2, .saved_sam9_mdr1
+ strne tmp2, [ramc1, #AT91_DDRSDRC_MDR]
ldrne tmp2, .saved_sam9_lpr1
strne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
@@ -262,5 +289,11 @@ ram_restored:
.saved_sam9_lpr1:
.word 0
+.saved_sam9_mdr:
+ .word 0
+
+.saved_sam9_mdr1:
+ .word 0
+
ENTRY(at91_slow_clock_sz)
.word .-at91_slow_clock
diff --git a/include/soc/at91/at91sam9_ddrsdr.h b/include/soc/at91/at91sam9_ddrsdr.h
index 0210797..dc10c52 100644
--- a/include/soc/at91/at91sam9_ddrsdr.h
+++ b/include/soc/at91/at91sam9_ddrsdr.h
@@ -92,7 +92,7 @@
#define AT91_DDRSDRC_UPD_MR (3 << 20) /* Update load mode register and extended mode register */
#define AT91_DDRSDRC_MDR 0x20 /* Memory Device Register */
-#define AT91_DDRSDRC_MD (3 << 0) /* Memory Device Type */
+#define AT91_DDRSDRC_MD (7 << 0) /* Memory Device Type */
#define AT91_DDRSDRC_MD_SDR 0
#define AT91_DDRSDRC_MD_LOW_POWER_SDR 1
#define AT91_DDRSDRC_MD_LOW_POWER_DDR 3
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 03/12] pm: at91: pm_slowclock: remove the unused code related with SLOWDOWN_MASTER_CLOCK
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:16 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
The SLOWDOWN_MASTER_CLOCK definition is not used, remove the redundant code.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm_slowclock.S | 37 -------------------------------------
1 file changed, 37 deletions(-)
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
index 5859f42..634d819 100644
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ b/arch/arm/mach-at91/pm_slowclock.S
@@ -11,19 +11,10 @@
* published by the Free Software Foundation.
*
*/
-
#include <linux/linkage.h>
#include <linux/clk/at91_pmc.h>
#include <mach/at91_ramc.h>
-/*
- * When SLOWDOWN_MASTER_CLOCK is defined we will also slow down the Master
- * clock during suspend by adjusting its prescalar and divisor.
- * NOTE: This hasn't been shown to be stable on SAM9s; and on the RM9200 there
- * are errata regarding adjusting the prescalar and divisor.
- */
-#undef SLOWDOWN_MASTER_CLOCK
-
pmc .req r0
sdramc .req r1
ramc1 .req r2
@@ -165,18 +156,6 @@ sdr_sr_done:
wait_mckrdy
-#ifdef SLOWDOWN_MASTER_CLOCK
- /*
- * Set the Master Clock PRES and MDIV fields.
- *
- * See AT91RM9200 errata #27 and #28 for details.
- */
- mov tmp1, #0
- str tmp1, [pmc, #AT91_PMC_MCKR]
-
- wait_mckrdy
-#endif
-
/* Save PLLA setting and disable it */
ldr tmp1, [pmc, #AT91_CKGR_PLLAR]
str tmp1, .saved_pllar
@@ -212,22 +191,6 @@ sdr_sr_done:
wait_pllalock
4:
-#ifdef SLOWDOWN_MASTER_CLOCK
- /*
- * First set PRES if it was not 0,
- * than set CSS and MDIV fields.
- *
- * See AT91RM9200 errata #27 and #28 for details.
- */
- ldr tmp1, .saved_mckr
- tst tmp1, #AT91_PMC_PRES
- beq 2f
- and tmp1, tmp1, #AT91_PMC_PRES
- str tmp1, [pmc, #AT91_PMC_MCKR]
-
- wait_mckrdy
-#endif
-
/*
* Restore master clock setting
*/
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 03/12] pm: at91: pm_slowclock: remove the unused code related with SLOWDOWN_MASTER_CLOCK
@ 2015-01-20 8:16 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: linux-arm-kernel
The SLOWDOWN_MASTER_CLOCK definition is not used, remove the redundant code.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm_slowclock.S | 37 -------------------------------------
1 file changed, 37 deletions(-)
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
index 5859f42..634d819 100644
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ b/arch/arm/mach-at91/pm_slowclock.S
@@ -11,19 +11,10 @@
* published by the Free Software Foundation.
*
*/
-
#include <linux/linkage.h>
#include <linux/clk/at91_pmc.h>
#include <mach/at91_ramc.h>
-/*
- * When SLOWDOWN_MASTER_CLOCK is defined we will also slow down the Master
- * clock during suspend by adjusting its prescalar and divisor.
- * NOTE: This hasn't been shown to be stable on SAM9s; and on the RM9200 there
- * are errata regarding adjusting the prescalar and divisor.
- */
-#undef SLOWDOWN_MASTER_CLOCK
-
pmc .req r0
sdramc .req r1
ramc1 .req r2
@@ -165,18 +156,6 @@ sdr_sr_done:
wait_mckrdy
-#ifdef SLOWDOWN_MASTER_CLOCK
- /*
- * Set the Master Clock PRES and MDIV fields.
- *
- * See AT91RM9200 errata #27 and #28 for details.
- */
- mov tmp1, #0
- str tmp1, [pmc, #AT91_PMC_MCKR]
-
- wait_mckrdy
-#endif
-
/* Save PLLA setting and disable it */
ldr tmp1, [pmc, #AT91_CKGR_PLLAR]
str tmp1, .saved_pllar
@@ -212,22 +191,6 @@ sdr_sr_done:
wait_pllalock
4:
-#ifdef SLOWDOWN_MASTER_CLOCK
- /*
- * First set PRES if it was not 0,
- * than set CSS and MDIV fields.
- *
- * See AT91RM9200 errata #27 and #28 for details.
- */
- ldr tmp1, .saved_mckr
- tst tmp1, #AT91_PMC_PRES
- beq 2f
- and tmp1, tmp1, #AT91_PMC_PRES
- str tmp1, [pmc, #AT91_PMC_MCKR]
-
- wait_mckrdy
-#endif
-
/*
* Restore master clock setting
*/
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 04/12] pm: at91: move the copying the sram function to the sram initializationi phase
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:16 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
To decrease the suspend time, move the copying the sram function
to the sram initialization phase, instead of every time go to suspend.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 0990486..609a4f8 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -169,10 +169,6 @@ static int at91_pm_enter(suspend_state_t state)
* turning off the main oscillator; reverse on wakeup.
*/
if (slow_clock) {
-#ifdef CONFIG_AT91_SLOW_CLOCK
- /* copy slow_clock handler to SRAM, and call it */
- memcpy(slow_clock, at91_slow_clock, at91_slow_clock_sz);
-#endif
slow_clock(at91_pmc_base, at91_ramc_base[0],
at91_ramc_base[1],
at91_pm_data.memctrl);
@@ -280,6 +276,9 @@ static void __init at91_pm_sram_init(void)
sram_pbase = gen_pool_virt_to_phys(sram_pool, sram_base);
slow_clock = __arm_ioremap_exec(sram_pbase, at91_slow_clock_sz, false);
+ /* Copy slow_clock handler to SRAM, and call it */
+ memcpy(slow_clock, at91_slow_clock, at91_slow_clock_sz);
+
put_node:
of_node_put(node);
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 04/12] pm: at91: move the copying the sram function to the sram initializationi phase
@ 2015-01-20 8:16 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: linux-arm-kernel
To decrease the suspend time, move the copying the sram function
to the sram initialization phase, instead of every time go to suspend.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 0990486..609a4f8 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -169,10 +169,6 @@ static int at91_pm_enter(suspend_state_t state)
* turning off the main oscillator; reverse on wakeup.
*/
if (slow_clock) {
-#ifdef CONFIG_AT91_SLOW_CLOCK
- /* copy slow_clock handler to SRAM, and call it */
- memcpy(slow_clock, at91_slow_clock, at91_slow_clock_sz);
-#endif
slow_clock(at91_pmc_base, at91_ramc_base[0],
at91_ramc_base[1],
at91_pm_data.memctrl);
@@ -280,6 +276,9 @@ static void __init at91_pm_sram_init(void)
sram_pbase = gen_pool_virt_to_phys(sram_pool, sram_base);
slow_clock = __arm_ioremap_exec(sram_pbase, at91_slow_clock_sz, false);
+ /* Copy slow_clock handler to SRAM, and call it */
+ memcpy(slow_clock, at91_slow_clock, at91_slow_clock_sz);
+
put_node:
of_node_put(node);
}
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:16 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
Because the CONFIG_AT91_SLOW_CLOCK will be removed
to simply the PM config, so move select SRAM.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
---
arch/arm/mach-at91/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
index 56dcede..7d5e201 100644
--- a/arch/arm/mach-at91/Kconfig
+++ b/arch/arm/mach-at91/Kconfig
@@ -6,6 +6,7 @@ config ARCH_AT91
select PINCTRL
select PINCTRL_AT91
select USE_OF
+ select SRAM if PM
bool
menuconfig SOC_SAM_V7
@@ -128,7 +129,6 @@ comment "AT91 Feature Selections"
config AT91_SLOW_CLOCK
bool "Suspend-to-RAM disables main oscillator"
- select SRAM
depends on SUSPEND
help
Select this if you want Suspend-to-RAM to save the most power
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91
@ 2015-01-20 8:16 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: linux-arm-kernel
Because the CONFIG_AT91_SLOW_CLOCK will be removed
to simply the PM config, so move select SRAM.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
---
arch/arm/mach-at91/Kconfig | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
index 56dcede..7d5e201 100644
--- a/arch/arm/mach-at91/Kconfig
+++ b/arch/arm/mach-at91/Kconfig
@@ -6,6 +6,7 @@ config ARCH_AT91
select PINCTRL
select PINCTRL_AT91
select USE_OF
+ select SRAM if PM
bool
menuconfig SOC_SAM_V7
@@ -128,7 +129,6 @@ comment "AT91 Feature Selections"
config AT91_SLOW_CLOCK
bool "Suspend-to-RAM disables main oscillator"
- select SRAM
depends on SUSPEND
help
Select this if you want Suspend-to-RAM to save the most power
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 06/12] pm: at91: remove the config item CONFIG_AT91_SLOW_CLOCK
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:16 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
The slow clock always exists, for the suspend to memory mode,
the master clock always switch to the slow clock.
To simplify the PM config, remove this config item, remove
the definition code as well.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/Kconfig | 13 -------------
arch/arm/mach-at91/Makefile | 2 +-
arch/arm/mach-at91/pm.c | 7 +------
3 files changed, 2 insertions(+), 20 deletions(-)
diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
index 7d5e201..2aed68f 100644
--- a/arch/arm/mach-at91/Kconfig
+++ b/arch/arm/mach-at91/Kconfig
@@ -127,19 +127,6 @@ menu "Atmel AT91 common options"
comment "AT91 Feature Selections"
-config AT91_SLOW_CLOCK
- bool "Suspend-to-RAM disables main oscillator"
- depends on SUSPEND
- help
- Select this if you want Suspend-to-RAM to save the most power
- possible (without powering off the CPU) by disabling the PLLs
- and main oscillator so that only the 32 KiHz clock is available.
-
- When only that slow-clock is available, some peripherals lose
- functionality. Many can't issue wakeup events unless faster
- clocks are available. Some lose their operating state and
- need to be completely re-initialized.
-
config AT91_TIMER_HZ
int "Kernel HZ (jiffies per second)"
range 32 1024
diff --git a/arch/arm/mach-at91/Makefile b/arch/arm/mach-at91/Makefile
index fe30e7e..9c755d9 100644
--- a/arch/arm/mach-at91/Makefile
+++ b/arch/arm/mach-at91/Makefile
@@ -18,7 +18,7 @@ obj-$(CONFIG_SOC_SAMA5) += board-dt-sama5.o
# Power Management
obj-$(CONFIG_PM) += pm.o
-obj-$(CONFIG_AT91_SLOW_CLOCK) += pm_slowclock.o
+obj-$(CONFIG_PM) += pm_slowclock.o
ifeq ($(CONFIG_PM_DEBUG),y)
CFLAGS_pm.o += -DDEBUG
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 609a4f8..691e6db 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -141,11 +141,9 @@ EXPORT_SYMBOL(at91_suspend_entering_slow_clock);
static void (*slow_clock)(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-#ifdef CONFIG_AT91_SLOW_CLOCK
extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
extern u32 at91_slow_clock_sz;
-#endif
static int at91_pm_enter(suspend_state_t state)
{
@@ -240,7 +238,6 @@ void at91_pm_set_standby(void (*at91_standby)(void))
}
}
-#ifdef CONFIG_AT91_SLOW_CLOCK
static void __init at91_pm_sram_init(void)
{
struct gen_pool *sram_pool;
@@ -282,7 +279,6 @@ static void __init at91_pm_sram_init(void)
put_node:
of_node_put(node);
}
-#endif
static struct of_device_id ramc_ids[] = {
{ .compatible = "atmel,at91rm9200-sdramc", .data = at91rm9200_standby },
@@ -324,9 +320,8 @@ static void at91_dt_ramc(void)
static void __init at91_pm_init(void)
{
at91_dt_ramc();
-#ifdef CONFIG_AT91_SLOW_CLOCK
+
at91_pm_sram_init();
-#endif
pr_info("AT91: Power Management%s\n", (slow_clock ? " (with slow clock mode)" : ""));
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 06/12] pm: at91: remove the config item CONFIG_AT91_SLOW_CLOCK
@ 2015-01-20 8:16 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:16 UTC (permalink / raw)
To: linux-arm-kernel
The slow clock always exists, for the suspend to memory mode,
the master clock always switch to the slow clock.
To simplify the PM config, remove this config item, remove
the definition code as well.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/Kconfig | 13 -------------
arch/arm/mach-at91/Makefile | 2 +-
arch/arm/mach-at91/pm.c | 7 +------
3 files changed, 2 insertions(+), 20 deletions(-)
diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
index 7d5e201..2aed68f 100644
--- a/arch/arm/mach-at91/Kconfig
+++ b/arch/arm/mach-at91/Kconfig
@@ -127,19 +127,6 @@ menu "Atmel AT91 common options"
comment "AT91 Feature Selections"
-config AT91_SLOW_CLOCK
- bool "Suspend-to-RAM disables main oscillator"
- depends on SUSPEND
- help
- Select this if you want Suspend-to-RAM to save the most power
- possible (without powering off the CPU) by disabling the PLLs
- and main oscillator so that only the 32 KiHz clock is available.
-
- When only that slow-clock is available, some peripherals lose
- functionality. Many can't issue wakeup events unless faster
- clocks are available. Some lose their operating state and
- need to be completely re-initialized.
-
config AT91_TIMER_HZ
int "Kernel HZ (jiffies per second)"
range 32 1024
diff --git a/arch/arm/mach-at91/Makefile b/arch/arm/mach-at91/Makefile
index fe30e7e..9c755d9 100644
--- a/arch/arm/mach-at91/Makefile
+++ b/arch/arm/mach-at91/Makefile
@@ -18,7 +18,7 @@ obj-$(CONFIG_SOC_SAMA5) += board-dt-sama5.o
# Power Management
obj-$(CONFIG_PM) += pm.o
-obj-$(CONFIG_AT91_SLOW_CLOCK) += pm_slowclock.o
+obj-$(CONFIG_PM) += pm_slowclock.o
ifeq ($(CONFIG_PM_DEBUG),y)
CFLAGS_pm.o += -DDEBUG
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 609a4f8..691e6db 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -141,11 +141,9 @@ EXPORT_SYMBOL(at91_suspend_entering_slow_clock);
static void (*slow_clock)(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-#ifdef CONFIG_AT91_SLOW_CLOCK
extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
extern u32 at91_slow_clock_sz;
-#endif
static int at91_pm_enter(suspend_state_t state)
{
@@ -240,7 +238,6 @@ void at91_pm_set_standby(void (*at91_standby)(void))
}
}
-#ifdef CONFIG_AT91_SLOW_CLOCK
static void __init at91_pm_sram_init(void)
{
struct gen_pool *sram_pool;
@@ -282,7 +279,6 @@ static void __init at91_pm_sram_init(void)
put_node:
of_node_put(node);
}
-#endif
static struct of_device_id ramc_ids[] = {
{ .compatible = "atmel,at91rm9200-sdramc", .data = at91rm9200_standby },
@@ -324,9 +320,8 @@ static void at91_dt_ramc(void)
static void __init at91_pm_init(void)
{
at91_dt_ramc();
-#ifdef CONFIG_AT91_SLOW_CLOCK
+
at91_pm_sram_init();
-#endif
pr_info("AT91: Power Management%s\n", (slow_clock ? " (with slow clock mode)" : ""));
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:17 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:17 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
To simply the PM code, the suspend to standby mode uses the same sram function
as the suspend to memory mode, running in the internal SRAM,
instead of the respective code for each mode.
But for the suspend to standby mode, the master clock doesn't
switch to the slow clock, and the main oscillator doesn't
turn off as well.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
---
arch/arm/mach-at91/pm.c | 81 ++++++++++++++++---------------------
arch/arm/mach-at91/pm.h | 6 +++
arch/arm/mach-at91/pm_slowclock.S | 18 +++++++++
3 files changed, 59 insertions(+), 46 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 691e6db..a1010f0 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
extern u32 at91_slow_clock_sz;
+static void at91_pm_suspend(suspend_state_t state)
+{
+ unsigned int pm_data = at91_pm_data.memctrl;
+
+ pm_data |= (state == PM_SUSPEND_MEM) ?
+ (AT91_PM_SLOW_CLOCK << AT91_PM_MODE_OFFSET) : 0;
+
+ slow_clock(at91_pmc_base, at91_ramc_base[0],
+ at91_ramc_base[1], pm_data);
+}
+
static int at91_pm_enter(suspend_state_t state)
{
at91_pinctrl_gpio_suspend();
switch (state) {
+ /*
+ * Suspend-to-RAM is like STANDBY plus slow clock mode, so
+ * drivers must suspend more deeply, the master clock switches
+ * to the clk32k and turns off the main oscillator
+ *
+ * STANDBY mode has *all* drivers suspended; ignores irqs not
+ * marked as 'wakeup' event sources; and reduces DRAM power.
+ * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
+ * nothing fancy done with main or cpu clocks.
+ */
+ case PM_SUSPEND_MEM:
+ case PM_SUSPEND_STANDBY:
/*
- * Suspend-to-RAM is like STANDBY plus slow clock mode, so
- * drivers must suspend more deeply: only the master clock
- * controller may be using the main oscillator.
+ * Ensure that clocks are in a valid state.
*/
- case PM_SUSPEND_MEM:
- /*
- * Ensure that clocks are in a valid state.
- */
- if (!at91_pm_verify_clocks())
- goto error;
-
- /*
- * Enter slow clock mode by switching over to clk32k and
- * turning off the main oscillator; reverse on wakeup.
- */
- if (slow_clock) {
- slow_clock(at91_pmc_base, at91_ramc_base[0],
- at91_ramc_base[1],
- at91_pm_data.memctrl);
- break;
- } else {
- pr_info("AT91: PM - no slow clock mode enabled ...\n");
- /* FALLTHROUGH leaving master clock alone */
- }
+ if (!at91_pm_verify_clocks())
+ goto error;
- /*
- * STANDBY mode has *all* drivers suspended; ignores irqs not
- * marked as 'wakeup' event sources; and reduces DRAM power.
- * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
- * nothing fancy done with main or cpu clocks.
- */
- case PM_SUSPEND_STANDBY:
- /*
- * NOTE: the Wait-for-Interrupt instruction needs to be
- * in icache so no SDRAM accesses are needed until the
- * wakeup IRQ occurs and self-refresh is terminated.
- * For ARM 926 based chips, this requirement is weaker
- * as at91sam9 can access a RAM in self-refresh mode.
- */
- if (at91_pm_standby)
- at91_pm_standby();
- break;
+ at91_pm_suspend(state);
- case PM_SUSPEND_ON:
- cpu_do_idle();
- break;
+ break;
- default:
- pr_debug("AT91: PM - bogus suspend state %d\n", state);
- goto error;
+ case PM_SUSPEND_ON:
+ cpu_do_idle();
+ break;
+
+ default:
+ pr_debug("AT91: PM - bogus suspend state %d\n", state);
+ goto error;
}
error:
diff --git a/arch/arm/mach-at91/pm.h b/arch/arm/mach-at91/pm.h
index 7664d39..fbfea42 100644
--- a/arch/arm/mach-at91/pm.h
+++ b/arch/arm/mach-at91/pm.h
@@ -15,6 +15,12 @@
#include <mach/at91_ramc.h>
+#define AT91_PM_MEMCTRL_MASK 0x0f
+#define AT91_PM_MODE_OFFSET 4
+#define AT91_PM_MODE_MASK 0x0f
+
+#define AT91_PM_SLOW_CLOCK 0x01
+
/*
* On all at91 except rm9200 and x40 have the System Controller starts
* at address 0xffffc000 and has a size of 16KiB.
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
index 634d819..92d7e63 100644
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ b/arch/arm/mach-at91/pm_slowclock.S
@@ -15,12 +15,15 @@
#include <linux/clk/at91_pmc.h>
#include <mach/at91_ramc.h>
+#include "pm.h"
+
pmc .req r0
sdramc .req r1
ramc1 .req r2
memctrl .req r3
tmp1 .req r4
tmp2 .req r5
+mode .req r6
/*
* Wait until master clock is ready (after switching master clock source)
@@ -72,6 +75,13 @@ ENTRY(at91_slow_clock)
mov tmp1, #0
mcr p15, 0, tmp1, c7, c10, 4
+ mov tmp1, memctrl
+ mov tmp2, tmp1, lsr#AT91_PM_MODE_OFFSET
+ and mode, tmp2, #AT91_PM_MODE_MASK
+
+ mov tmp1, memctrl
+ and memctrl, tmp1, #AT91_PM_MEMCTRL_MASK
+
cmp memctrl, #AT91_MEMCTRL_MC
bne ddr_sr_enable
@@ -144,6 +154,9 @@ sdr_sr_enable:
str tmp1, [sdramc, #AT91_SDRAMC_LPR]
sdr_sr_done:
+ tst mode, #AT91_PM_SLOW_CLOCK
+ beq skip_disable_clock
+
/* Save Master clock setting */
ldr tmp1, [pmc, #AT91_PMC_MCKR]
str tmp1, .saved_mckr
@@ -169,9 +182,13 @@ sdr_sr_done:
bic tmp1, tmp1, #AT91_PMC_MOSCEN
str tmp1, [pmc, #AT91_CKGR_MOR]
+skip_disable_clock:
/* Wait for interrupt */
mcr p15, 0, tmp1, c7, c0, 4
+ tst mode, #AT91_PM_SLOW_CLOCK
+ beq skip_enable_clock
+
/* Turn on the main oscillator */
ldr tmp1, [pmc, #AT91_CKGR_MOR]
orr tmp1, tmp1, #AT91_PMC_MOSCEN
@@ -199,6 +216,7 @@ sdr_sr_done:
wait_mckrdy
+skip_enable_clock:
/*
* at91rm9200 Memory controller
* Do nothing - self-refresh is automatically disabled.
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-20 8:17 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:17 UTC (permalink / raw)
To: linux-arm-kernel
To simply the PM code, the suspend to standby mode uses the same sram function
as the suspend to memory mode, running in the internal SRAM,
instead of the respective code for each mode.
But for the suspend to standby mode, the master clock doesn't
switch to the slow clock, and the main oscillator doesn't
turn off as well.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
---
arch/arm/mach-at91/pm.c | 81 ++++++++++++++++---------------------
arch/arm/mach-at91/pm.h | 6 +++
arch/arm/mach-at91/pm_slowclock.S | 18 +++++++++
3 files changed, 59 insertions(+), 46 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 691e6db..a1010f0 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
extern u32 at91_slow_clock_sz;
+static void at91_pm_suspend(suspend_state_t state)
+{
+ unsigned int pm_data = at91_pm_data.memctrl;
+
+ pm_data |= (state == PM_SUSPEND_MEM) ?
+ (AT91_PM_SLOW_CLOCK << AT91_PM_MODE_OFFSET) : 0;
+
+ slow_clock(at91_pmc_base, at91_ramc_base[0],
+ at91_ramc_base[1], pm_data);
+}
+
static int at91_pm_enter(suspend_state_t state)
{
at91_pinctrl_gpio_suspend();
switch (state) {
+ /*
+ * Suspend-to-RAM is like STANDBY plus slow clock mode, so
+ * drivers must suspend more deeply, the master clock switches
+ * to the clk32k and turns off the main oscillator
+ *
+ * STANDBY mode has *all* drivers suspended; ignores irqs not
+ * marked as 'wakeup' event sources; and reduces DRAM power.
+ * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
+ * nothing fancy done with main or cpu clocks.
+ */
+ case PM_SUSPEND_MEM:
+ case PM_SUSPEND_STANDBY:
/*
- * Suspend-to-RAM is like STANDBY plus slow clock mode, so
- * drivers must suspend more deeply: only the master clock
- * controller may be using the main oscillator.
+ * Ensure that clocks are in a valid state.
*/
- case PM_SUSPEND_MEM:
- /*
- * Ensure that clocks are in a valid state.
- */
- if (!at91_pm_verify_clocks())
- goto error;
-
- /*
- * Enter slow clock mode by switching over to clk32k and
- * turning off the main oscillator; reverse on wakeup.
- */
- if (slow_clock) {
- slow_clock(at91_pmc_base, at91_ramc_base[0],
- at91_ramc_base[1],
- at91_pm_data.memctrl);
- break;
- } else {
- pr_info("AT91: PM - no slow clock mode enabled ...\n");
- /* FALLTHROUGH leaving master clock alone */
- }
+ if (!at91_pm_verify_clocks())
+ goto error;
- /*
- * STANDBY mode has *all* drivers suspended; ignores irqs not
- * marked as 'wakeup' event sources; and reduces DRAM power.
- * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
- * nothing fancy done with main or cpu clocks.
- */
- case PM_SUSPEND_STANDBY:
- /*
- * NOTE: the Wait-for-Interrupt instruction needs to be
- * in icache so no SDRAM accesses are needed until the
- * wakeup IRQ occurs and self-refresh is terminated.
- * For ARM 926 based chips, this requirement is weaker
- * as at91sam9 can access a RAM in self-refresh mode.
- */
- if (at91_pm_standby)
- at91_pm_standby();
- break;
+ at91_pm_suspend(state);
- case PM_SUSPEND_ON:
- cpu_do_idle();
- break;
+ break;
- default:
- pr_debug("AT91: PM - bogus suspend state %d\n", state);
- goto error;
+ case PM_SUSPEND_ON:
+ cpu_do_idle();
+ break;
+
+ default:
+ pr_debug("AT91: PM - bogus suspend state %d\n", state);
+ goto error;
}
error:
diff --git a/arch/arm/mach-at91/pm.h b/arch/arm/mach-at91/pm.h
index 7664d39..fbfea42 100644
--- a/arch/arm/mach-at91/pm.h
+++ b/arch/arm/mach-at91/pm.h
@@ -15,6 +15,12 @@
#include <mach/at91_ramc.h>
+#define AT91_PM_MEMCTRL_MASK 0x0f
+#define AT91_PM_MODE_OFFSET 4
+#define AT91_PM_MODE_MASK 0x0f
+
+#define AT91_PM_SLOW_CLOCK 0x01
+
/*
* On all at91 except rm9200 and x40 have the System Controller starts
* at address 0xffffc000 and has a size of 16KiB.
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
index 634d819..92d7e63 100644
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ b/arch/arm/mach-at91/pm_slowclock.S
@@ -15,12 +15,15 @@
#include <linux/clk/at91_pmc.h>
#include <mach/at91_ramc.h>
+#include "pm.h"
+
pmc .req r0
sdramc .req r1
ramc1 .req r2
memctrl .req r3
tmp1 .req r4
tmp2 .req r5
+mode .req r6
/*
* Wait until master clock is ready (after switching master clock source)
@@ -72,6 +75,13 @@ ENTRY(at91_slow_clock)
mov tmp1, #0
mcr p15, 0, tmp1, c7, c10, 4
+ mov tmp1, memctrl
+ mov tmp2, tmp1, lsr#AT91_PM_MODE_OFFSET
+ and mode, tmp2, #AT91_PM_MODE_MASK
+
+ mov tmp1, memctrl
+ and memctrl, tmp1, #AT91_PM_MEMCTRL_MASK
+
cmp memctrl, #AT91_MEMCTRL_MC
bne ddr_sr_enable
@@ -144,6 +154,9 @@ sdr_sr_enable:
str tmp1, [sdramc, #AT91_SDRAMC_LPR]
sdr_sr_done:
+ tst mode, #AT91_PM_SLOW_CLOCK
+ beq skip_disable_clock
+
/* Save Master clock setting */
ldr tmp1, [pmc, #AT91_PMC_MCKR]
str tmp1, .saved_mckr
@@ -169,9 +182,13 @@ sdr_sr_done:
bic tmp1, tmp1, #AT91_PMC_MOSCEN
str tmp1, [pmc, #AT91_CKGR_MOR]
+skip_disable_clock:
/* Wait for interrupt */
mcr p15, 0, tmp1, c7, c0, 4
+ tst mode, #AT91_PM_SLOW_CLOCK
+ beq skip_enable_clock
+
/* Turn on the main oscillator */
ldr tmp1, [pmc, #AT91_CKGR_MOR]
orr tmp1, tmp1, #AT91_PMC_MOSCEN
@@ -199,6 +216,7 @@ sdr_sr_done:
wait_mckrdy
+skip_enable_clock:
/*
* at91rm9200 Memory controller
* Do nothing - self-refresh is automatically disabled.
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:17 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:17 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
Because the sram function is used for the suspend to standby mode as well,
more than suspend to memory, so renaming is more elegant.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/Makefile | 2 +-
arch/arm/mach-at91/pm_slowclock.S | 280 -------------------------------------
arch/arm/mach-at91/pm_suspend.S | 280 +++++++++++++++++++++++++++++++++++++
3 files changed, 281 insertions(+), 281 deletions(-)
delete mode 100644 arch/arm/mach-at91/pm_slowclock.S
create mode 100644 arch/arm/mach-at91/pm_suspend.S
diff --git a/arch/arm/mach-at91/Makefile b/arch/arm/mach-at91/Makefile
index 9c755d9..80473d8 100644
--- a/arch/arm/mach-at91/Makefile
+++ b/arch/arm/mach-at91/Makefile
@@ -18,7 +18,7 @@ obj-$(CONFIG_SOC_SAMA5) += board-dt-sama5.o
# Power Management
obj-$(CONFIG_PM) += pm.o
-obj-$(CONFIG_PM) += pm_slowclock.o
+obj-$(CONFIG_PM) += pm_suspend.o
ifeq ($(CONFIG_PM_DEBUG),y)
CFLAGS_pm.o += -DDEBUG
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
deleted file mode 100644
index 92d7e63..0000000
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ /dev/null
@@ -1,280 +0,0 @@
-/*
- * arch/arm/mach-at91/pm_slow_clock.S
- *
- * Copyright (C) 2006 Savin Zlobec
- *
- * AT91SAM9 support:
- * Copyright (C) 2007 Anti Sullin <anti.sullin@artecdesign.ee
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- *
- */
-#include <linux/linkage.h>
-#include <linux/clk/at91_pmc.h>
-#include <mach/at91_ramc.h>
-
-#include "pm.h"
-
-pmc .req r0
-sdramc .req r1
-ramc1 .req r2
-memctrl .req r3
-tmp1 .req r4
-tmp2 .req r5
-mode .req r6
-
-/*
- * Wait until master clock is ready (after switching master clock source)
- */
- .macro wait_mckrdy
-1: ldr tmp1, [pmc, #AT91_PMC_SR]
- tst tmp1, #AT91_PMC_MCKRDY
- beq 1b
- .endm
-
-/*
- * Wait until master oscillator has stabilized.
- */
- .macro wait_moscrdy
-1: ldr tmp1, [pmc, #AT91_PMC_SR]
- tst tmp1, #AT91_PMC_MOSCS
- beq 1b
- .endm
-
-/*
- * Wait until PLLA has locked.
- */
- .macro wait_pllalock
-1: ldr tmp1, [pmc, #AT91_PMC_SR]
- tst tmp1, #AT91_PMC_LOCKA
- beq 1b
- .endm
-
- .text
-
-/* void at91_slow_clock(void __iomem *pmc, void __iomem *sdramc,
- * void __iomem *ramc1, int memctrl)
- */
-ENTRY(at91_slow_clock)
- /* Save registers on stack */
- stmfd sp!, {r4 - r12, lr}
-
- /*
- * Register usage:
- * R0 = Base address of AT91_PMC
- * R1 = Base address of RAM Controller (SDRAM, DDRSDR, or AT91_SYS)
- * R2 = Base address of second RAM Controller or 0 if not present
- * R3 = Memory controller
- * R4 = temporary register
- * R5 = temporary register
- */
-
- /* Drain write buffer */
- mov tmp1, #0
- mcr p15, 0, tmp1, c7, c10, 4
-
- mov tmp1, memctrl
- mov tmp2, tmp1, lsr#AT91_PM_MODE_OFFSET
- and mode, tmp2, #AT91_PM_MODE_MASK
-
- mov tmp1, memctrl
- and memctrl, tmp1, #AT91_PM_MEMCTRL_MASK
-
- cmp memctrl, #AT91_MEMCTRL_MC
- bne ddr_sr_enable
-
- /*
- * at91rm9200 Memory controller
- */
- /* Put SDRAM in self-refresh mode */
- mov tmp1, #1
- str tmp1, [sdramc, #AT91RM9200_SDRAMC_SRR]
- b sdr_sr_done
-
- /*
- * DDRSDR Memory controller
- */
-ddr_sr_enable:
- cmp memctrl, #AT91_MEMCTRL_DDRSDR
- bne sdr_sr_enable
-
- /* LPDDR1 --> force DDR2 mode during self-refresh */
- ldr tmp1, [sdramc, #AT91_DDRSDRC_MDR]
- str tmp1, .saved_sam9_mdr
- bic tmp1, tmp1, #~AT91_DDRSDRC_MD
- cmp tmp1, #AT91_DDRSDRC_MD_LOW_POWER_DDR
- ldreq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
- biceq tmp1, tmp1, #AT91_DDRSDRC_MD
- orreq tmp1, tmp1, #AT91_DDRSDRC_MD_DDR2
- streq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
-
- /* prepare for DDRAM self-refresh mode */
- ldr tmp1, [sdramc, #AT91_DDRSDRC_LPR]
- str tmp1, .saved_sam9_lpr
- bic tmp1, #AT91_DDRSDRC_LPCB
- orr tmp1, #AT91_DDRSDRC_LPCB_SELF_REFRESH
-
- /* figure out if we use the second ram controller */
- cmp ramc1, #0
- beq ddr_no_2nd_ctrl
-
- ldr tmp2, [ramc1, #AT91_DDRSDRC_MDR]
- str tmp2, .saved_sam9_mdr1
- bic tmp2, tmp2, #~AT91_DDRSDRC_MD
- cmp tmp2, #AT91_DDRSDRC_MD_LOW_POWER_DDR
- ldreq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
- biceq tmp2, tmp2, #AT91_DDRSDRC_MD
- orreq tmp2, tmp2, #AT91_DDRSDRC_MD_DDR2
- streq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
-
- ldr tmp2, [ramc1, #AT91_DDRSDRC_LPR]
- str tmp2, .saved_sam9_lpr1
- bic tmp2, #AT91_DDRSDRC_LPCB
- orr tmp2, #AT91_DDRSDRC_LPCB_SELF_REFRESH
-
- /* Enable DDRAM self-refresh mode */
- str tmp2, [ramc1, #AT91_DDRSDRC_LPR]
-ddr_no_2nd_ctrl:
- str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
-
- b sdr_sr_done
-
- /*
- * SDRAMC Memory controller
- */
-sdr_sr_enable:
- /* Enable SDRAM self-refresh mode */
- ldr tmp1, [sdramc, #AT91_SDRAMC_LPR]
- str tmp1, .saved_sam9_lpr
-
- bic tmp1, #AT91_SDRAMC_LPCB
- orr tmp1, #AT91_SDRAMC_LPCB_SELF_REFRESH
- str tmp1, [sdramc, #AT91_SDRAMC_LPR]
-
-sdr_sr_done:
- tst mode, #AT91_PM_SLOW_CLOCK
- beq skip_disable_clock
-
- /* Save Master clock setting */
- ldr tmp1, [pmc, #AT91_PMC_MCKR]
- str tmp1, .saved_mckr
-
- /*
- * Set the Master clock source to slow clock
- */
- bic tmp1, tmp1, #AT91_PMC_CSS
- str tmp1, [pmc, #AT91_PMC_MCKR]
-
- wait_mckrdy
-
- /* Save PLLA setting and disable it */
- ldr tmp1, [pmc, #AT91_CKGR_PLLAR]
- str tmp1, .saved_pllar
-
- mov tmp1, #AT91_PMC_PLLCOUNT
- orr tmp1, tmp1, #(1 << 29) /* bit 29 always set */
- str tmp1, [pmc, #AT91_CKGR_PLLAR]
-
- /* Turn off the main oscillator */
- ldr tmp1, [pmc, #AT91_CKGR_MOR]
- bic tmp1, tmp1, #AT91_PMC_MOSCEN
- str tmp1, [pmc, #AT91_CKGR_MOR]
-
-skip_disable_clock:
- /* Wait for interrupt */
- mcr p15, 0, tmp1, c7, c0, 4
-
- tst mode, #AT91_PM_SLOW_CLOCK
- beq skip_enable_clock
-
- /* Turn on the main oscillator */
- ldr tmp1, [pmc, #AT91_CKGR_MOR]
- orr tmp1, tmp1, #AT91_PMC_MOSCEN
- str tmp1, [pmc, #AT91_CKGR_MOR]
-
- wait_moscrdy
-
- /* Restore PLLA setting */
- ldr tmp1, .saved_pllar
- str tmp1, [pmc, #AT91_CKGR_PLLAR]
-
- tst tmp1, #(AT91_PMC_MUL & 0xff0000)
- bne 3f
- tst tmp1, #(AT91_PMC_MUL & ~0xff0000)
- beq 4f
-3:
- wait_pllalock
-4:
-
- /*
- * Restore master clock setting
- */
-2: ldr tmp1, .saved_mckr
- str tmp1, [pmc, #AT91_PMC_MCKR]
-
- wait_mckrdy
-
-skip_enable_clock:
- /*
- * at91rm9200 Memory controller
- * Do nothing - self-refresh is automatically disabled.
- */
- cmp memctrl, #AT91_MEMCTRL_MC
- beq ram_restored
-
- /*
- * DDRSDR Memory controller
- */
- cmp memctrl, #AT91_MEMCTRL_DDRSDR
- bne sdr_en_restore
- /* Restore MDR in case of LPDDR1 */
- ldr tmp1, .saved_sam9_mdr
- str tmp1, [sdramc, #AT91_DDRSDRC_MDR]
- /* Restore LPR on AT91 with DDRAM */
- ldr tmp1, .saved_sam9_lpr
- str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
-
- /* if we use the second ram controller */
- cmp ramc1, #0
- ldrne tmp2, .saved_sam9_mdr1
- strne tmp2, [ramc1, #AT91_DDRSDRC_MDR]
- ldrne tmp2, .saved_sam9_lpr1
- strne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
-
- b ram_restored
-
- /*
- * SDRAMC Memory controller
- */
-sdr_en_restore:
- /* Restore LPR on AT91 with SDRAM */
- ldr tmp1, .saved_sam9_lpr
- str tmp1, [sdramc, #AT91_SDRAMC_LPR]
-
-ram_restored:
- /* Restore registers, and return */
- ldmfd sp!, {r4 - r12, pc}
-
-
-.saved_mckr:
- .word 0
-
-.saved_pllar:
- .word 0
-
-.saved_sam9_lpr:
- .word 0
-
-.saved_sam9_lpr1:
- .word 0
-
-.saved_sam9_mdr:
- .word 0
-
-.saved_sam9_mdr1:
- .word 0
-
-ENTRY(at91_slow_clock_sz)
- .word .-at91_slow_clock
diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
new file mode 100644
index 0000000..420e730
--- /dev/null
+++ b/arch/arm/mach-at91/pm_suspend.S
@@ -0,0 +1,280 @@
+/*
+ * arch/arm/mach-at91/pm_suspend.S
+ *
+ * Copyright (C) 2006 Savin Zlobec
+ *
+ * AT91SAM9 support:
+ * Copyright (C) 2007 Anti Sullin <anti.sullin@artecdesign.ee
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ */
+#include <linux/linkage.h>
+#include <linux/clk/at91_pmc.h>
+#include <mach/at91_ramc.h>
+
+#include "pm.h"
+
+pmc .req r0
+sdramc .req r1
+ramc1 .req r2
+memctrl .req r3
+tmp1 .req r4
+tmp2 .req r5
+mode .req r6
+
+/*
+ * Wait until master clock is ready (after switching master clock source)
+ */
+ .macro wait_mckrdy
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
+ tst tmp1, #AT91_PMC_MCKRDY
+ beq 1b
+ .endm
+
+/*
+ * Wait until master oscillator has stabilized.
+ */
+ .macro wait_moscrdy
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
+ tst tmp1, #AT91_PMC_MOSCS
+ beq 1b
+ .endm
+
+/*
+ * Wait until PLLA has locked.
+ */
+ .macro wait_pllalock
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
+ tst tmp1, #AT91_PMC_LOCKA
+ beq 1b
+ .endm
+
+ .text
+
+/* void at91_slow_clock(void __iomem *pmc, void __iomem *sdramc,
+ * void __iomem *ramc1, int memctrl)
+ */
+ENTRY(at91_slow_clock)
+ /* Save registers on stack */
+ stmfd sp!, {r4 - r12, lr}
+
+ /*
+ * Register usage:
+ * R0 = Base address of AT91_PMC
+ * R1 = Base address of RAM Controller (SDRAM, DDRSDR, or AT91_SYS)
+ * R2 = Base address of second RAM Controller or 0 if not present
+ * R3 = Memory controller
+ * R4 = temporary register
+ * R5 = temporary register
+ */
+
+ /* Drain write buffer */
+ mov tmp1, #0
+ mcr p15, 0, tmp1, c7, c10, 4
+
+ mov tmp1, memctrl
+ mov tmp2, tmp1, lsr#AT91_PM_MODE_OFFSET
+ and mode, tmp2, #AT91_PM_MODE_MASK
+
+ mov tmp1, memctrl
+ and memctrl, tmp1, #AT91_PM_MEMCTRL_MASK
+
+ cmp memctrl, #AT91_MEMCTRL_MC
+ bne ddr_sr_enable
+
+ /*
+ * at91rm9200 Memory controller
+ */
+ /* Put SDRAM in self-refresh mode */
+ mov tmp1, #1
+ str tmp1, [sdramc, #AT91RM9200_SDRAMC_SRR]
+ b sdr_sr_done
+
+ /*
+ * DDRSDR Memory controller
+ */
+ddr_sr_enable:
+ cmp memctrl, #AT91_MEMCTRL_DDRSDR
+ bne sdr_sr_enable
+
+ /* LPDDR1 --> force DDR2 mode during self-refresh */
+ ldr tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ str tmp1, .saved_sam9_mdr
+ bic tmp1, tmp1, #~AT91_DDRSDRC_MD
+ cmp tmp1, #AT91_DDRSDRC_MD_LOW_POWER_DDR
+ ldreq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ biceq tmp1, tmp1, #AT91_DDRSDRC_MD
+ orreq tmp1, tmp1, #AT91_DDRSDRC_MD_DDR2
+ streq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+
+ /* prepare for DDRAM self-refresh mode */
+ ldr tmp1, [sdramc, #AT91_DDRSDRC_LPR]
+ str tmp1, .saved_sam9_lpr
+ bic tmp1, #AT91_DDRSDRC_LPCB
+ orr tmp1, #AT91_DDRSDRC_LPCB_SELF_REFRESH
+
+ /* figure out if we use the second ram controller */
+ cmp ramc1, #0
+ beq ddr_no_2nd_ctrl
+
+ ldr tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ str tmp2, .saved_sam9_mdr1
+ bic tmp2, tmp2, #~AT91_DDRSDRC_MD
+ cmp tmp2, #AT91_DDRSDRC_MD_LOW_POWER_DDR
+ ldreq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ biceq tmp2, tmp2, #AT91_DDRSDRC_MD
+ orreq tmp2, tmp2, #AT91_DDRSDRC_MD_DDR2
+ streq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+
+ ldr tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+ str tmp2, .saved_sam9_lpr1
+ bic tmp2, #AT91_DDRSDRC_LPCB
+ orr tmp2, #AT91_DDRSDRC_LPCB_SELF_REFRESH
+
+ /* Enable DDRAM self-refresh mode */
+ str tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+ddr_no_2nd_ctrl:
+ str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
+
+ b sdr_sr_done
+
+ /*
+ * SDRAMC Memory controller
+ */
+sdr_sr_enable:
+ /* Enable SDRAM self-refresh mode */
+ ldr tmp1, [sdramc, #AT91_SDRAMC_LPR]
+ str tmp1, .saved_sam9_lpr
+
+ bic tmp1, #AT91_SDRAMC_LPCB
+ orr tmp1, #AT91_SDRAMC_LPCB_SELF_REFRESH
+ str tmp1, [sdramc, #AT91_SDRAMC_LPR]
+
+sdr_sr_done:
+ tst mode, #AT91_PM_SLOW_CLOCK
+ beq skip_disable_clock
+
+ /* Save Master clock setting */
+ ldr tmp1, [pmc, #AT91_PMC_MCKR]
+ str tmp1, .saved_mckr
+
+ /*
+ * Set the Master clock source to slow clock
+ */
+ bic tmp1, tmp1, #AT91_PMC_CSS
+ str tmp1, [pmc, #AT91_PMC_MCKR]
+
+ wait_mckrdy
+
+ /* Save PLLA setting and disable it */
+ ldr tmp1, [pmc, #AT91_CKGR_PLLAR]
+ str tmp1, .saved_pllar
+
+ mov tmp1, #AT91_PMC_PLLCOUNT
+ orr tmp1, tmp1, #(1 << 29) /* bit 29 always set */
+ str tmp1, [pmc, #AT91_CKGR_PLLAR]
+
+ /* Turn off the main oscillator */
+ ldr tmp1, [pmc, #AT91_CKGR_MOR]
+ bic tmp1, tmp1, #AT91_PMC_MOSCEN
+ str tmp1, [pmc, #AT91_CKGR_MOR]
+
+skip_disable_clock:
+ /* Wait for interrupt */
+ mcr p15, 0, tmp1, c7, c0, 4
+
+ tst mode, #AT91_PM_SLOW_CLOCK
+ beq skip_enable_clock
+
+ /* Turn on the main oscillator */
+ ldr tmp1, [pmc, #AT91_CKGR_MOR]
+ orr tmp1, tmp1, #AT91_PMC_MOSCEN
+ str tmp1, [pmc, #AT91_CKGR_MOR]
+
+ wait_moscrdy
+
+ /* Restore PLLA setting */
+ ldr tmp1, .saved_pllar
+ str tmp1, [pmc, #AT91_CKGR_PLLAR]
+
+ tst tmp1, #(AT91_PMC_MUL & 0xff0000)
+ bne 3f
+ tst tmp1, #(AT91_PMC_MUL & ~0xff0000)
+ beq 4f
+3:
+ wait_pllalock
+4:
+
+ /*
+ * Restore master clock setting
+ */
+2: ldr tmp1, .saved_mckr
+ str tmp1, [pmc, #AT91_PMC_MCKR]
+
+ wait_mckrdy
+
+skip_enable_clock:
+ /*
+ * at91rm9200 Memory controller
+ * Do nothing - self-refresh is automatically disabled.
+ */
+ cmp memctrl, #AT91_MEMCTRL_MC
+ beq ram_restored
+
+ /*
+ * DDRSDR Memory controller
+ */
+ cmp memctrl, #AT91_MEMCTRL_DDRSDR
+ bne sdr_en_restore
+ /* Restore MDR in case of LPDDR1 */
+ ldr tmp1, .saved_sam9_mdr
+ str tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ /* Restore LPR on AT91 with DDRAM */
+ ldr tmp1, .saved_sam9_lpr
+ str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
+
+ /* if we use the second ram controller */
+ cmp ramc1, #0
+ ldrne tmp2, .saved_sam9_mdr1
+ strne tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ ldrne tmp2, .saved_sam9_lpr1
+ strne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+
+ b ram_restored
+
+ /*
+ * SDRAMC Memory controller
+ */
+sdr_en_restore:
+ /* Restore LPR on AT91 with SDRAM */
+ ldr tmp1, .saved_sam9_lpr
+ str tmp1, [sdramc, #AT91_SDRAMC_LPR]
+
+ram_restored:
+ /* Restore registers, and return */
+ ldmfd sp!, {r4 - r12, pc}
+
+
+.saved_mckr:
+ .word 0
+
+.saved_pllar:
+ .word 0
+
+.saved_sam9_lpr:
+ .word 0
+
+.saved_sam9_lpr1:
+ .word 0
+
+.saved_sam9_mdr:
+ .word 0
+
+.saved_sam9_mdr1:
+ .word 0
+
+ENTRY(at91_slow_clock_sz)
+ .word .-at91_slow_clock
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
@ 2015-01-20 8:17 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:17 UTC (permalink / raw)
To: linux-arm-kernel
Because the sram function is used for the suspend to standby mode as well,
more than suspend to memory, so renaming is more elegant.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/Makefile | 2 +-
arch/arm/mach-at91/pm_slowclock.S | 280 -------------------------------------
arch/arm/mach-at91/pm_suspend.S | 280 +++++++++++++++++++++++++++++++++++++
3 files changed, 281 insertions(+), 281 deletions(-)
delete mode 100644 arch/arm/mach-at91/pm_slowclock.S
create mode 100644 arch/arm/mach-at91/pm_suspend.S
diff --git a/arch/arm/mach-at91/Makefile b/arch/arm/mach-at91/Makefile
index 9c755d9..80473d8 100644
--- a/arch/arm/mach-at91/Makefile
+++ b/arch/arm/mach-at91/Makefile
@@ -18,7 +18,7 @@ obj-$(CONFIG_SOC_SAMA5) += board-dt-sama5.o
# Power Management
obj-$(CONFIG_PM) += pm.o
-obj-$(CONFIG_PM) += pm_slowclock.o
+obj-$(CONFIG_PM) += pm_suspend.o
ifeq ($(CONFIG_PM_DEBUG),y)
CFLAGS_pm.o += -DDEBUG
diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
deleted file mode 100644
index 92d7e63..0000000
--- a/arch/arm/mach-at91/pm_slowclock.S
+++ /dev/null
@@ -1,280 +0,0 @@
-/*
- * arch/arm/mach-at91/pm_slow_clock.S
- *
- * Copyright (C) 2006 Savin Zlobec
- *
- * AT91SAM9 support:
- * Copyright (C) 2007 Anti Sullin <anti.sullin@artecdesign.ee
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License version 2 as
- * published by the Free Software Foundation.
- *
- */
-#include <linux/linkage.h>
-#include <linux/clk/at91_pmc.h>
-#include <mach/at91_ramc.h>
-
-#include "pm.h"
-
-pmc .req r0
-sdramc .req r1
-ramc1 .req r2
-memctrl .req r3
-tmp1 .req r4
-tmp2 .req r5
-mode .req r6
-
-/*
- * Wait until master clock is ready (after switching master clock source)
- */
- .macro wait_mckrdy
-1: ldr tmp1, [pmc, #AT91_PMC_SR]
- tst tmp1, #AT91_PMC_MCKRDY
- beq 1b
- .endm
-
-/*
- * Wait until master oscillator has stabilized.
- */
- .macro wait_moscrdy
-1: ldr tmp1, [pmc, #AT91_PMC_SR]
- tst tmp1, #AT91_PMC_MOSCS
- beq 1b
- .endm
-
-/*
- * Wait until PLLA has locked.
- */
- .macro wait_pllalock
-1: ldr tmp1, [pmc, #AT91_PMC_SR]
- tst tmp1, #AT91_PMC_LOCKA
- beq 1b
- .endm
-
- .text
-
-/* void at91_slow_clock(void __iomem *pmc, void __iomem *sdramc,
- * void __iomem *ramc1, int memctrl)
- */
-ENTRY(at91_slow_clock)
- /* Save registers on stack */
- stmfd sp!, {r4 - r12, lr}
-
- /*
- * Register usage:
- * R0 = Base address of AT91_PMC
- * R1 = Base address of RAM Controller (SDRAM, DDRSDR, or AT91_SYS)
- * R2 = Base address of second RAM Controller or 0 if not present
- * R3 = Memory controller
- * R4 = temporary register
- * R5 = temporary register
- */
-
- /* Drain write buffer */
- mov tmp1, #0
- mcr p15, 0, tmp1, c7, c10, 4
-
- mov tmp1, memctrl
- mov tmp2, tmp1, lsr#AT91_PM_MODE_OFFSET
- and mode, tmp2, #AT91_PM_MODE_MASK
-
- mov tmp1, memctrl
- and memctrl, tmp1, #AT91_PM_MEMCTRL_MASK
-
- cmp memctrl, #AT91_MEMCTRL_MC
- bne ddr_sr_enable
-
- /*
- * at91rm9200 Memory controller
- */
- /* Put SDRAM in self-refresh mode */
- mov tmp1, #1
- str tmp1, [sdramc, #AT91RM9200_SDRAMC_SRR]
- b sdr_sr_done
-
- /*
- * DDRSDR Memory controller
- */
-ddr_sr_enable:
- cmp memctrl, #AT91_MEMCTRL_DDRSDR
- bne sdr_sr_enable
-
- /* LPDDR1 --> force DDR2 mode during self-refresh */
- ldr tmp1, [sdramc, #AT91_DDRSDRC_MDR]
- str tmp1, .saved_sam9_mdr
- bic tmp1, tmp1, #~AT91_DDRSDRC_MD
- cmp tmp1, #AT91_DDRSDRC_MD_LOW_POWER_DDR
- ldreq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
- biceq tmp1, tmp1, #AT91_DDRSDRC_MD
- orreq tmp1, tmp1, #AT91_DDRSDRC_MD_DDR2
- streq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
-
- /* prepare for DDRAM self-refresh mode */
- ldr tmp1, [sdramc, #AT91_DDRSDRC_LPR]
- str tmp1, .saved_sam9_lpr
- bic tmp1, #AT91_DDRSDRC_LPCB
- orr tmp1, #AT91_DDRSDRC_LPCB_SELF_REFRESH
-
- /* figure out if we use the second ram controller */
- cmp ramc1, #0
- beq ddr_no_2nd_ctrl
-
- ldr tmp2, [ramc1, #AT91_DDRSDRC_MDR]
- str tmp2, .saved_sam9_mdr1
- bic tmp2, tmp2, #~AT91_DDRSDRC_MD
- cmp tmp2, #AT91_DDRSDRC_MD_LOW_POWER_DDR
- ldreq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
- biceq tmp2, tmp2, #AT91_DDRSDRC_MD
- orreq tmp2, tmp2, #AT91_DDRSDRC_MD_DDR2
- streq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
-
- ldr tmp2, [ramc1, #AT91_DDRSDRC_LPR]
- str tmp2, .saved_sam9_lpr1
- bic tmp2, #AT91_DDRSDRC_LPCB
- orr tmp2, #AT91_DDRSDRC_LPCB_SELF_REFRESH
-
- /* Enable DDRAM self-refresh mode */
- str tmp2, [ramc1, #AT91_DDRSDRC_LPR]
-ddr_no_2nd_ctrl:
- str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
-
- b sdr_sr_done
-
- /*
- * SDRAMC Memory controller
- */
-sdr_sr_enable:
- /* Enable SDRAM self-refresh mode */
- ldr tmp1, [sdramc, #AT91_SDRAMC_LPR]
- str tmp1, .saved_sam9_lpr
-
- bic tmp1, #AT91_SDRAMC_LPCB
- orr tmp1, #AT91_SDRAMC_LPCB_SELF_REFRESH
- str tmp1, [sdramc, #AT91_SDRAMC_LPR]
-
-sdr_sr_done:
- tst mode, #AT91_PM_SLOW_CLOCK
- beq skip_disable_clock
-
- /* Save Master clock setting */
- ldr tmp1, [pmc, #AT91_PMC_MCKR]
- str tmp1, .saved_mckr
-
- /*
- * Set the Master clock source to slow clock
- */
- bic tmp1, tmp1, #AT91_PMC_CSS
- str tmp1, [pmc, #AT91_PMC_MCKR]
-
- wait_mckrdy
-
- /* Save PLLA setting and disable it */
- ldr tmp1, [pmc, #AT91_CKGR_PLLAR]
- str tmp1, .saved_pllar
-
- mov tmp1, #AT91_PMC_PLLCOUNT
- orr tmp1, tmp1, #(1 << 29) /* bit 29 always set */
- str tmp1, [pmc, #AT91_CKGR_PLLAR]
-
- /* Turn off the main oscillator */
- ldr tmp1, [pmc, #AT91_CKGR_MOR]
- bic tmp1, tmp1, #AT91_PMC_MOSCEN
- str tmp1, [pmc, #AT91_CKGR_MOR]
-
-skip_disable_clock:
- /* Wait for interrupt */
- mcr p15, 0, tmp1, c7, c0, 4
-
- tst mode, #AT91_PM_SLOW_CLOCK
- beq skip_enable_clock
-
- /* Turn on the main oscillator */
- ldr tmp1, [pmc, #AT91_CKGR_MOR]
- orr tmp1, tmp1, #AT91_PMC_MOSCEN
- str tmp1, [pmc, #AT91_CKGR_MOR]
-
- wait_moscrdy
-
- /* Restore PLLA setting */
- ldr tmp1, .saved_pllar
- str tmp1, [pmc, #AT91_CKGR_PLLAR]
-
- tst tmp1, #(AT91_PMC_MUL & 0xff0000)
- bne 3f
- tst tmp1, #(AT91_PMC_MUL & ~0xff0000)
- beq 4f
-3:
- wait_pllalock
-4:
-
- /*
- * Restore master clock setting
- */
-2: ldr tmp1, .saved_mckr
- str tmp1, [pmc, #AT91_PMC_MCKR]
-
- wait_mckrdy
-
-skip_enable_clock:
- /*
- * at91rm9200 Memory controller
- * Do nothing - self-refresh is automatically disabled.
- */
- cmp memctrl, #AT91_MEMCTRL_MC
- beq ram_restored
-
- /*
- * DDRSDR Memory controller
- */
- cmp memctrl, #AT91_MEMCTRL_DDRSDR
- bne sdr_en_restore
- /* Restore MDR in case of LPDDR1 */
- ldr tmp1, .saved_sam9_mdr
- str tmp1, [sdramc, #AT91_DDRSDRC_MDR]
- /* Restore LPR on AT91 with DDRAM */
- ldr tmp1, .saved_sam9_lpr
- str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
-
- /* if we use the second ram controller */
- cmp ramc1, #0
- ldrne tmp2, .saved_sam9_mdr1
- strne tmp2, [ramc1, #AT91_DDRSDRC_MDR]
- ldrne tmp2, .saved_sam9_lpr1
- strne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
-
- b ram_restored
-
- /*
- * SDRAMC Memory controller
- */
-sdr_en_restore:
- /* Restore LPR on AT91 with SDRAM */
- ldr tmp1, .saved_sam9_lpr
- str tmp1, [sdramc, #AT91_SDRAMC_LPR]
-
-ram_restored:
- /* Restore registers, and return */
- ldmfd sp!, {r4 - r12, pc}
-
-
-.saved_mckr:
- .word 0
-
-.saved_pllar:
- .word 0
-
-.saved_sam9_lpr:
- .word 0
-
-.saved_sam9_lpr1:
- .word 0
-
-.saved_sam9_mdr:
- .word 0
-
-.saved_sam9_mdr1:
- .word 0
-
-ENTRY(at91_slow_clock_sz)
- .word .-at91_slow_clock
diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
new file mode 100644
index 0000000..420e730
--- /dev/null
+++ b/arch/arm/mach-at91/pm_suspend.S
@@ -0,0 +1,280 @@
+/*
+ * arch/arm/mach-at91/pm_suspend.S
+ *
+ * Copyright (C) 2006 Savin Zlobec
+ *
+ * AT91SAM9 support:
+ * Copyright (C) 2007 Anti Sullin <anti.sullin@artecdesign.ee
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2 as
+ * published by the Free Software Foundation.
+ *
+ */
+#include <linux/linkage.h>
+#include <linux/clk/at91_pmc.h>
+#include <mach/at91_ramc.h>
+
+#include "pm.h"
+
+pmc .req r0
+sdramc .req r1
+ramc1 .req r2
+memctrl .req r3
+tmp1 .req r4
+tmp2 .req r5
+mode .req r6
+
+/*
+ * Wait until master clock is ready (after switching master clock source)
+ */
+ .macro wait_mckrdy
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
+ tst tmp1, #AT91_PMC_MCKRDY
+ beq 1b
+ .endm
+
+/*
+ * Wait until master oscillator has stabilized.
+ */
+ .macro wait_moscrdy
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
+ tst tmp1, #AT91_PMC_MOSCS
+ beq 1b
+ .endm
+
+/*
+ * Wait until PLLA has locked.
+ */
+ .macro wait_pllalock
+1: ldr tmp1, [pmc, #AT91_PMC_SR]
+ tst tmp1, #AT91_PMC_LOCKA
+ beq 1b
+ .endm
+
+ .text
+
+/* void at91_slow_clock(void __iomem *pmc, void __iomem *sdramc,
+ * void __iomem *ramc1, int memctrl)
+ */
+ENTRY(at91_slow_clock)
+ /* Save registers on stack */
+ stmfd sp!, {r4 - r12, lr}
+
+ /*
+ * Register usage:
+ * R0 = Base address of AT91_PMC
+ * R1 = Base address of RAM Controller (SDRAM, DDRSDR, or AT91_SYS)
+ * R2 = Base address of second RAM Controller or 0 if not present
+ * R3 = Memory controller
+ * R4 = temporary register
+ * R5 = temporary register
+ */
+
+ /* Drain write buffer */
+ mov tmp1, #0
+ mcr p15, 0, tmp1, c7, c10, 4
+
+ mov tmp1, memctrl
+ mov tmp2, tmp1, lsr#AT91_PM_MODE_OFFSET
+ and mode, tmp2, #AT91_PM_MODE_MASK
+
+ mov tmp1, memctrl
+ and memctrl, tmp1, #AT91_PM_MEMCTRL_MASK
+
+ cmp memctrl, #AT91_MEMCTRL_MC
+ bne ddr_sr_enable
+
+ /*
+ * at91rm9200 Memory controller
+ */
+ /* Put SDRAM in self-refresh mode */
+ mov tmp1, #1
+ str tmp1, [sdramc, #AT91RM9200_SDRAMC_SRR]
+ b sdr_sr_done
+
+ /*
+ * DDRSDR Memory controller
+ */
+ddr_sr_enable:
+ cmp memctrl, #AT91_MEMCTRL_DDRSDR
+ bne sdr_sr_enable
+
+ /* LPDDR1 --> force DDR2 mode during self-refresh */
+ ldr tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ str tmp1, .saved_sam9_mdr
+ bic tmp1, tmp1, #~AT91_DDRSDRC_MD
+ cmp tmp1, #AT91_DDRSDRC_MD_LOW_POWER_DDR
+ ldreq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ biceq tmp1, tmp1, #AT91_DDRSDRC_MD
+ orreq tmp1, tmp1, #AT91_DDRSDRC_MD_DDR2
+ streq tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+
+ /* prepare for DDRAM self-refresh mode */
+ ldr tmp1, [sdramc, #AT91_DDRSDRC_LPR]
+ str tmp1, .saved_sam9_lpr
+ bic tmp1, #AT91_DDRSDRC_LPCB
+ orr tmp1, #AT91_DDRSDRC_LPCB_SELF_REFRESH
+
+ /* figure out if we use the second ram controller */
+ cmp ramc1, #0
+ beq ddr_no_2nd_ctrl
+
+ ldr tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ str tmp2, .saved_sam9_mdr1
+ bic tmp2, tmp2, #~AT91_DDRSDRC_MD
+ cmp tmp2, #AT91_DDRSDRC_MD_LOW_POWER_DDR
+ ldreq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ biceq tmp2, tmp2, #AT91_DDRSDRC_MD
+ orreq tmp2, tmp2, #AT91_DDRSDRC_MD_DDR2
+ streq tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+
+ ldr tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+ str tmp2, .saved_sam9_lpr1
+ bic tmp2, #AT91_DDRSDRC_LPCB
+ orr tmp2, #AT91_DDRSDRC_LPCB_SELF_REFRESH
+
+ /* Enable DDRAM self-refresh mode */
+ str tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+ddr_no_2nd_ctrl:
+ str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
+
+ b sdr_sr_done
+
+ /*
+ * SDRAMC Memory controller
+ */
+sdr_sr_enable:
+ /* Enable SDRAM self-refresh mode */
+ ldr tmp1, [sdramc, #AT91_SDRAMC_LPR]
+ str tmp1, .saved_sam9_lpr
+
+ bic tmp1, #AT91_SDRAMC_LPCB
+ orr tmp1, #AT91_SDRAMC_LPCB_SELF_REFRESH
+ str tmp1, [sdramc, #AT91_SDRAMC_LPR]
+
+sdr_sr_done:
+ tst mode, #AT91_PM_SLOW_CLOCK
+ beq skip_disable_clock
+
+ /* Save Master clock setting */
+ ldr tmp1, [pmc, #AT91_PMC_MCKR]
+ str tmp1, .saved_mckr
+
+ /*
+ * Set the Master clock source to slow clock
+ */
+ bic tmp1, tmp1, #AT91_PMC_CSS
+ str tmp1, [pmc, #AT91_PMC_MCKR]
+
+ wait_mckrdy
+
+ /* Save PLLA setting and disable it */
+ ldr tmp1, [pmc, #AT91_CKGR_PLLAR]
+ str tmp1, .saved_pllar
+
+ mov tmp1, #AT91_PMC_PLLCOUNT
+ orr tmp1, tmp1, #(1 << 29) /* bit 29 always set */
+ str tmp1, [pmc, #AT91_CKGR_PLLAR]
+
+ /* Turn off the main oscillator */
+ ldr tmp1, [pmc, #AT91_CKGR_MOR]
+ bic tmp1, tmp1, #AT91_PMC_MOSCEN
+ str tmp1, [pmc, #AT91_CKGR_MOR]
+
+skip_disable_clock:
+ /* Wait for interrupt */
+ mcr p15, 0, tmp1, c7, c0, 4
+
+ tst mode, #AT91_PM_SLOW_CLOCK
+ beq skip_enable_clock
+
+ /* Turn on the main oscillator */
+ ldr tmp1, [pmc, #AT91_CKGR_MOR]
+ orr tmp1, tmp1, #AT91_PMC_MOSCEN
+ str tmp1, [pmc, #AT91_CKGR_MOR]
+
+ wait_moscrdy
+
+ /* Restore PLLA setting */
+ ldr tmp1, .saved_pllar
+ str tmp1, [pmc, #AT91_CKGR_PLLAR]
+
+ tst tmp1, #(AT91_PMC_MUL & 0xff0000)
+ bne 3f
+ tst tmp1, #(AT91_PMC_MUL & ~0xff0000)
+ beq 4f
+3:
+ wait_pllalock
+4:
+
+ /*
+ * Restore master clock setting
+ */
+2: ldr tmp1, .saved_mckr
+ str tmp1, [pmc, #AT91_PMC_MCKR]
+
+ wait_mckrdy
+
+skip_enable_clock:
+ /*
+ * at91rm9200 Memory controller
+ * Do nothing - self-refresh is automatically disabled.
+ */
+ cmp memctrl, #AT91_MEMCTRL_MC
+ beq ram_restored
+
+ /*
+ * DDRSDR Memory controller
+ */
+ cmp memctrl, #AT91_MEMCTRL_DDRSDR
+ bne sdr_en_restore
+ /* Restore MDR in case of LPDDR1 */
+ ldr tmp1, .saved_sam9_mdr
+ str tmp1, [sdramc, #AT91_DDRSDRC_MDR]
+ /* Restore LPR on AT91 with DDRAM */
+ ldr tmp1, .saved_sam9_lpr
+ str tmp1, [sdramc, #AT91_DDRSDRC_LPR]
+
+ /* if we use the second ram controller */
+ cmp ramc1, #0
+ ldrne tmp2, .saved_sam9_mdr1
+ strne tmp2, [ramc1, #AT91_DDRSDRC_MDR]
+ ldrne tmp2, .saved_sam9_lpr1
+ strne tmp2, [ramc1, #AT91_DDRSDRC_LPR]
+
+ b ram_restored
+
+ /*
+ * SDRAMC Memory controller
+ */
+sdr_en_restore:
+ /* Restore LPR on AT91 with SDRAM */
+ ldr tmp1, .saved_sam9_lpr
+ str tmp1, [sdramc, #AT91_SDRAMC_LPR]
+
+ram_restored:
+ /* Restore registers, and return */
+ ldmfd sp!, {r4 - r12, pc}
+
+
+.saved_mckr:
+ .word 0
+
+.saved_pllar:
+ .word 0
+
+.saved_sam9_lpr:
+ .word 0
+
+.saved_sam9_lpr1:
+ .word 0
+
+.saved_sam9_mdr:
+ .word 0
+
+.saved_sam9_mdr1:
+ .word 0
+
+ENTRY(at91_slow_clock_sz)
+ .word .-at91_slow_clock
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 09/12] pm: at91: rename function name: at91_slow_clock()-->at91_pm_suspend_sram_fn
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:17 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:17 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
As the file name's renaming, rename the file name at91_slow_clock()-->
at91_pm_suspend_sram_fn, rename the function handler's name at the same time.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 23 +++++++++++------------
arch/arm/mach-at91/pm_suspend.S | 26 ++++++++++++--------------
2 files changed, 23 insertions(+), 26 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index a1010f0..98dfeb2 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -137,13 +137,12 @@ int at91_suspend_entering_slow_clock(void)
}
EXPORT_SYMBOL(at91_suspend_entering_slow_clock);
-
-static void (*slow_clock)(void __iomem *pmc, void __iomem *ramc0,
+static void (*at91_suspend_sram_fn)(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
+extern void at91_pm_suspend_in_sram(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-extern u32 at91_slow_clock_sz;
+extern u32 at91_pm_suspend_in_sram_sz;
static void at91_pm_suspend(suspend_state_t state)
{
@@ -152,8 +151,8 @@ static void at91_pm_suspend(suspend_state_t state)
pm_data |= (state == PM_SUSPEND_MEM) ?
(AT91_PM_SLOW_CLOCK << AT91_PM_MODE_OFFSET) : 0;
- slow_clock(at91_pmc_base, at91_ramc_base[0],
- at91_ramc_base[1], pm_data);
+ at91_suspend_sram_fn(at91_pmc_base, at91_ramc_base[0],
+ at91_ramc_base[1], pm_data);
}
static int at91_pm_enter(suspend_state_t state)
@@ -253,17 +252,19 @@ static void __init at91_pm_sram_init(void)
goto put_node;
}
- sram_base = gen_pool_alloc(sram_pool, at91_slow_clock_sz);
+ sram_base = gen_pool_alloc(sram_pool, at91_pm_suspend_in_sram_sz);
if (!sram_base) {
pr_warn("%s: unable to alloc ocram!\n", __func__);
goto put_node;
}
sram_pbase = gen_pool_virt_to_phys(sram_pool, sram_base);
- slow_clock = __arm_ioremap_exec(sram_pbase, at91_slow_clock_sz, false);
+ at91_suspend_sram_fn = __arm_ioremap_exec(sram_pbase,
+ at91_pm_suspend_in_sram_sz, false);
- /* Copy slow_clock handler to SRAM, and call it */
- memcpy(slow_clock, at91_slow_clock, at91_slow_clock_sz);
+ /* Copy pm suspend handler to SRAM, and call it */
+ memcpy(at91_suspend_sram_fn,
+ at91_pm_suspend_in_sram, at91_pm_suspend_in_sram_sz);
put_node:
of_node_put(node);
@@ -312,8 +313,6 @@ static void __init at91_pm_init(void)
at91_pm_sram_init();
- pr_info("AT91: Power Management%s\n", (slow_clock ? " (with slow clock mode)" : ""));
-
if (at91_cpuidle_device.dev.platform_data)
platform_device_register(&at91_cpuidle_device);
diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
index 420e730..6c3fc50 100644
--- a/arch/arm/mach-at91/pm_suspend.S
+++ b/arch/arm/mach-at91/pm_suspend.S
@@ -54,23 +54,21 @@ mode .req r6
.text
-/* void at91_slow_clock(void __iomem *pmc, void __iomem *sdramc,
+/*
+ * void at91_pm_suspend_in_sram(void __iomem *pmc, void __iomem *sdramc,
* void __iomem *ramc1, int memctrl)
+ * @input param:
+ * @r0: the base address of AT91_PMC
+ * @r1: the base address of SDRAM Controller (SDRAM, DDRSDR, or AT91_SYS)
+ * @r2: the base address of second SDRAM Controller or 0 if not present
+ * @r3: the pm information,
+ * least byte: SDRAM controller type
+ * 2nd byte: 1 -> suspend to memory, 0 -> standby
*/
-ENTRY(at91_slow_clock)
+ENTRY(at91_pm_suspend_in_sram)
/* Save registers on stack */
stmfd sp!, {r4 - r12, lr}
- /*
- * Register usage:
- * R0 = Base address of AT91_PMC
- * R1 = Base address of RAM Controller (SDRAM, DDRSDR, or AT91_SYS)
- * R2 = Base address of second RAM Controller or 0 if not present
- * R3 = Memory controller
- * R4 = temporary register
- * R5 = temporary register
- */
-
/* Drain write buffer */
mov tmp1, #0
mcr p15, 0, tmp1, c7, c10, 4
@@ -276,5 +274,5 @@ ram_restored:
.saved_sam9_mdr1:
.word 0
-ENTRY(at91_slow_clock_sz)
- .word .-at91_slow_clock
+ENTRY(at91_pm_suspend_in_sram_sz)
+ .word .-at91_pm_suspend_in_sram
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 09/12] pm: at91: rename function name: at91_slow_clock()-->at91_pm_suspend_sram_fn
@ 2015-01-20 8:17 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:17 UTC (permalink / raw)
To: linux-arm-kernel
As the file name's renaming, rename the file name at91_slow_clock()-->
at91_pm_suspend_sram_fn, rename the function handler's name at the same time.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 23 +++++++++++------------
arch/arm/mach-at91/pm_suspend.S | 26 ++++++++++++--------------
2 files changed, 23 insertions(+), 26 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index a1010f0..98dfeb2 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -137,13 +137,12 @@ int at91_suspend_entering_slow_clock(void)
}
EXPORT_SYMBOL(at91_suspend_entering_slow_clock);
-
-static void (*slow_clock)(void __iomem *pmc, void __iomem *ramc0,
+static void (*at91_suspend_sram_fn)(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
+extern void at91_pm_suspend_in_sram(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-extern u32 at91_slow_clock_sz;
+extern u32 at91_pm_suspend_in_sram_sz;
static void at91_pm_suspend(suspend_state_t state)
{
@@ -152,8 +151,8 @@ static void at91_pm_suspend(suspend_state_t state)
pm_data |= (state == PM_SUSPEND_MEM) ?
(AT91_PM_SLOW_CLOCK << AT91_PM_MODE_OFFSET) : 0;
- slow_clock(at91_pmc_base, at91_ramc_base[0],
- at91_ramc_base[1], pm_data);
+ at91_suspend_sram_fn(at91_pmc_base, at91_ramc_base[0],
+ at91_ramc_base[1], pm_data);
}
static int at91_pm_enter(suspend_state_t state)
@@ -253,17 +252,19 @@ static void __init at91_pm_sram_init(void)
goto put_node;
}
- sram_base = gen_pool_alloc(sram_pool, at91_slow_clock_sz);
+ sram_base = gen_pool_alloc(sram_pool, at91_pm_suspend_in_sram_sz);
if (!sram_base) {
pr_warn("%s: unable to alloc ocram!\n", __func__);
goto put_node;
}
sram_pbase = gen_pool_virt_to_phys(sram_pool, sram_base);
- slow_clock = __arm_ioremap_exec(sram_pbase, at91_slow_clock_sz, false);
+ at91_suspend_sram_fn = __arm_ioremap_exec(sram_pbase,
+ at91_pm_suspend_in_sram_sz, false);
- /* Copy slow_clock handler to SRAM, and call it */
- memcpy(slow_clock, at91_slow_clock, at91_slow_clock_sz);
+ /* Copy pm suspend handler to SRAM, and call it */
+ memcpy(at91_suspend_sram_fn,
+ at91_pm_suspend_in_sram, at91_pm_suspend_in_sram_sz);
put_node:
of_node_put(node);
@@ -312,8 +313,6 @@ static void __init at91_pm_init(void)
at91_pm_sram_init();
- pr_info("AT91: Power Management%s\n", (slow_clock ? " (with slow clock mode)" : ""));
-
if (at91_cpuidle_device.dev.platform_data)
platform_device_register(&at91_cpuidle_device);
diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
index 420e730..6c3fc50 100644
--- a/arch/arm/mach-at91/pm_suspend.S
+++ b/arch/arm/mach-at91/pm_suspend.S
@@ -54,23 +54,21 @@ mode .req r6
.text
-/* void at91_slow_clock(void __iomem *pmc, void __iomem *sdramc,
+/*
+ * void at91_pm_suspend_in_sram(void __iomem *pmc, void __iomem *sdramc,
* void __iomem *ramc1, int memctrl)
+ * @input param:
+ * @r0: the base address of AT91_PMC
+ * @r1: the base address of SDRAM Controller (SDRAM, DDRSDR, or AT91_SYS)
+ * @r2: the base address of second SDRAM Controller or 0 if not present
+ * @r3: the pm information,
+ * least byte: SDRAM controller type
+ * 2nd byte: 1 -> suspend to memory, 0 -> standby
*/
-ENTRY(at91_slow_clock)
+ENTRY(at91_pm_suspend_in_sram)
/* Save registers on stack */
stmfd sp!, {r4 - r12, lr}
- /*
- * Register usage:
- * R0 = Base address of AT91_PMC
- * R1 = Base address of RAM Controller (SDRAM, DDRSDR, or AT91_SYS)
- * R2 = Base address of second RAM Controller or 0 if not present
- * R3 = Memory controller
- * R4 = temporary register
- * R5 = temporary register
- */
-
/* Drain write buffer */
mov tmp1, #0
mcr p15, 0, tmp1, c7, c10, 4
@@ -276,5 +274,5 @@ ram_restored:
.saved_sam9_mdr1:
.word 0
-ENTRY(at91_slow_clock_sz)
- .word .-at91_slow_clock
+ENTRY(at91_pm_suspend_in_sram_sz)
+ .word .-at91_pm_suspend_in_sram
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 09/12] pm: at91: rename function name: at91_slow_clock()-->at91_pm_suspend_sram_fn
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:24 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:24 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
As the file name's renaming, rename the file name at91_slow_clock()-->
at91_pm_suspend_sram_fn, rename the function handler's name at the same time.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 23 +++++++++++------------
arch/arm/mach-at91/pm_suspend.S | 26 ++++++++++++--------------
2 files changed, 23 insertions(+), 26 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index a1010f0..98dfeb2 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -137,13 +137,12 @@ int at91_suspend_entering_slow_clock(void)
}
EXPORT_SYMBOL(at91_suspend_entering_slow_clock);
-
-static void (*slow_clock)(void __iomem *pmc, void __iomem *ramc0,
+static void (*at91_suspend_sram_fn)(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
+extern void at91_pm_suspend_in_sram(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-extern u32 at91_slow_clock_sz;
+extern u32 at91_pm_suspend_in_sram_sz;
static void at91_pm_suspend(suspend_state_t state)
{
@@ -152,8 +151,8 @@ static void at91_pm_suspend(suspend_state_t state)
pm_data |= (state == PM_SUSPEND_MEM) ?
(AT91_PM_SLOW_CLOCK << AT91_PM_MODE_OFFSET) : 0;
- slow_clock(at91_pmc_base, at91_ramc_base[0],
- at91_ramc_base[1], pm_data);
+ at91_suspend_sram_fn(at91_pmc_base, at91_ramc_base[0],
+ at91_ramc_base[1], pm_data);
}
static int at91_pm_enter(suspend_state_t state)
@@ -253,17 +252,19 @@ static void __init at91_pm_sram_init(void)
goto put_node;
}
- sram_base = gen_pool_alloc(sram_pool, at91_slow_clock_sz);
+ sram_base = gen_pool_alloc(sram_pool, at91_pm_suspend_in_sram_sz);
if (!sram_base) {
pr_warn("%s: unable to alloc ocram!\n", __func__);
goto put_node;
}
sram_pbase = gen_pool_virt_to_phys(sram_pool, sram_base);
- slow_clock = __arm_ioremap_exec(sram_pbase, at91_slow_clock_sz, false);
+ at91_suspend_sram_fn = __arm_ioremap_exec(sram_pbase,
+ at91_pm_suspend_in_sram_sz, false);
- /* Copy slow_clock handler to SRAM, and call it */
- memcpy(slow_clock, at91_slow_clock, at91_slow_clock_sz);
+ /* Copy pm suspend handler to SRAM, and call it */
+ memcpy(at91_suspend_sram_fn,
+ at91_pm_suspend_in_sram, at91_pm_suspend_in_sram_sz);
put_node:
of_node_put(node);
@@ -312,8 +313,6 @@ static void __init at91_pm_init(void)
at91_pm_sram_init();
- pr_info("AT91: Power Management%s\n", (slow_clock ? " (with slow clock mode)" : ""));
-
if (at91_cpuidle_device.dev.platform_data)
platform_device_register(&at91_cpuidle_device);
diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
index 420e730..6c3fc50 100644
--- a/arch/arm/mach-at91/pm_suspend.S
+++ b/arch/arm/mach-at91/pm_suspend.S
@@ -54,23 +54,21 @@ mode .req r6
.text
-/* void at91_slow_clock(void __iomem *pmc, void __iomem *sdramc,
+/*
+ * void at91_pm_suspend_in_sram(void __iomem *pmc, void __iomem *sdramc,
* void __iomem *ramc1, int memctrl)
+ * @input param:
+ * @r0: the base address of AT91_PMC
+ * @r1: the base address of SDRAM Controller (SDRAM, DDRSDR, or AT91_SYS)
+ * @r2: the base address of second SDRAM Controller or 0 if not present
+ * @r3: the pm information,
+ * least byte: SDRAM controller type
+ * 2nd byte: 1 -> suspend to memory, 0 -> standby
*/
-ENTRY(at91_slow_clock)
+ENTRY(at91_pm_suspend_in_sram)
/* Save registers on stack */
stmfd sp!, {r4 - r12, lr}
- /*
- * Register usage:
- * R0 = Base address of AT91_PMC
- * R1 = Base address of RAM Controller (SDRAM, DDRSDR, or AT91_SYS)
- * R2 = Base address of second RAM Controller or 0 if not present
- * R3 = Memory controller
- * R4 = temporary register
- * R5 = temporary register
- */
-
/* Drain write buffer */
mov tmp1, #0
mcr p15, 0, tmp1, c7, c10, 4
@@ -276,5 +274,5 @@ ram_restored:
.saved_sam9_mdr1:
.word 0
-ENTRY(at91_slow_clock_sz)
- .word .-at91_slow_clock
+ENTRY(at91_pm_suspend_in_sram_sz)
+ .word .-at91_pm_suspend_in_sram
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 09/12] pm: at91: rename function name: at91_slow_clock()-->at91_pm_suspend_sram_fn
@ 2015-01-20 8:24 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:24 UTC (permalink / raw)
To: linux-arm-kernel
As the file name's renaming, rename the file name at91_slow_clock()-->
at91_pm_suspend_sram_fn, rename the function handler's name at the same time.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 23 +++++++++++------------
arch/arm/mach-at91/pm_suspend.S | 26 ++++++++++++--------------
2 files changed, 23 insertions(+), 26 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index a1010f0..98dfeb2 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -137,13 +137,12 @@ int at91_suspend_entering_slow_clock(void)
}
EXPORT_SYMBOL(at91_suspend_entering_slow_clock);
-
-static void (*slow_clock)(void __iomem *pmc, void __iomem *ramc0,
+static void (*at91_suspend_sram_fn)(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
+extern void at91_pm_suspend_in_sram(void __iomem *pmc, void __iomem *ramc0,
void __iomem *ramc1, int memctrl);
-extern u32 at91_slow_clock_sz;
+extern u32 at91_pm_suspend_in_sram_sz;
static void at91_pm_suspend(suspend_state_t state)
{
@@ -152,8 +151,8 @@ static void at91_pm_suspend(suspend_state_t state)
pm_data |= (state == PM_SUSPEND_MEM) ?
(AT91_PM_SLOW_CLOCK << AT91_PM_MODE_OFFSET) : 0;
- slow_clock(at91_pmc_base, at91_ramc_base[0],
- at91_ramc_base[1], pm_data);
+ at91_suspend_sram_fn(at91_pmc_base, at91_ramc_base[0],
+ at91_ramc_base[1], pm_data);
}
static int at91_pm_enter(suspend_state_t state)
@@ -253,17 +252,19 @@ static void __init at91_pm_sram_init(void)
goto put_node;
}
- sram_base = gen_pool_alloc(sram_pool, at91_slow_clock_sz);
+ sram_base = gen_pool_alloc(sram_pool, at91_pm_suspend_in_sram_sz);
if (!sram_base) {
pr_warn("%s: unable to alloc ocram!\n", __func__);
goto put_node;
}
sram_pbase = gen_pool_virt_to_phys(sram_pool, sram_base);
- slow_clock = __arm_ioremap_exec(sram_pbase, at91_slow_clock_sz, false);
+ at91_suspend_sram_fn = __arm_ioremap_exec(sram_pbase,
+ at91_pm_suspend_in_sram_sz, false);
- /* Copy slow_clock handler to SRAM, and call it */
- memcpy(slow_clock, at91_slow_clock, at91_slow_clock_sz);
+ /* Copy pm suspend handler to SRAM, and call it */
+ memcpy(at91_suspend_sram_fn,
+ at91_pm_suspend_in_sram, at91_pm_suspend_in_sram_sz);
put_node:
of_node_put(node);
@@ -312,8 +313,6 @@ static void __init at91_pm_init(void)
at91_pm_sram_init();
- pr_info("AT91: Power Management%s\n", (slow_clock ? " (with slow clock mode)" : ""));
-
if (at91_cpuidle_device.dev.platform_data)
platform_device_register(&at91_cpuidle_device);
diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
index 420e730..6c3fc50 100644
--- a/arch/arm/mach-at91/pm_suspend.S
+++ b/arch/arm/mach-at91/pm_suspend.S
@@ -54,23 +54,21 @@ mode .req r6
.text
-/* void at91_slow_clock(void __iomem *pmc, void __iomem *sdramc,
+/*
+ * void at91_pm_suspend_in_sram(void __iomem *pmc, void __iomem *sdramc,
* void __iomem *ramc1, int memctrl)
+ * @input param:
+ * @r0: the base address of AT91_PMC
+ * @r1: the base address of SDRAM Controller (SDRAM, DDRSDR, or AT91_SYS)
+ * @r2: the base address of second SDRAM Controller or 0 if not present
+ * @r3: the pm information,
+ * least byte: SDRAM controller type
+ * 2nd byte: 1 -> suspend to memory, 0 -> standby
*/
-ENTRY(at91_slow_clock)
+ENTRY(at91_pm_suspend_in_sram)
/* Save registers on stack */
stmfd sp!, {r4 - r12, lr}
- /*
- * Register usage:
- * R0 = Base address of AT91_PMC
- * R1 = Base address of RAM Controller (SDRAM, DDRSDR, or AT91_SYS)
- * R2 = Base address of second RAM Controller or 0 if not present
- * R3 = Memory controller
- * R4 = temporary register
- * R5 = temporary register
- */
-
/* Drain write buffer */
mov tmp1, #0
mcr p15, 0, tmp1, c7, c10, 4
@@ -276,5 +274,5 @@ ram_restored:
.saved_sam9_mdr1:
.word 0
-ENTRY(at91_slow_clock_sz)
- .word .-at91_slow_clock
+ENTRY(at91_pm_suspend_in_sram_sz)
+ .word .-at91_pm_suspend_in_sram
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 10/12] pm: at91: remove the at91_xxx_standby() function definitions in the pm.h
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:24 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:24 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
Because the the suspend to standby mode uses the sram function,
these functions will not used, remove the redundant code.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.h | 110 -----------------------------------------------
1 file changed, 110 deletions(-)
diff --git a/arch/arm/mach-at91/pm.h b/arch/arm/mach-at91/pm.h
index fbfea42..46e18bb 100644
--- a/arch/arm/mach-at91/pm.h
+++ b/arch/arm/mach-at91/pm.h
@@ -12,7 +12,6 @@
#define __ARCH_ARM_MACH_AT91_PM
#include <asm/proc-fns.h>
-
#include <mach/at91_ramc.h>
#define AT91_PM_MEMCTRL_MASK 0x0f
@@ -21,113 +20,4 @@
#define AT91_PM_SLOW_CLOCK 0x01
-/*
- * On all at91 except rm9200 and x40 have the System Controller starts
- * at address 0xffffc000 and has a size of 16KiB.
- *
- * On rm9200 it's start at 0xfffe4000 of 111KiB with non reserved data starting
- * at 0xfffff000
- *
- * Removes the individual definitions of AT91_BASE_SYS and
- * replaces them with a common version at base 0xfffffc000 and size 16KiB
- * and map the same memory space
- */
-#define AT91_BASE_SYS 0xffffc000
-
-#ifdef CONFIG_PM
-extern void at91_pm_set_standby(void (*at91_standby)(void));
-#else
-static inline void at91_pm_set_standby(void (*at91_standby)(void)) { }
-#endif
-
-/*
- * The AT91RM9200 goes into self-refresh mode with this command, and will
- * terminate self-refresh automatically on the next SDRAM access.
- *
- * Self-refresh mode is exited as soon as a memory access is made, but we don't
- * know for sure when that happens. However, we need to restore the low-power
- * mode if it was enabled before going idle. Restoring low-power mode while
- * still in self-refresh is "not recommended", but seems to work.
- */
-
-static inline void at91rm9200_standby(void)
-{
- u32 lpr = at91_ramc_read(0, AT91RM9200_SDRAMC_LPR);
-
- asm volatile(
- "b 1f\n\t"
- ".align 5\n\t"
- "1: mcr p15, 0, %0, c7, c10, 4\n\t"
- " str %0, [%1, %2]\n\t"
- " str %3, [%1, %4]\n\t"
- " mcr p15, 0, %0, c7, c0, 4\n\t"
- " str %5, [%1, %2]"
- :
- : "r" (0), "r" (AT91_BASE_SYS), "r" (AT91RM9200_SDRAMC_LPR),
- "r" (1), "r" (AT91RM9200_SDRAMC_SRR),
- "r" (lpr));
-}
-
-/* We manage both DDRAM/SDRAM controllers, we need more than one value to
- * remember.
- */
-static inline void at91_ddr_standby(void)
-{
- /* Those two values allow us to delay self-refresh activation
- * to the maximum. */
- u32 lpr0, lpr1 = 0;
- u32 saved_lpr0, saved_lpr1 = 0;
-
- if (at91_ramc_base[1]) {
- saved_lpr1 = at91_ramc_read(1, AT91_DDRSDRC_LPR);
- lpr1 = saved_lpr1 & ~AT91_DDRSDRC_LPCB;
- lpr1 |= AT91_DDRSDRC_LPCB_SELF_REFRESH;
- }
-
- saved_lpr0 = at91_ramc_read(0, AT91_DDRSDRC_LPR);
- lpr0 = saved_lpr0 & ~AT91_DDRSDRC_LPCB;
- lpr0 |= AT91_DDRSDRC_LPCB_SELF_REFRESH;
-
- /* self-refresh mode now */
- at91_ramc_write(0, AT91_DDRSDRC_LPR, lpr0);
- if (at91_ramc_base[1])
- at91_ramc_write(1, AT91_DDRSDRC_LPR, lpr1);
-
- cpu_do_idle();
-
- at91_ramc_write(0, AT91_DDRSDRC_LPR, saved_lpr0);
- if (at91_ramc_base[1])
- at91_ramc_write(1, AT91_DDRSDRC_LPR, saved_lpr1);
-}
-
-/* We manage both DDRAM/SDRAM controllers, we need more than one value to
- * remember.
- */
-static inline void at91sam9_sdram_standby(void)
-{
- u32 lpr0, lpr1 = 0;
- u32 saved_lpr0, saved_lpr1 = 0;
-
- if (at91_ramc_base[1]) {
- saved_lpr1 = at91_ramc_read(1, AT91_SDRAMC_LPR);
- lpr1 = saved_lpr1 & ~AT91_SDRAMC_LPCB;
- lpr1 |= AT91_SDRAMC_LPCB_SELF_REFRESH;
- }
-
- saved_lpr0 = at91_ramc_read(0, AT91_SDRAMC_LPR);
- lpr0 = saved_lpr0 & ~AT91_SDRAMC_LPCB;
- lpr0 |= AT91_SDRAMC_LPCB_SELF_REFRESH;
-
- /* self-refresh mode now */
- at91_ramc_write(0, AT91_SDRAMC_LPR, lpr0);
- if (at91_ramc_base[1])
- at91_ramc_write(1, AT91_SDRAMC_LPR, lpr1);
-
- cpu_do_idle();
-
- at91_ramc_write(0, AT91_SDRAMC_LPR, saved_lpr0);
- if (at91_ramc_base[1])
- at91_ramc_write(1, AT91_SDRAMC_LPR, saved_lpr1);
-}
-
#endif
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 10/12] pm: at91: remove the at91_xxx_standby() function definitions in the pm.h
@ 2015-01-20 8:24 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:24 UTC (permalink / raw)
To: linux-arm-kernel
Because the the suspend to standby mode uses the sram function,
these functions will not used, remove the redundant code.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.h | 110 -----------------------------------------------
1 file changed, 110 deletions(-)
diff --git a/arch/arm/mach-at91/pm.h b/arch/arm/mach-at91/pm.h
index fbfea42..46e18bb 100644
--- a/arch/arm/mach-at91/pm.h
+++ b/arch/arm/mach-at91/pm.h
@@ -12,7 +12,6 @@
#define __ARCH_ARM_MACH_AT91_PM
#include <asm/proc-fns.h>
-
#include <mach/at91_ramc.h>
#define AT91_PM_MEMCTRL_MASK 0x0f
@@ -21,113 +20,4 @@
#define AT91_PM_SLOW_CLOCK 0x01
-/*
- * On all at91 except rm9200 and x40 have the System Controller starts
- * at address 0xffffc000 and has a size of 16KiB.
- *
- * On rm9200 it's start at 0xfffe4000 of 111KiB with non reserved data starting
- * at 0xfffff000
- *
- * Removes the individual definitions of AT91_BASE_SYS and
- * replaces them with a common version at base 0xfffffc000 and size 16KiB
- * and map the same memory space
- */
-#define AT91_BASE_SYS 0xffffc000
-
-#ifdef CONFIG_PM
-extern void at91_pm_set_standby(void (*at91_standby)(void));
-#else
-static inline void at91_pm_set_standby(void (*at91_standby)(void)) { }
-#endif
-
-/*
- * The AT91RM9200 goes into self-refresh mode with this command, and will
- * terminate self-refresh automatically on the next SDRAM access.
- *
- * Self-refresh mode is exited as soon as a memory access is made, but we don't
- * know for sure when that happens. However, we need to restore the low-power
- * mode if it was enabled before going idle. Restoring low-power mode while
- * still in self-refresh is "not recommended", but seems to work.
- */
-
-static inline void at91rm9200_standby(void)
-{
- u32 lpr = at91_ramc_read(0, AT91RM9200_SDRAMC_LPR);
-
- asm volatile(
- "b 1f\n\t"
- ".align 5\n\t"
- "1: mcr p15, 0, %0, c7, c10, 4\n\t"
- " str %0, [%1, %2]\n\t"
- " str %3, [%1, %4]\n\t"
- " mcr p15, 0, %0, c7, c0, 4\n\t"
- " str %5, [%1, %2]"
- :
- : "r" (0), "r" (AT91_BASE_SYS), "r" (AT91RM9200_SDRAMC_LPR),
- "r" (1), "r" (AT91RM9200_SDRAMC_SRR),
- "r" (lpr));
-}
-
-/* We manage both DDRAM/SDRAM controllers, we need more than one value to
- * remember.
- */
-static inline void at91_ddr_standby(void)
-{
- /* Those two values allow us to delay self-refresh activation
- * to the maximum. */
- u32 lpr0, lpr1 = 0;
- u32 saved_lpr0, saved_lpr1 = 0;
-
- if (at91_ramc_base[1]) {
- saved_lpr1 = at91_ramc_read(1, AT91_DDRSDRC_LPR);
- lpr1 = saved_lpr1 & ~AT91_DDRSDRC_LPCB;
- lpr1 |= AT91_DDRSDRC_LPCB_SELF_REFRESH;
- }
-
- saved_lpr0 = at91_ramc_read(0, AT91_DDRSDRC_LPR);
- lpr0 = saved_lpr0 & ~AT91_DDRSDRC_LPCB;
- lpr0 |= AT91_DDRSDRC_LPCB_SELF_REFRESH;
-
- /* self-refresh mode now */
- at91_ramc_write(0, AT91_DDRSDRC_LPR, lpr0);
- if (at91_ramc_base[1])
- at91_ramc_write(1, AT91_DDRSDRC_LPR, lpr1);
-
- cpu_do_idle();
-
- at91_ramc_write(0, AT91_DDRSDRC_LPR, saved_lpr0);
- if (at91_ramc_base[1])
- at91_ramc_write(1, AT91_DDRSDRC_LPR, saved_lpr1);
-}
-
-/* We manage both DDRAM/SDRAM controllers, we need more than one value to
- * remember.
- */
-static inline void at91sam9_sdram_standby(void)
-{
- u32 lpr0, lpr1 = 0;
- u32 saved_lpr0, saved_lpr1 = 0;
-
- if (at91_ramc_base[1]) {
- saved_lpr1 = at91_ramc_read(1, AT91_SDRAMC_LPR);
- lpr1 = saved_lpr1 & ~AT91_SDRAMC_LPCB;
- lpr1 |= AT91_SDRAMC_LPCB_SELF_REFRESH;
- }
-
- saved_lpr0 = at91_ramc_read(0, AT91_SDRAMC_LPR);
- lpr0 = saved_lpr0 & ~AT91_SDRAMC_LPCB;
- lpr0 |= AT91_SDRAMC_LPCB_SELF_REFRESH;
-
- /* self-refresh mode now */
- at91_ramc_write(0, AT91_SDRAMC_LPR, lpr0);
- if (at91_ramc_base[1])
- at91_ramc_write(1, AT91_SDRAMC_LPR, lpr1);
-
- cpu_do_idle();
-
- at91_ramc_write(0, AT91_SDRAMC_LPR, saved_lpr0);
- if (at91_ramc_base[1])
- at91_ramc_write(1, AT91_SDRAMC_LPR, saved_lpr1);
-}
-
#endif
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 11/12] pm: at91: remove the struct ramc_ids .data at91_xxx_standby members
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:25 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:25 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
Because the at91_xxx_standby function is removed,
remove the struct ramc_ids .data members code.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 19 ++++---------------
1 file changed, 4 insertions(+), 15 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 98dfeb2..07b723f 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -271,10 +271,10 @@ put_node:
}
static struct of_device_id ramc_ids[] = {
- { .compatible = "atmel,at91rm9200-sdramc", .data = at91rm9200_standby },
- { .compatible = "atmel,at91sam9260-sdramc", .data = at91sam9_sdram_standby },
- { .compatible = "atmel,at91sam9g45-ddramc", .data = at91_ddr_standby },
- { .compatible = "atmel,sama5d3-ddramc", .data = at91_ddr_standby },
+ { .compatible = "atmel,at91rm9200-sdramc" },
+ { .compatible = "atmel,at91sam9260-sdramc" },
+ { .compatible = "atmel,at91sam9g45-ddramc" },
+ { .compatible = "atmel,sama5d3-ddramc" },
{ /*sentinel*/ }
};
@@ -283,28 +283,17 @@ static void at91_dt_ramc(void)
struct device_node *np;
const struct of_device_id *of_id;
int idx = 0;
- const void *standby = NULL;
for_each_matching_node_and_match(np, ramc_ids, &of_id) {
at91_ramc_base[idx] = of_iomap(np, 0);
if (!at91_ramc_base[idx])
panic(pr_fmt("unable to map ramc[%d] cpu registers\n"), idx);
- if (!standby)
- standby = of_id->data;
-
idx++;
}
if (!idx)
panic(pr_fmt("unable to find compatible ram controller node in dtb\n"));
-
- if (!standby) {
- pr_warn("ramc no standby function available\n");
- return;
- }
-
- at91_pm_set_standby(standby);
}
static void __init at91_pm_init(void)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 11/12] pm: at91: remove the struct ramc_ids .data at91_xxx_standby members
@ 2015-01-20 8:25 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:25 UTC (permalink / raw)
To: linux-arm-kernel
Because the at91_xxx_standby function is removed,
remove the struct ramc_ids .data members code.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 19 ++++---------------
1 file changed, 4 insertions(+), 15 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 98dfeb2..07b723f 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -271,10 +271,10 @@ put_node:
}
static struct of_device_id ramc_ids[] = {
- { .compatible = "atmel,at91rm9200-sdramc", .data = at91rm9200_standby },
- { .compatible = "atmel,at91sam9260-sdramc", .data = at91sam9_sdram_standby },
- { .compatible = "atmel,at91sam9g45-ddramc", .data = at91_ddr_standby },
- { .compatible = "atmel,sama5d3-ddramc", .data = at91_ddr_standby },
+ { .compatible = "atmel,at91rm9200-sdramc" },
+ { .compatible = "atmel,at91sam9260-sdramc" },
+ { .compatible = "atmel,at91sam9g45-ddramc" },
+ { .compatible = "atmel,sama5d3-ddramc" },
{ /*sentinel*/ }
};
@@ -283,28 +283,17 @@ static void at91_dt_ramc(void)
struct device_node *np;
const struct of_device_id *of_id;
int idx = 0;
- const void *standby = NULL;
for_each_matching_node_and_match(np, ramc_ids, &of_id) {
at91_ramc_base[idx] = of_iomap(np, 0);
if (!at91_ramc_base[idx])
panic(pr_fmt("unable to map ramc[%d] cpu registers\n"), idx);
- if (!standby)
- standby = of_id->data;
-
idx++;
}
if (!idx)
panic(pr_fmt("unable to find compatible ram controller node in dtb\n"));
-
- if (!standby) {
- pr_warn("ramc no standby function available\n");
- return;
- }
-
- at91_pm_set_standby(standby);
}
static void __init at91_pm_init(void)
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 12/12] pm: at91: amend the pm_suspend entry for at91_cpuidle_device
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-20 8:26 ` Wenyou Yang
-1 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:26 UTC (permalink / raw)
To: nicolas.ferre, linux
Cc: linux-arm-kernel, linux-kernel, alexandre.belloni,
sylvain.rochet, peda, wenyou.yang
Because the at91_xxx_standby() function is substitued by the at91_pm_suspend(),
the pm_suspend entry for at91_cpuidle_device changes as well.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 17 ++++++++++-------
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 07b723f..041adad 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -48,8 +48,6 @@ static struct {
int memctrl;
} at91_pm_data;
-static void (*at91_pm_standby)(void);
-
static int at91_pm_valid_state(suspend_state_t state)
{
switch (state) {
@@ -218,12 +216,15 @@ static struct platform_device at91_cpuidle_device = {
.name = "cpuidle-at91",
};
-void at91_pm_set_standby(void (*at91_standby)(void))
+static void at91_cpuidle_suspend(void)
{
- if (at91_standby) {
- at91_cpuidle_device.dev.platform_data = at91_standby;
- at91_pm_standby = at91_standby;
- }
+ at91_pm_suspend(PM_SUSPEND_ON);
+}
+
+static void at91_cpuidle_set_suspend(void (*at91_suspend)(void))
+{
+ if (at91_suspend)
+ at91_cpuidle_device.dev.platform_data = at91_suspend;
}
static void __init at91_pm_sram_init(void)
@@ -302,6 +303,8 @@ static void __init at91_pm_init(void)
at91_pm_sram_init();
+ at91_cpuidle_set_suspend(at91_cpuidle_suspend);
+
if (at91_cpuidle_device.dev.platform_data)
platform_device_register(&at91_cpuidle_device);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* [PATCH 12/12] pm: at91: amend the pm_suspend entry for at91_cpuidle_device
@ 2015-01-20 8:26 ` Wenyou Yang
0 siblings, 0 replies; 56+ messages in thread
From: Wenyou Yang @ 2015-01-20 8:26 UTC (permalink / raw)
To: linux-arm-kernel
Because the at91_xxx_standby() function is substitued by the at91_pm_suspend(),
the pm_suspend entry for at91_cpuidle_device changes as well.
Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
---
arch/arm/mach-at91/pm.c | 17 ++++++++++-------
1 file changed, 10 insertions(+), 7 deletions(-)
diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
index 07b723f..041adad 100644
--- a/arch/arm/mach-at91/pm.c
+++ b/arch/arm/mach-at91/pm.c
@@ -48,8 +48,6 @@ static struct {
int memctrl;
} at91_pm_data;
-static void (*at91_pm_standby)(void);
-
static int at91_pm_valid_state(suspend_state_t state)
{
switch (state) {
@@ -218,12 +216,15 @@ static struct platform_device at91_cpuidle_device = {
.name = "cpuidle-at91",
};
-void at91_pm_set_standby(void (*at91_standby)(void))
+static void at91_cpuidle_suspend(void)
{
- if (at91_standby) {
- at91_cpuidle_device.dev.platform_data = at91_standby;
- at91_pm_standby = at91_standby;
- }
+ at91_pm_suspend(PM_SUSPEND_ON);
+}
+
+static void at91_cpuidle_set_suspend(void (*at91_suspend)(void))
+{
+ if (at91_suspend)
+ at91_cpuidle_device.dev.platform_data = at91_suspend;
}
static void __init at91_pm_sram_init(void)
@@ -302,6 +303,8 @@ static void __init at91_pm_init(void)
at91_pm_sram_init();
+ at91_cpuidle_set_suspend(at91_cpuidle_suspend);
+
if (at91_cpuidle_device.dev.platform_data)
platform_device_register(&at91_cpuidle_device);
--
1.7.9.5
^ permalink raw reply related [flat|nested] 56+ messages in thread
* Re: [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91
2015-01-20 8:16 ` Wenyou Yang
@ 2015-01-23 10:24 ` Alexandre Belloni
-1 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 10:24 UTC (permalink / raw)
To: Wenyou Yang
Cc: nicolas.ferre, linux, linux-arm-kernel, linux-kernel,
sylvain.rochet, peda
On 20/01/2015 at 16:16:58 +0800, Wenyou Yang wrote :
> Because the CONFIG_AT91_SLOW_CLOCK will be removed
> to simply the PM config, so move select SRAM.
>
> Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> ---
> arch/arm/mach-at91/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
> index 56dcede..7d5e201 100644
> --- a/arch/arm/mach-at91/Kconfig
> +++ b/arch/arm/mach-at91/Kconfig
> @@ -6,6 +6,7 @@ config ARCH_AT91
> select PINCTRL
> select PINCTRL_AT91
> select USE_OF
> + select SRAM if PM
> bool
>
> menuconfig SOC_SAM_V7
> @@ -128,7 +129,6 @@ comment "AT91 Feature Selections"
>
> config AT91_SLOW_CLOCK
> bool "Suspend-to-RAM disables main oscillator"
> - select SRAM
> depends on SUSPEND
> help
> Select this if you want Suspend-to-RAM to save the most power
> --
> 1.7.9.5
>
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91
@ 2015-01-23 10:24 ` Alexandre Belloni
0 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 10:24 UTC (permalink / raw)
To: linux-arm-kernel
On 20/01/2015 at 16:16:58 +0800, Wenyou Yang wrote :
> Because the CONFIG_AT91_SLOW_CLOCK will be removed
> to simply the PM config, so move select SRAM.
>
> Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> ---
> arch/arm/mach-at91/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
> index 56dcede..7d5e201 100644
> --- a/arch/arm/mach-at91/Kconfig
> +++ b/arch/arm/mach-at91/Kconfig
> @@ -6,6 +6,7 @@ config ARCH_AT91
> select PINCTRL
> select PINCTRL_AT91
> select USE_OF
> + select SRAM if PM
> bool
>
> menuconfig SOC_SAM_V7
> @@ -128,7 +129,6 @@ comment "AT91 Feature Selections"
>
> config AT91_SLOW_CLOCK
> bool "Suspend-to-RAM disables main oscillator"
> - select SRAM
> depends on SUSPEND
> help
> Select this if you want Suspend-to-RAM to save the most power
> --
> 1.7.9.5
>
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-20 8:17 ` Wenyou Yang
@ 2015-01-23 10:30 ` Alexandre Belloni
-1 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 10:30 UTC (permalink / raw)
To: Wenyou Yang
Cc: nicolas.ferre, linux, linux-arm-kernel, linux-kernel,
sylvain.rochet, peda
On 20/01/2015 at 16:17:00 +0800, Wenyou Yang wrote :
> To simply the PM code, the suspend to standby mode uses the same sram function
> as the suspend to memory mode, running in the internal SRAM,
> instead of the respective code for each mode.
>
> But for the suspend to standby mode, the master clock doesn't
> switch to the slow clock, and the main oscillator doesn't
> turn off as well.
>
> Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
That looks quite better, thanks.
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> ---
> arch/arm/mach-at91/pm.c | 81 ++++++++++++++++---------------------
> arch/arm/mach-at91/pm.h | 6 +++
> arch/arm/mach-at91/pm_slowclock.S | 18 +++++++++
> 3 files changed, 59 insertions(+), 46 deletions(-)
>
> diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> index 691e6db..a1010f0 100644
> --- a/arch/arm/mach-at91/pm.c
> +++ b/arch/arm/mach-at91/pm.c
> @@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
> void __iomem *ramc1, int memctrl);
> extern u32 at91_slow_clock_sz;
>
> +static void at91_pm_suspend(suspend_state_t state)
> +{
> + unsigned int pm_data = at91_pm_data.memctrl;
> +
> + pm_data |= (state == PM_SUSPEND_MEM) ?
> + (AT91_PM_SLOW_CLOCK << AT91_PM_MODE_OFFSET) : 0;
> +
> + slow_clock(at91_pmc_base, at91_ramc_base[0],
> + at91_ramc_base[1], pm_data);
> +}
> +
> static int at91_pm_enter(suspend_state_t state)
> {
> at91_pinctrl_gpio_suspend();
>
> switch (state) {
> + /*
> + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> + * drivers must suspend more deeply, the master clock switches
> + * to the clk32k and turns off the main oscillator
> + *
> + * STANDBY mode has *all* drivers suspended; ignores irqs not
> + * marked as 'wakeup' event sources; and reduces DRAM power.
> + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> + * nothing fancy done with main or cpu clocks.
> + */
> + case PM_SUSPEND_MEM:
> + case PM_SUSPEND_STANDBY:
> /*
> - * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> - * drivers must suspend more deeply: only the master clock
> - * controller may be using the main oscillator.
> + * Ensure that clocks are in a valid state.
> */
> - case PM_SUSPEND_MEM:
> - /*
> - * Ensure that clocks are in a valid state.
> - */
> - if (!at91_pm_verify_clocks())
> - goto error;
> -
> - /*
> - * Enter slow clock mode by switching over to clk32k and
> - * turning off the main oscillator; reverse on wakeup.
> - */
> - if (slow_clock) {
> - slow_clock(at91_pmc_base, at91_ramc_base[0],
> - at91_ramc_base[1],
> - at91_pm_data.memctrl);
> - break;
> - } else {
> - pr_info("AT91: PM - no slow clock mode enabled ...\n");
> - /* FALLTHROUGH leaving master clock alone */
> - }
> + if (!at91_pm_verify_clocks())
> + goto error;
>
> - /*
> - * STANDBY mode has *all* drivers suspended; ignores irqs not
> - * marked as 'wakeup' event sources; and reduces DRAM power.
> - * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> - * nothing fancy done with main or cpu clocks.
> - */
> - case PM_SUSPEND_STANDBY:
> - /*
> - * NOTE: the Wait-for-Interrupt instruction needs to be
> - * in icache so no SDRAM accesses are needed until the
> - * wakeup IRQ occurs and self-refresh is terminated.
> - * For ARM 926 based chips, this requirement is weaker
> - * as at91sam9 can access a RAM in self-refresh mode.
> - */
> - if (at91_pm_standby)
> - at91_pm_standby();
> - break;
> + at91_pm_suspend(state);
>
> - case PM_SUSPEND_ON:
> - cpu_do_idle();
> - break;
> + break;
>
> - default:
> - pr_debug("AT91: PM - bogus suspend state %d\n", state);
> - goto error;
> + case PM_SUSPEND_ON:
> + cpu_do_idle();
> + break;
> +
> + default:
> + pr_debug("AT91: PM - bogus suspend state %d\n", state);
> + goto error;
> }
>
> error:
> diff --git a/arch/arm/mach-at91/pm.h b/arch/arm/mach-at91/pm.h
> index 7664d39..fbfea42 100644
> --- a/arch/arm/mach-at91/pm.h
> +++ b/arch/arm/mach-at91/pm.h
> @@ -15,6 +15,12 @@
>
> #include <mach/at91_ramc.h>
>
> +#define AT91_PM_MEMCTRL_MASK 0x0f
> +#define AT91_PM_MODE_OFFSET 4
> +#define AT91_PM_MODE_MASK 0x0f
> +
> +#define AT91_PM_SLOW_CLOCK 0x01
> +
> /*
> * On all at91 except rm9200 and x40 have the System Controller starts
> * at address 0xffffc000 and has a size of 16KiB.
> diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
> index 634d819..92d7e63 100644
> --- a/arch/arm/mach-at91/pm_slowclock.S
> +++ b/arch/arm/mach-at91/pm_slowclock.S
> @@ -15,12 +15,15 @@
> #include <linux/clk/at91_pmc.h>
> #include <mach/at91_ramc.h>
>
> +#include "pm.h"
> +
> pmc .req r0
> sdramc .req r1
> ramc1 .req r2
> memctrl .req r3
> tmp1 .req r4
> tmp2 .req r5
> +mode .req r6
>
> /*
> * Wait until master clock is ready (after switching master clock source)
> @@ -72,6 +75,13 @@ ENTRY(at91_slow_clock)
> mov tmp1, #0
> mcr p15, 0, tmp1, c7, c10, 4
>
> + mov tmp1, memctrl
> + mov tmp2, tmp1, lsr#AT91_PM_MODE_OFFSET
> + and mode, tmp2, #AT91_PM_MODE_MASK
> +
> + mov tmp1, memctrl
> + and memctrl, tmp1, #AT91_PM_MEMCTRL_MASK
> +
> cmp memctrl, #AT91_MEMCTRL_MC
> bne ddr_sr_enable
>
> @@ -144,6 +154,9 @@ sdr_sr_enable:
> str tmp1, [sdramc, #AT91_SDRAMC_LPR]
>
> sdr_sr_done:
> + tst mode, #AT91_PM_SLOW_CLOCK
> + beq skip_disable_clock
> +
> /* Save Master clock setting */
> ldr tmp1, [pmc, #AT91_PMC_MCKR]
> str tmp1, .saved_mckr
> @@ -169,9 +182,13 @@ sdr_sr_done:
> bic tmp1, tmp1, #AT91_PMC_MOSCEN
> str tmp1, [pmc, #AT91_CKGR_MOR]
>
> +skip_disable_clock:
> /* Wait for interrupt */
> mcr p15, 0, tmp1, c7, c0, 4
>
> + tst mode, #AT91_PM_SLOW_CLOCK
> + beq skip_enable_clock
> +
> /* Turn on the main oscillator */
> ldr tmp1, [pmc, #AT91_CKGR_MOR]
> orr tmp1, tmp1, #AT91_PMC_MOSCEN
> @@ -199,6 +216,7 @@ sdr_sr_done:
>
> wait_mckrdy
>
> +skip_enable_clock:
> /*
> * at91rm9200 Memory controller
> * Do nothing - self-refresh is automatically disabled.
> --
> 1.7.9.5
>
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-23 10:30 ` Alexandre Belloni
0 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 10:30 UTC (permalink / raw)
To: linux-arm-kernel
On 20/01/2015 at 16:17:00 +0800, Wenyou Yang wrote :
> To simply the PM code, the suspend to standby mode uses the same sram function
> as the suspend to memory mode, running in the internal SRAM,
> instead of the respective code for each mode.
>
> But for the suspend to standby mode, the master clock doesn't
> switch to the slow clock, and the main oscillator doesn't
> turn off as well.
>
> Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
That looks quite better, thanks.
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
> ---
> arch/arm/mach-at91/pm.c | 81 ++++++++++++++++---------------------
> arch/arm/mach-at91/pm.h | 6 +++
> arch/arm/mach-at91/pm_slowclock.S | 18 +++++++++
> 3 files changed, 59 insertions(+), 46 deletions(-)
>
> diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> index 691e6db..a1010f0 100644
> --- a/arch/arm/mach-at91/pm.c
> +++ b/arch/arm/mach-at91/pm.c
> @@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
> void __iomem *ramc1, int memctrl);
> extern u32 at91_slow_clock_sz;
>
> +static void at91_pm_suspend(suspend_state_t state)
> +{
> + unsigned int pm_data = at91_pm_data.memctrl;
> +
> + pm_data |= (state == PM_SUSPEND_MEM) ?
> + (AT91_PM_SLOW_CLOCK << AT91_PM_MODE_OFFSET) : 0;
> +
> + slow_clock(at91_pmc_base, at91_ramc_base[0],
> + at91_ramc_base[1], pm_data);
> +}
> +
> static int at91_pm_enter(suspend_state_t state)
> {
> at91_pinctrl_gpio_suspend();
>
> switch (state) {
> + /*
> + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> + * drivers must suspend more deeply, the master clock switches
> + * to the clk32k and turns off the main oscillator
> + *
> + * STANDBY mode has *all* drivers suspended; ignores irqs not
> + * marked as 'wakeup' event sources; and reduces DRAM power.
> + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> + * nothing fancy done with main or cpu clocks.
> + */
> + case PM_SUSPEND_MEM:
> + case PM_SUSPEND_STANDBY:
> /*
> - * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> - * drivers must suspend more deeply: only the master clock
> - * controller may be using the main oscillator.
> + * Ensure that clocks are in a valid state.
> */
> - case PM_SUSPEND_MEM:
> - /*
> - * Ensure that clocks are in a valid state.
> - */
> - if (!at91_pm_verify_clocks())
> - goto error;
> -
> - /*
> - * Enter slow clock mode by switching over to clk32k and
> - * turning off the main oscillator; reverse on wakeup.
> - */
> - if (slow_clock) {
> - slow_clock(at91_pmc_base, at91_ramc_base[0],
> - at91_ramc_base[1],
> - at91_pm_data.memctrl);
> - break;
> - } else {
> - pr_info("AT91: PM - no slow clock mode enabled ...\n");
> - /* FALLTHROUGH leaving master clock alone */
> - }
> + if (!at91_pm_verify_clocks())
> + goto error;
>
> - /*
> - * STANDBY mode has *all* drivers suspended; ignores irqs not
> - * marked as 'wakeup' event sources; and reduces DRAM power.
> - * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> - * nothing fancy done with main or cpu clocks.
> - */
> - case PM_SUSPEND_STANDBY:
> - /*
> - * NOTE: the Wait-for-Interrupt instruction needs to be
> - * in icache so no SDRAM accesses are needed until the
> - * wakeup IRQ occurs and self-refresh is terminated.
> - * For ARM 926 based chips, this requirement is weaker
> - * as at91sam9 can access a RAM in self-refresh mode.
> - */
> - if (at91_pm_standby)
> - at91_pm_standby();
> - break;
> + at91_pm_suspend(state);
>
> - case PM_SUSPEND_ON:
> - cpu_do_idle();
> - break;
> + break;
>
> - default:
> - pr_debug("AT91: PM - bogus suspend state %d\n", state);
> - goto error;
> + case PM_SUSPEND_ON:
> + cpu_do_idle();
> + break;
> +
> + default:
> + pr_debug("AT91: PM - bogus suspend state %d\n", state);
> + goto error;
> }
>
> error:
> diff --git a/arch/arm/mach-at91/pm.h b/arch/arm/mach-at91/pm.h
> index 7664d39..fbfea42 100644
> --- a/arch/arm/mach-at91/pm.h
> +++ b/arch/arm/mach-at91/pm.h
> @@ -15,6 +15,12 @@
>
> #include <mach/at91_ramc.h>
>
> +#define AT91_PM_MEMCTRL_MASK 0x0f
> +#define AT91_PM_MODE_OFFSET 4
> +#define AT91_PM_MODE_MASK 0x0f
> +
> +#define AT91_PM_SLOW_CLOCK 0x01
> +
> /*
> * On all at91 except rm9200 and x40 have the System Controller starts
> * at address 0xffffc000 and has a size of 16KiB.
> diff --git a/arch/arm/mach-at91/pm_slowclock.S b/arch/arm/mach-at91/pm_slowclock.S
> index 634d819..92d7e63 100644
> --- a/arch/arm/mach-at91/pm_slowclock.S
> +++ b/arch/arm/mach-at91/pm_slowclock.S
> @@ -15,12 +15,15 @@
> #include <linux/clk/at91_pmc.h>
> #include <mach/at91_ramc.h>
>
> +#include "pm.h"
> +
> pmc .req r0
> sdramc .req r1
> ramc1 .req r2
> memctrl .req r3
> tmp1 .req r4
> tmp2 .req r5
> +mode .req r6
>
> /*
> * Wait until master clock is ready (after switching master clock source)
> @@ -72,6 +75,13 @@ ENTRY(at91_slow_clock)
> mov tmp1, #0
> mcr p15, 0, tmp1, c7, c10, 4
>
> + mov tmp1, memctrl
> + mov tmp2, tmp1, lsr#AT91_PM_MODE_OFFSET
> + and mode, tmp2, #AT91_PM_MODE_MASK
> +
> + mov tmp1, memctrl
> + and memctrl, tmp1, #AT91_PM_MEMCTRL_MASK
> +
> cmp memctrl, #AT91_MEMCTRL_MC
> bne ddr_sr_enable
>
> @@ -144,6 +154,9 @@ sdr_sr_enable:
> str tmp1, [sdramc, #AT91_SDRAMC_LPR]
>
> sdr_sr_done:
> + tst mode, #AT91_PM_SLOW_CLOCK
> + beq skip_disable_clock
> +
> /* Save Master clock setting */
> ldr tmp1, [pmc, #AT91_PMC_MCKR]
> str tmp1, .saved_mckr
> @@ -169,9 +182,13 @@ sdr_sr_done:
> bic tmp1, tmp1, #AT91_PMC_MOSCEN
> str tmp1, [pmc, #AT91_CKGR_MOR]
>
> +skip_disable_clock:
> /* Wait for interrupt */
> mcr p15, 0, tmp1, c7, c0, 4
>
> + tst mode, #AT91_PM_SLOW_CLOCK
> + beq skip_enable_clock
> +
> /* Turn on the main oscillator */
> ldr tmp1, [pmc, #AT91_CKGR_MOR]
> orr tmp1, tmp1, #AT91_PMC_MOSCEN
> @@ -199,6 +216,7 @@ sdr_sr_done:
>
> wait_mckrdy
>
> +skip_enable_clock:
> /*
> * at91rm9200 Memory controller
> * Do nothing - self-refresh is automatically disabled.
> --
> 1.7.9.5
>
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-20 8:17 ` Wenyou Yang
@ 2015-01-23 16:50 ` Sylvain Rochet
-1 siblings, 0 replies; 56+ messages in thread
From: Sylvain Rochet @ 2015-01-23 16:50 UTC (permalink / raw)
To: Wenyou Yang
Cc: nicolas.ferre, linux, linux-kernel, alexandre.belloni, peda,
linux-arm-kernel
Hello Wenyou,
On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
>
> diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> index 691e6db..a1010f0 100644
> --- a/arch/arm/mach-at91/pm.c
> +++ b/arch/arm/mach-at91/pm.c
> @@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
> void __iomem *ramc1, int memctrl);
> extern u32 at91_slow_clock_sz;
>
> +static void at91_pm_suspend(suspend_state_t state)
> +{
(...)
> + slow_clock(at91_pmc_base, at91_ramc_base[0],
> + at91_ramc_base[1], pm_data);
> +}
> - if (slow_clock) {
> - slow_clock(at91_pmc_base, at91_ramc_base[0],
> - at91_ramc_base[1],
> - at91_pm_data.memctrl);
(...)
> + at91_pm_suspend(state);
By doing that you removed the condition "if (slow_clock)".
But slow_clock can still be NULL, see commit d2e4679, there are multiple
reasons which ends up with a NULL slow_clock.
Sylvain
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-23 16:50 ` Sylvain Rochet
0 siblings, 0 replies; 56+ messages in thread
From: Sylvain Rochet @ 2015-01-23 16:50 UTC (permalink / raw)
To: linux-arm-kernel
Hello Wenyou,
On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
>
> diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> index 691e6db..a1010f0 100644
> --- a/arch/arm/mach-at91/pm.c
> +++ b/arch/arm/mach-at91/pm.c
> @@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
> void __iomem *ramc1, int memctrl);
> extern u32 at91_slow_clock_sz;
>
> +static void at91_pm_suspend(suspend_state_t state)
> +{
(...)
> + slow_clock(at91_pmc_base, at91_ramc_base[0],
> + at91_ramc_base[1], pm_data);
> +}
> - if (slow_clock) {
> - slow_clock(at91_pmc_base, at91_ramc_base[0],
> - at91_ramc_base[1],
> - at91_pm_data.memctrl);
(...)
> + at91_pm_suspend(state);
By doing that you removed the condition "if (slow_clock)".
But slow_clock can still be NULL, see commit d2e4679, there are multiple
reasons which ends up with a NULL slow_clock.
Sylvain
^ permalink raw reply [flat|nested] 56+ messages in thread
* Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-20 8:17 ` Wenyou Yang
@ 2015-01-23 17:32 ` Sylvain Rochet
-1 siblings, 0 replies; 56+ messages in thread
From: Sylvain Rochet @ 2015-01-23 17:32 UTC (permalink / raw)
To: Wenyou Yang
Cc: nicolas.ferre, linux, linux-kernel, alexandre.belloni, peda,
linux-arm-kernel
Hello Wenyou,
On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
>
> diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> index 691e6db..a1010f0 100644
> --- a/arch/arm/mach-at91/pm.c
> +++ b/arch/arm/mach-at91/pm.c
(...)
> static int at91_pm_enter(suspend_state_t state)
> {
> at91_pinctrl_gpio_suspend();
>
> switch (state) {
> + /*
> + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> + * drivers must suspend more deeply, the master clock switches
> + * to the clk32k and turns off the main oscillator
> + *
> + * STANDBY mode has *all* drivers suspended; ignores irqs not
> + * marked as 'wakeup' event sources; and reduces DRAM power.
> + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> + * nothing fancy done with main or cpu clocks.
> + */
> + case PM_SUSPEND_MEM:
> + case PM_SUSPEND_STANDBY:
(...)
> - case PM_SUSPEND_MEM:
> - /*
> - * Ensure that clocks are in a valid state.
> - */
> - if (!at91_pm_verify_clocks())
> - goto error;
(...)
> + if (!at91_pm_verify_clocks())
> + goto error;
>
(...)
> - case PM_SUSPEND_STANDBY:
> - /*
> - * NOTE: the Wait-for-Interrupt instruction needs to be
By doing that at91_pm_verify_clocks() is now called for both MEM and
STANDBY targets.
In my opinion this function is misnamed and should be called
at91_pm_verify_clocks_for_slow_clock_mode(). This function actually
checks if we can safely switch to slow clock mode, if some peripherals
are still using the master clock, we abort the suspend because we can't
suspend in good condition. Hard unclocking peripherals which ask for a
soft stop, like USB controllers, is something we should avoid doing.
This function checks if USB PLL and PLL B are stopped, if PCK0..PCK3 are
stopped too (or just using the 32k clock). If all drivers suspended
correctly this is the state we expect and we can suspend in a deep
state.
Not this is currently not the case in linux-next, suspend/resume support
to all Atmel USB drivers (ehci-atmel,ohci-at91,atmel_usba,at91_udc) are
in my series:
[PATCHv7 0/6] USB: host: Atmel OHCI and EHCI drivers improvements
<1421761144-11767-1-git-send-email-sylvain.rochet@finsecur.com>
[PATCHv6 0/5] USB: gadget: atmel_usba_udc: Driver improvements
<1421945805-31129-1-git-send-email-sylvain.rochet@finsecur.com>
We are not going to change any clock for STANDBY target, there is no
clock to check, so we don't need to call at91_pm_verify_clocks() for
this target.
Sylvain
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-23 17:32 ` Sylvain Rochet
0 siblings, 0 replies; 56+ messages in thread
From: Sylvain Rochet @ 2015-01-23 17:32 UTC (permalink / raw)
To: linux-arm-kernel
Hello Wenyou,
On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
>
> diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> index 691e6db..a1010f0 100644
> --- a/arch/arm/mach-at91/pm.c
> +++ b/arch/arm/mach-at91/pm.c
(...)
> static int at91_pm_enter(suspend_state_t state)
> {
> at91_pinctrl_gpio_suspend();
>
> switch (state) {
> + /*
> + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> + * drivers must suspend more deeply, the master clock switches
> + * to the clk32k and turns off the main oscillator
> + *
> + * STANDBY mode has *all* drivers suspended; ignores irqs not
> + * marked as 'wakeup' event sources; and reduces DRAM power.
> + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> + * nothing fancy done with main or cpu clocks.
> + */
> + case PM_SUSPEND_MEM:
> + case PM_SUSPEND_STANDBY:
(...)
> - case PM_SUSPEND_MEM:
> - /*
> - * Ensure that clocks are in a valid state.
> - */
> - if (!at91_pm_verify_clocks())
> - goto error;
(...)
> + if (!at91_pm_verify_clocks())
> + goto error;
>
(...)
> - case PM_SUSPEND_STANDBY:
> - /*
> - * NOTE: the Wait-for-Interrupt instruction needs to be
By doing that at91_pm_verify_clocks() is now called for both MEM and
STANDBY targets.
In my opinion this function is misnamed and should be called
at91_pm_verify_clocks_for_slow_clock_mode(). This function actually
checks if we can safely switch to slow clock mode, if some peripherals
are still using the master clock, we abort the suspend because we can't
suspend in good condition. Hard unclocking peripherals which ask for a
soft stop, like USB controllers, is something we should avoid doing.
This function checks if USB PLL and PLL B are stopped, if PCK0..PCK3 are
stopped too (or just using the 32k clock). If all drivers suspended
correctly this is the state we expect and we can suspend in a deep
state.
Not this is currently not the case in linux-next, suspend/resume support
to all Atmel USB drivers (ehci-atmel,ohci-at91,atmel_usba,at91_udc) are
in my series:
[PATCHv7 0/6] USB: host: Atmel OHCI and EHCI drivers improvements
<1421761144-11767-1-git-send-email-sylvain.rochet@finsecur.com>
[PATCHv6 0/5] USB: gadget: atmel_usba_udc: Driver improvements
<1421945805-31129-1-git-send-email-sylvain.rochet@finsecur.com>
We are not going to change any clock for STANDBY target, there is no
clock to check, so we don't need to call at91_pm_verify_clocks() for
this target.
Sylvain
^ permalink raw reply [flat|nested] 56+ messages in thread
* Re: [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
2015-01-20 8:17 ` Wenyou Yang
@ 2015-01-23 19:17 ` Sylvain Rochet
-1 siblings, 0 replies; 56+ messages in thread
From: Sylvain Rochet @ 2015-01-23 19:17 UTC (permalink / raw)
To: Wenyou Yang
Cc: nicolas.ferre, linux, linux-kernel, alexandre.belloni, peda,
linux-arm-kernel
Hello Wenyou,
On Tue, Jan 20, 2015 at 04:17:01PM +0800, Wenyou Yang wrote:
>
> diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
> new file mode 100644
> index 0000000..420e730
> --- /dev/null
> + /* Turn off the main oscillator */
> + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> + bic tmp1, tmp1, #AT91_PMC_MOSCEN
at91sam9x5 and probably others need a key here:
orr tmp1, tmp1, #AT91_PMC_KEY
> + str tmp1, [pmc, #AT91_CKGR_MOR]
> + /* Wait for interrupt */
> + mcr p15, 0, tmp1, c7, c0, 4
The linux-3.10-at91 branch uses a different approach which seem
necessary for newer board, you probably forget to merge the following:
/*
* Put the processor to enter into Standby mode, wait for interrupt to wakeup
*/
.macro _do_wfi
#if defined(CONFIG_CPU_V7)
dsb
/* Disable the processor clock */
mov tmp1, #AT91_PMC_PCK
str tmp1, [pmc, #AT91_PMC_SCDR]
wfi @ Wait For Interrupt
#else
mcr p15, 0, tmp1, c7, c0, 4
#endif
.endm
.text
ENTRY(at91_slow_clock)
(...)
/* Wait for interrupt */
_do_wfi
(...)
> + /* Turn on the main oscillator */
> + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> + orr tmp1, tmp1, #AT91_PMC_MOSCEN
at91sam9x5 and probably others need a key here:
orr tmp1, tmp1, #AT91_PMC_KEY
> + str tmp1, [pmc, #AT91_CKGR_MOR]
What about the following parts which are also in linux-3.10-at91 branch
but not in this rework, are they necessary ?
sdr_sr_done:
/* Disable MPDDRC Clock*/
cmp ddrcid, #0
beq 2f
bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
mov tmp1, #0x01
mov tmp1, tmp1, lsl tmp2
tst ddrcid, #0x20 /* > 32 ? */
beq 1f
str tmp1, [pmc, #AT91_PMC_PCDR1]
b 2f
1:
str tmp1, [pmc, #AT91_PMC_PCDR]
2:
/* Disable DDR Clock */
mov tmp1, #AT91_PMC_SYS_DDR
str tmp1, [pmc, #AT91_PMC_SCDR]
/* Enable MPDDRC Clock*/
cmp ddrcid, #0
beq 4f
bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
mov tmp1, #0x01
mov tmp1, tmp1, lsl tmp2
tst ddrcid, #0x20 /* > 32 ? */
beq 3f
str tmp1, [pmc, #AT91_PMC_PCER1]
b 4f
3:
str tmp1, [pmc, #AT91_PMC_PCER]
4:
/* Enable DDR clock */
mov tmp1, #AT91_PMC_SYS_DDR
str tmp1, [pmc, #AT91_PMC_SCER]
Sylvain
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
@ 2015-01-23 19:17 ` Sylvain Rochet
0 siblings, 0 replies; 56+ messages in thread
From: Sylvain Rochet @ 2015-01-23 19:17 UTC (permalink / raw)
To: linux-arm-kernel
Hello Wenyou,
On Tue, Jan 20, 2015 at 04:17:01PM +0800, Wenyou Yang wrote:
>
> diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
> new file mode 100644
> index 0000000..420e730
> --- /dev/null
> + /* Turn off the main oscillator */
> + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> + bic tmp1, tmp1, #AT91_PMC_MOSCEN
at91sam9x5 and probably others need a key here:
orr tmp1, tmp1, #AT91_PMC_KEY
> + str tmp1, [pmc, #AT91_CKGR_MOR]
> + /* Wait for interrupt */
> + mcr p15, 0, tmp1, c7, c0, 4
The linux-3.10-at91 branch uses a different approach which seem
necessary for newer board, you probably forget to merge the following:
/*
* Put the processor to enter into Standby mode, wait for interrupt to wakeup
*/
.macro _do_wfi
#if defined(CONFIG_CPU_V7)
dsb
/* Disable the processor clock */
mov tmp1, #AT91_PMC_PCK
str tmp1, [pmc, #AT91_PMC_SCDR]
wfi @ Wait For Interrupt
#else
mcr p15, 0, tmp1, c7, c0, 4
#endif
.endm
.text
ENTRY(at91_slow_clock)
(...)
/* Wait for interrupt */
_do_wfi
(...)
> + /* Turn on the main oscillator */
> + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> + orr tmp1, tmp1, #AT91_PMC_MOSCEN
at91sam9x5 and probably others need a key here:
orr tmp1, tmp1, #AT91_PMC_KEY
> + str tmp1, [pmc, #AT91_CKGR_MOR]
What about the following parts which are also in linux-3.10-at91 branch
but not in this rework, are they necessary ?
sdr_sr_done:
/* Disable MPDDRC Clock*/
cmp ddrcid, #0
beq 2f
bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
mov tmp1, #0x01
mov tmp1, tmp1, lsl tmp2
tst ddrcid, #0x20 /* > 32 ? */
beq 1f
str tmp1, [pmc, #AT91_PMC_PCDR1]
b 2f
1:
str tmp1, [pmc, #AT91_PMC_PCDR]
2:
/* Disable DDR Clock */
mov tmp1, #AT91_PMC_SYS_DDR
str tmp1, [pmc, #AT91_PMC_SCDR]
/* Enable MPDDRC Clock*/
cmp ddrcid, #0
beq 4f
bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
mov tmp1, #0x01
mov tmp1, tmp1, lsl tmp2
tst ddrcid, #0x20 /* > 32 ? */
beq 3f
str tmp1, [pmc, #AT91_PMC_PCER1]
b 4f
3:
str tmp1, [pmc, #AT91_PMC_PCER]
4:
/* Enable DDR clock */
mov tmp1, #AT91_PMC_SYS_DDR
str tmp1, [pmc, #AT91_PMC_SCER]
Sylvain
^ permalink raw reply [flat|nested] 56+ messages in thread
* Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-23 17:32 ` Sylvain Rochet
@ 2015-01-23 23:02 ` Alexandre Belloni
-1 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 23:02 UTC (permalink / raw)
To: Sylvain Rochet
Cc: Wenyou Yang, nicolas.ferre, linux, linux-kernel, peda, linux-arm-kernel
On 23/01/2015 at 18:32:34 +0100, Sylvain Rochet wrote :
> Hello Wenyou,
>
> On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> > index 691e6db..a1010f0 100644
> > --- a/arch/arm/mach-at91/pm.c
> > +++ b/arch/arm/mach-at91/pm.c
> (...)
> > static int at91_pm_enter(suspend_state_t state)
> > {
> > at91_pinctrl_gpio_suspend();
> >
> > switch (state) {
> > + /*
> > + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> > + * drivers must suspend more deeply, the master clock switches
> > + * to the clk32k and turns off the main oscillator
> > + *
> > + * STANDBY mode has *all* drivers suspended; ignores irqs not
> > + * marked as 'wakeup' event sources; and reduces DRAM power.
> > + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> > + * nothing fancy done with main or cpu clocks.
> > + */
> > + case PM_SUSPEND_MEM:
> > + case PM_SUSPEND_STANDBY:
> (...)
> > - case PM_SUSPEND_MEM:
> > - /*
> > - * Ensure that clocks are in a valid state.
> > - */
> > - if (!at91_pm_verify_clocks())
> > - goto error;
> (...)
> > + if (!at91_pm_verify_clocks())
> > + goto error;
> >
> (...)
> > - case PM_SUSPEND_STANDBY:
> > - /*
> > - * NOTE: the Wait-for-Interrupt instruction needs to be
>
> By doing that at91_pm_verify_clocks() is now called for both MEM and
> STANDBY targets.
>
> In my opinion this function is misnamed and should be called
> at91_pm_verify_clocks_for_slow_clock_mode(). This function actually
> checks if we can safely switch to slow clock mode, if some peripherals
> are still using the master clock, we abort the suspend because we can't
> suspend in good condition. Hard unclocking peripherals which ask for a
> soft stop, like USB controllers, is something we should avoid doing.
>
> This function checks if USB PLL and PLL B are stopped, if PCK0..PCK3 are
> stopped too (or just using the 32k clock). If all drivers suspended
> correctly this is the state we expect and we can suspend in a deep
> state.
>
> Not this is currently not the case in linux-next, suspend/resume support
> to all Atmel USB drivers (ehci-atmel,ohci-at91,atmel_usba,at91_udc) are
> in my series:
> [PATCHv7 0/6] USB: host: Atmel OHCI and EHCI drivers improvements
> <1421761144-11767-1-git-send-email-sylvain.rochet@finsecur.com>
> [PATCHv6 0/5] USB: gadget: atmel_usba_udc: Driver improvements
> <1421945805-31129-1-git-send-email-sylvain.rochet@finsecur.com>
>
> We are not going to change any clock for STANDBY target, there is no
> clock to check, so we don't need to call at91_pm_verify_clocks() for
> this target.
>
I think we should actually stop checking those clocks. In the meantime,
you are right and at91_pm_verify_clocks must not be called
unconditionally.
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-23 23:02 ` Alexandre Belloni
0 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 23:02 UTC (permalink / raw)
To: linux-arm-kernel
On 23/01/2015 at 18:32:34 +0100, Sylvain Rochet wrote :
> Hello Wenyou,
>
> On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> > index 691e6db..a1010f0 100644
> > --- a/arch/arm/mach-at91/pm.c
> > +++ b/arch/arm/mach-at91/pm.c
> (...)
> > static int at91_pm_enter(suspend_state_t state)
> > {
> > at91_pinctrl_gpio_suspend();
> >
> > switch (state) {
> > + /*
> > + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> > + * drivers must suspend more deeply, the master clock switches
> > + * to the clk32k and turns off the main oscillator
> > + *
> > + * STANDBY mode has *all* drivers suspended; ignores irqs not
> > + * marked as 'wakeup' event sources; and reduces DRAM power.
> > + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> > + * nothing fancy done with main or cpu clocks.
> > + */
> > + case PM_SUSPEND_MEM:
> > + case PM_SUSPEND_STANDBY:
> (...)
> > - case PM_SUSPEND_MEM:
> > - /*
> > - * Ensure that clocks are in a valid state.
> > - */
> > - if (!at91_pm_verify_clocks())
> > - goto error;
> (...)
> > + if (!at91_pm_verify_clocks())
> > + goto error;
> >
> (...)
> > - case PM_SUSPEND_STANDBY:
> > - /*
> > - * NOTE: the Wait-for-Interrupt instruction needs to be
>
> By doing that at91_pm_verify_clocks() is now called for both MEM and
> STANDBY targets.
>
> In my opinion this function is misnamed and should be called
> at91_pm_verify_clocks_for_slow_clock_mode(). This function actually
> checks if we can safely switch to slow clock mode, if some peripherals
> are still using the master clock, we abort the suspend because we can't
> suspend in good condition. Hard unclocking peripherals which ask for a
> soft stop, like USB controllers, is something we should avoid doing.
>
> This function checks if USB PLL and PLL B are stopped, if PCK0..PCK3 are
> stopped too (or just using the 32k clock). If all drivers suspended
> correctly this is the state we expect and we can suspend in a deep
> state.
>
> Not this is currently not the case in linux-next, suspend/resume support
> to all Atmel USB drivers (ehci-atmel,ohci-at91,atmel_usba,at91_udc) are
> in my series:
> [PATCHv7 0/6] USB: host: Atmel OHCI and EHCI drivers improvements
> <1421761144-11767-1-git-send-email-sylvain.rochet@finsecur.com>
> [PATCHv6 0/5] USB: gadget: atmel_usba_udc: Driver improvements
> <1421945805-31129-1-git-send-email-sylvain.rochet@finsecur.com>
>
> We are not going to change any clock for STANDBY target, there is no
> clock to check, so we don't need to call at91_pm_verify_clocks() for
> this target.
>
I think we should actually stop checking those clocks. In the meantime,
you are right and at91_pm_verify_clocks must not be called
unconditionally.
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-23 16:50 ` Sylvain Rochet
@ 2015-01-23 23:13 ` Alexandre Belloni
-1 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 23:13 UTC (permalink / raw)
To: Sylvain Rochet
Cc: Wenyou Yang, nicolas.ferre, linux, linux-kernel, peda, linux-arm-kernel
On 23/01/2015 at 17:50:20 +0100, Sylvain Rochet wrote :
> Hello Wenyou,
>
>
> On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> > index 691e6db..a1010f0 100644
> > --- a/arch/arm/mach-at91/pm.c
> > +++ b/arch/arm/mach-at91/pm.c
>
>
> > @@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
> > void __iomem *ramc1, int memctrl);
> > extern u32 at91_slow_clock_sz;
> >
> > +static void at91_pm_suspend(suspend_state_t state)
> > +{
> (...)
> > + slow_clock(at91_pmc_base, at91_ramc_base[0],
> > + at91_ramc_base[1], pm_data);
> > +}
>
>
> > - if (slow_clock) {
> > - slow_clock(at91_pmc_base, at91_ramc_base[0],
> > - at91_ramc_base[1],
> > - at91_pm_data.memctrl);
> (...)
> > + at91_pm_suspend(state);
>
>
> By doing that you removed the condition "if (slow_clock)".
>
> But slow_clock can still be NULL, see commit d2e4679, there are multiple
> reasons which ends up with a NULL slow_clock.
>
I would fix that by not calling suspend_set_ops(&at91_pm_ops) when
slow_clock is NULL in patch 6 (quick and easy) or copying the whole
at91_pm_sram_init() in at91_pm_init() and handle failures from there.
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-23 23:13 ` Alexandre Belloni
0 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 23:13 UTC (permalink / raw)
To: linux-arm-kernel
On 23/01/2015 at 17:50:20 +0100, Sylvain Rochet wrote :
> Hello Wenyou,
>
>
> On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c
> > index 691e6db..a1010f0 100644
> > --- a/arch/arm/mach-at91/pm.c
> > +++ b/arch/arm/mach-at91/pm.c
>
>
> > @@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc, void __iomem *ramc0,
> > void __iomem *ramc1, int memctrl);
> > extern u32 at91_slow_clock_sz;
> >
> > +static void at91_pm_suspend(suspend_state_t state)
> > +{
> (...)
> > + slow_clock(at91_pmc_base, at91_ramc_base[0],
> > + at91_ramc_base[1], pm_data);
> > +}
>
>
> > - if (slow_clock) {
> > - slow_clock(at91_pmc_base, at91_ramc_base[0],
> > - at91_ramc_base[1],
> > - at91_pm_data.memctrl);
> (...)
> > + at91_pm_suspend(state);
>
>
> By doing that you removed the condition "if (slow_clock)".
>
> But slow_clock can still be NULL, see commit d2e4679, there are multiple
> reasons which ends up with a NULL slow_clock.
>
I would fix that by not calling suspend_set_ops(&at91_pm_ops) when
slow_clock is NULL in patch 6 (quick and easy) or copying the whole
at91_pm_sram_init() in at91_pm_init() and handle failures from there.
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* Re: [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
2015-01-23 19:17 ` Sylvain Rochet
@ 2015-01-23 23:17 ` Alexandre Belloni
-1 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 23:17 UTC (permalink / raw)
To: Sylvain Rochet
Cc: Wenyou Yang, nicolas.ferre, linux, linux-kernel, peda, linux-arm-kernel
Hi,
On 23/01/2015 at 20:17:19 +0100, Sylvain Rochet wrote :
> Hello Wenyou,
>
> On Tue, Jan 20, 2015 at 04:17:01PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
> > new file mode 100644
> > index 0000000..420e730
> > --- /dev/null
>
>
> > + /* Turn off the main oscillator */
> > + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> > + bic tmp1, tmp1, #AT91_PMC_MOSCEN
>
> at91sam9x5 and probably others need a key here:
> orr tmp1, tmp1, #AT91_PMC_KEY
>
> > + str tmp1, [pmc, #AT91_CKGR_MOR]
>
>
>
> > + /* Wait for interrupt */
> > + mcr p15, 0, tmp1, c7, c0, 4
>
> The linux-3.10-at91 branch uses a different approach which seem
> necessary for newer board, you probably forget to merge the following:
>
> /*
> * Put the processor to enter into Standby mode, wait for interrupt to wakeup
> */
> .macro _do_wfi
>
> #if defined(CONFIG_CPU_V7)
> dsb
>
> /* Disable the processor clock */
> mov tmp1, #AT91_PMC_PCK
> str tmp1, [pmc, #AT91_PMC_SCDR]
>
> wfi @ Wait For Interrupt
> #else
> mcr p15, 0, tmp1, c7, c0, 4
> #endif
>
> .endm
>
> .text
>
> ENTRY(at91_slow_clock)
> (...)
> /* Wait for interrupt */
> _do_wfi
> (...)
>
>
>
>
> > + /* Turn on the main oscillator */
> > + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> > + orr tmp1, tmp1, #AT91_PMC_MOSCEN
>
> at91sam9x5 and probably others need a key here:
> orr tmp1, tmp1, #AT91_PMC_KEY
>
> > + str tmp1, [pmc, #AT91_CKGR_MOR]
>
>
>
> What about the following parts which are also in linux-3.10-at91 branch
> but not in this rework, are they necessary ?
>
> sdr_sr_done:
> /* Disable MPDDRC Clock*/
> cmp ddrcid, #0
> beq 2f
> bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
> mov tmp1, #0x01
> mov tmp1, tmp1, lsl tmp2
>
> tst ddrcid, #0x20 /* > 32 ? */
> beq 1f
> str tmp1, [pmc, #AT91_PMC_PCDR1]
> b 2f
> 1:
> str tmp1, [pmc, #AT91_PMC_PCDR]
> 2:
>
> /* Disable DDR Clock */
> mov tmp1, #AT91_PMC_SYS_DDR
> str tmp1, [pmc, #AT91_PMC_SCDR]
>
>
>
>
> /* Enable MPDDRC Clock*/
> cmp ddrcid, #0
> beq 4f
> bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
> mov tmp1, #0x01
> mov tmp1, tmp1, lsl tmp2
>
> tst ddrcid, #0x20 /* > 32 ? */
> beq 3f
> str tmp1, [pmc, #AT91_PMC_PCER1]
> b 4f
> 3:
> str tmp1, [pmc, #AT91_PMC_PCER]
> 4:
>
> /* Enable DDR clock */
> mov tmp1, #AT91_PMC_SYS_DDR
> str tmp1, [pmc, #AT91_PMC_SCER]
>
>
This is a rework, what is part of linux-3.10-at91 and not yet present in
mainline should be part of a following series. I would prefer not mixing
reworks and "new" functionalities (they have been present in the atmel
tree for a while but never mainlined). I would say that PM on 9x5, n12
and sama5 in mainline is clearly not well tested and is lagging behind
the atmel tree.
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
@ 2015-01-23 23:17 ` Alexandre Belloni
0 siblings, 0 replies; 56+ messages in thread
From: Alexandre Belloni @ 2015-01-23 23:17 UTC (permalink / raw)
To: linux-arm-kernel
Hi,
On 23/01/2015 at 20:17:19 +0100, Sylvain Rochet wrote :
> Hello Wenyou,
>
> On Tue, Jan 20, 2015 at 04:17:01PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm_suspend.S b/arch/arm/mach-at91/pm_suspend.S
> > new file mode 100644
> > index 0000000..420e730
> > --- /dev/null
>
>
> > + /* Turn off the main oscillator */
> > + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> > + bic tmp1, tmp1, #AT91_PMC_MOSCEN
>
> at91sam9x5 and probably others need a key here:
> orr tmp1, tmp1, #AT91_PMC_KEY
>
> > + str tmp1, [pmc, #AT91_CKGR_MOR]
>
>
>
> > + /* Wait for interrupt */
> > + mcr p15, 0, tmp1, c7, c0, 4
>
> The linux-3.10-at91 branch uses a different approach which seem
> necessary for newer board, you probably forget to merge the following:
>
> /*
> * Put the processor to enter into Standby mode, wait for interrupt to wakeup
> */
> .macro _do_wfi
>
> #if defined(CONFIG_CPU_V7)
> dsb
>
> /* Disable the processor clock */
> mov tmp1, #AT91_PMC_PCK
> str tmp1, [pmc, #AT91_PMC_SCDR]
>
> wfi @ Wait For Interrupt
> #else
> mcr p15, 0, tmp1, c7, c0, 4
> #endif
>
> .endm
>
> .text
>
> ENTRY(at91_slow_clock)
> (...)
> /* Wait for interrupt */
> _do_wfi
> (...)
>
>
>
>
> > + /* Turn on the main oscillator */
> > + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> > + orr tmp1, tmp1, #AT91_PMC_MOSCEN
>
> at91sam9x5 and probably others need a key here:
> orr tmp1, tmp1, #AT91_PMC_KEY
>
> > + str tmp1, [pmc, #AT91_CKGR_MOR]
>
>
>
> What about the following parts which are also in linux-3.10-at91 branch
> but not in this rework, are they necessary ?
>
> sdr_sr_done:
> /* Disable MPDDRC Clock*/
> cmp ddrcid, #0
> beq 2f
> bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
> mov tmp1, #0x01
> mov tmp1, tmp1, lsl tmp2
>
> tst ddrcid, #0x20 /* > 32 ? */
> beq 1f
> str tmp1, [pmc, #AT91_PMC_PCDR1]
> b 2f
> 1:
> str tmp1, [pmc, #AT91_PMC_PCDR]
> 2:
>
> /* Disable DDR Clock */
> mov tmp1, #AT91_PMC_SYS_DDR
> str tmp1, [pmc, #AT91_PMC_SCDR]
>
>
>
>
> /* Enable MPDDRC Clock*/
> cmp ddrcid, #0
> beq 4f
> bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
> mov tmp1, #0x01
> mov tmp1, tmp1, lsl tmp2
>
> tst ddrcid, #0x20 /* > 32 ? */
> beq 3f
> str tmp1, [pmc, #AT91_PMC_PCER1]
> b 4f
> 3:
> str tmp1, [pmc, #AT91_PMC_PCER]
> 4:
>
> /* Enable DDR clock */
> mov tmp1, #AT91_PMC_SYS_DDR
> str tmp1, [pmc, #AT91_PMC_SCER]
>
>
This is a rework, what is part of linux-3.10-at91 and not yet present in
mainline should be part of a following series. I would prefer not mixing
reworks and "new" functionalities (they have been present in the atmel
tree for a while but never mainlined). I would say that PM on 9x5, n12
and sama5 in mainline is clearly not well tested and is lagging behind
the atmel tree.
--
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
^ permalink raw reply [flat|nested] 56+ messages in thread
* Re: [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
2015-01-23 23:17 ` Alexandre Belloni
@ 2015-01-25 13:30 ` Sylvain Rochet
-1 siblings, 0 replies; 56+ messages in thread
From: Sylvain Rochet @ 2015-01-25 13:30 UTC (permalink / raw)
To: Alexandre Belloni
Cc: Wenyou Yang, nicolas.ferre, linux, linux-kernel, peda, linux-arm-kernel
Hello Alexandre,
On Sat, Jan 24, 2015 at 12:17:38AM +0100, Alexandre Belloni wrote:
>
> This is a rework, what is part of linux-3.10-at91 and not yet present in
> mainline should be part of a following series. I would prefer not mixing
> reworks and "new" functionalities (they have been present in the atmel
> tree for a while but never mainlined).
I agree, I just didn't know a new series will follow, maybe I missed
this point.
Maybe I am a bit too picky (or boring: if I am, please told me), but
this series by itself adds regression to all users of >= 9x5 boards
(sama5, …) because it merges MEM target and MEM+SLOW_CLOCK target, which
used to be too different target states, not selectable at runtime indeed
but this is still in practice two different target states. Note that I
am not saying that MEM target and MEM+SLOW_CLOCK target should not be
merged, they should, absolutely ;-). For >= 9x5 boards (sama5, …), MEM
target works and MEM+SLOW_CLOCK target does not work, MEM and
MEM+SLOW_CLOCK merge breaks MEM target for those boards.
There is however a good news !, at91_pm_verify_clocks() used to be
called for MEM target without considering if it was MEM (~STANDBY) or
MEM+SLOW_CLOCK. It means that all MEM target users can with very good
chance go to a deeper sleep state without issue because
at91_pm_verify_clocks() successfully checked on those boards and is why
we can merge MEM and MEM+SLOW_CLOCK without adding a regression.
Care should be taken to pull-request at the same time both the rework
and the above cited following series about slow clock support for all
known boards so we don't break MEM target for a release cycle.
> I would say that PM on 9x5, n12 and sama5 in mainline is clearly not
> well tested and is lagging behind the atmel tree.
Well, at the current mainline state, everything works fine for me except
slow clock mode, I thoroughly checked everything else.
Sylvain
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
@ 2015-01-25 13:30 ` Sylvain Rochet
0 siblings, 0 replies; 56+ messages in thread
From: Sylvain Rochet @ 2015-01-25 13:30 UTC (permalink / raw)
To: linux-arm-kernel
Hello Alexandre,
On Sat, Jan 24, 2015 at 12:17:38AM +0100, Alexandre Belloni wrote:
>
> This is a rework, what is part of linux-3.10-at91 and not yet present in
> mainline should be part of a following series. I would prefer not mixing
> reworks and "new" functionalities (they have been present in the atmel
> tree for a while but never mainlined).
I agree, I just didn't know a new series will follow, maybe I missed
this point.
Maybe I am a bit too picky (or boring: if I am, please told me), but
this series by itself adds regression to all users of >= 9x5 boards
(sama5, ?) because it merges MEM target and MEM+SLOW_CLOCK target, which
used to be too different target states, not selectable at runtime indeed
but this is still in practice two different target states. Note that I
am not saying that MEM target and MEM+SLOW_CLOCK target should not be
merged, they should, absolutely ;-). For >= 9x5 boards (sama5, ?), MEM
target works and MEM+SLOW_CLOCK target does not work, MEM and
MEM+SLOW_CLOCK merge breaks MEM target for those boards.
There is however a good news !, at91_pm_verify_clocks() used to be
called for MEM target without considering if it was MEM (~STANDBY) or
MEM+SLOW_CLOCK. It means that all MEM target users can with very good
chance go to a deeper sleep state without issue because
at91_pm_verify_clocks() successfully checked on those boards and is why
we can merge MEM and MEM+SLOW_CLOCK without adding a regression.
Care should be taken to pull-request at the same time both the rework
and the above cited following series about slow clock support for all
known boards so we don't break MEM target for a release cycle.
> I would say that PM on 9x5, n12 and sama5 in mainline is clearly not
> well tested and is lagging behind the atmel tree.
Well, at the current mainline state, everything works fine for me except
slow clock mode, I thoroughly checked everything else.
Sylvain
^ permalink raw reply [flat|nested] 56+ messages in thread
* RE: [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91
2015-01-23 10:24 ` Alexandre Belloni
@ 2015-01-26 1:10 ` Yang, Wenyou
-1 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-26 1:10 UTC (permalink / raw)
To: Alexandre Belloni
Cc: Ferre, Nicolas, linux, linux-arm-kernel, linux-kernel,
sylvain.rochet, peda
Hi Alexandre,
Thank you for your review.
> -----Original Message-----
> From: Alexandre Belloni [mailto:alexandre.belloni@free-electrons.com]
> Sent: Friday, January 23, 2015 6:24 PM
> To: Yang, Wenyou
> Cc: Ferre, Nicolas; linux@arm.linux.org.uk; linux-arm-kernel@lists.infradead.org;
> linux-kernel@vger.kernel.org; sylvain.rochet@finsecur.com; peda@axentia.se
> Subject: Re: [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91
>
> On 20/01/2015 at 16:16:58 +0800, Wenyou Yang wrote :
> > Because the CONFIG_AT91_SLOW_CLOCK will be removed to simply the PM
> > config, so move select SRAM.
> >
> > Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
> Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>
> > ---
> > arch/arm/mach-at91/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
> > index 56dcede..7d5e201 100644
> > --- a/arch/arm/mach-at91/Kconfig
> > +++ b/arch/arm/mach-at91/Kconfig
> > @@ -6,6 +6,7 @@ config ARCH_AT91
> > select PINCTRL
> > select PINCTRL_AT91
> > select USE_OF
> > + select SRAM if PM
> > bool
> >
> > menuconfig SOC_SAM_V7
> > @@ -128,7 +129,6 @@ comment "AT91 Feature Selections"
> >
> > config AT91_SLOW_CLOCK
> > bool "Suspend-to-RAM disables main oscillator"
> > - select SRAM
> > depends on SUSPEND
> > help
> > Select this if you want Suspend-to-RAM to save the most power
> > --
> > 1.7.9.5
> >
>
> --
> Alexandre Belloni, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
Best Regards,
Wenyou Yang
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91
@ 2015-01-26 1:10 ` Yang, Wenyou
0 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-26 1:10 UTC (permalink / raw)
To: linux-arm-kernel
Hi Alexandre,
Thank you for your review.
> -----Original Message-----
> From: Alexandre Belloni [mailto:alexandre.belloni at free-electrons.com]
> Sent: Friday, January 23, 2015 6:24 PM
> To: Yang, Wenyou
> Cc: Ferre, Nicolas; linux at arm.linux.org.uk; linux-arm-kernel at lists.infradead.org;
> linux-kernel at vger.kernel.org; sylvain.rochet at finsecur.com; peda at axentia.se
> Subject: Re: [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91
>
> On 20/01/2015 at 16:16:58 +0800, Wenyou Yang wrote :
> > Because the CONFIG_AT91_SLOW_CLOCK will be removed to simply the PM
> > config, so move select SRAM.
> >
> > Signed-off-by: Wenyou Yang <wenyou.yang@atmel.com>
> Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>
>
> > ---
> > arch/arm/mach-at91/Kconfig | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/mach-at91/Kconfig b/arch/arm/mach-at91/Kconfig
> > index 56dcede..7d5e201 100644
> > --- a/arch/arm/mach-at91/Kconfig
> > +++ b/arch/arm/mach-at91/Kconfig
> > @@ -6,6 +6,7 @@ config ARCH_AT91
> > select PINCTRL
> > select PINCTRL_AT91
> > select USE_OF
> > + select SRAM if PM
> > bool
> >
> > menuconfig SOC_SAM_V7
> > @@ -128,7 +129,6 @@ comment "AT91 Feature Selections"
> >
> > config AT91_SLOW_CLOCK
> > bool "Suspend-to-RAM disables main oscillator"
> > - select SRAM
> > depends on SUSPEND
> > help
> > Select this if you want Suspend-to-RAM to save the most power
> > --
> > 1.7.9.5
> >
>
> --
> Alexandre Belloni, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com
Best Regards,
Wenyou Yang
^ permalink raw reply [flat|nested] 56+ messages in thread
* RE: [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
2015-01-23 19:17 ` Sylvain Rochet
@ 2015-01-26 1:25 ` Yang, Wenyou
-1 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-26 1:25 UTC (permalink / raw)
To: Sylvain Rochet
Cc: Ferre, Nicolas, linux, linux-kernel, alexandre.belloni, peda,
linux-arm-kernel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 3219 bytes --]
Hi Sylvain,
Thank you for your review.
> -----Original Message-----
> From: Sylvain Rochet [mailto:sylvain.rochet@finsecur.com]
> Sent: Saturday, January 24, 2015 3:17 AM
> To: Yang, Wenyou
> Cc: Ferre, Nicolas; linux@arm.linux.org.uk; linux-kernel@vger.kernel.org;
> alexandre.belloni@free-electrons.com; peda@axentia.se; linux-arm-
> kernel@lists.infradead.org
> Subject: Re: [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S --
> >pm_suspend.S
>
> Hello Wenyou,
>
> On Tue, Jan 20, 2015 at 04:17:01PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm_suspend.S
> > b/arch/arm/mach-at91/pm_suspend.S new file mode 100644 index
> > 0000000..420e730
> > --- /dev/null
>
>
> > + /* Turn off the main oscillator */
> > + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> > + bic tmp1, tmp1, #AT91_PMC_MOSCEN
>
> at91sam9x5 and probably others need a key here:
> orr tmp1, tmp1, #AT91_PMC_KEY
>
Yes, I am preparing the patches to support other SoCs.
> > + str tmp1, [pmc, #AT91_CKGR_MOR]
>
>
>
> > + /* Wait for interrupt */
> > + mcr p15, 0, tmp1, c7, c0, 4
>
> The linux-3.10-at91 branch uses a different approach which seem necessary for
> newer board, you probably forget to merge the following:
>
> /*
> * Put the processor to enter into Standby mode, wait for interrupt to wakeup */
> .macro _do_wfi
>
> #if defined(CONFIG_CPU_V7)
> dsb
>
> /* Disable the processor clock */
> mov tmp1, #AT91_PMC_PCK
> str tmp1, [pmc, #AT91_PMC_SCDR]
>
> wfi @ Wait For Interrupt
> #else
> mcr p15, 0, tmp1, c7, c0, 4
> #endif
>
> .endm
>
> .text
>
I will send the new patch serials to support it in this week. Thanks.
> ENTRY(at91_slow_clock)
> (...)
> /* Wait for interrupt */
> _do_wfi
> (...)
>
>
>
>
> > + /* Turn on the main oscillator */
> > + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> > + orr tmp1, tmp1, #AT91_PMC_MOSCEN
>
> at91sam9x5 and probably others need a key here:
> orr tmp1, tmp1, #AT91_PMC_KEY
>
> > + str tmp1, [pmc, #AT91_CKGR_MOR]
>
>
>
> What about the following parts which are also in linux-3.10-at91 branch but not in
> this rework, are they necessary ?
Yes, they are necessary, I will send the next patch serial in this week.
>
> sdr_sr_done:
> /* Disable MPDDRC Clock*/
> cmp ddrcid, #0
> beq 2f
> bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
> mov tmp1, #0x01
> mov tmp1, tmp1, lsl tmp2
>
> tst ddrcid, #0x20 /* > 32 ? */
> beq 1f
> str tmp1, [pmc, #AT91_PMC_PCDR1]
> b 2f
> 1:
> str tmp1, [pmc, #AT91_PMC_PCDR]
> 2:
>
> /* Disable DDR Clock */
> mov tmp1, #AT91_PMC_SYS_DDR
> str tmp1, [pmc, #AT91_PMC_SCDR]
>
>
>
>
> /* Enable MPDDRC Clock*/
> cmp ddrcid, #0
> beq 4f
> bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
> mov tmp1, #0x01
> mov tmp1, tmp1, lsl tmp2
>
> tst ddrcid, #0x20 /* > 32 ? */
> beq 3f
> str tmp1, [pmc, #AT91_PMC_PCER1]
> b 4f
> 3:
> str tmp1, [pmc, #AT91_PMC_PCER]
> 4:
>
> /* Enable DDR clock */
> mov tmp1, #AT91_PMC_SYS_DDR
> str tmp1, [pmc, #AT91_PMC_SCER]
>
>
>
> Sylvain
Best Regards,
Wenyou Yang
ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S
@ 2015-01-26 1:25 ` Yang, Wenyou
0 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-26 1:25 UTC (permalink / raw)
To: linux-arm-kernel
Hi Sylvain,
Thank you for your review.
> -----Original Message-----
> From: Sylvain Rochet [mailto:sylvain.rochet at finsecur.com]
> Sent: Saturday, January 24, 2015 3:17 AM
> To: Yang, Wenyou
> Cc: Ferre, Nicolas; linux at arm.linux.org.uk; linux-kernel at vger.kernel.org;
> alexandre.belloni at free-electrons.com; peda at axentia.se; linux-arm-
> kernel at lists.infradead.org
> Subject: Re: [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S --
> >pm_suspend.S
>
> Hello Wenyou,
>
> On Tue, Jan 20, 2015 at 04:17:01PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm_suspend.S
> > b/arch/arm/mach-at91/pm_suspend.S new file mode 100644 index
> > 0000000..420e730
> > --- /dev/null
>
>
> > + /* Turn off the main oscillator */
> > + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> > + bic tmp1, tmp1, #AT91_PMC_MOSCEN
>
> at91sam9x5 and probably others need a key here:
> orr tmp1, tmp1, #AT91_PMC_KEY
>
Yes, I am preparing the patches to support other SoCs.
> > + str tmp1, [pmc, #AT91_CKGR_MOR]
>
>
>
> > + /* Wait for interrupt */
> > + mcr p15, 0, tmp1, c7, c0, 4
>
> The linux-3.10-at91 branch uses a different approach which seem necessary for
> newer board, you probably forget to merge the following:
>
> /*
> * Put the processor to enter into Standby mode, wait for interrupt to wakeup */
> .macro _do_wfi
>
> #if defined(CONFIG_CPU_V7)
> dsb
>
> /* Disable the processor clock */
> mov tmp1, #AT91_PMC_PCK
> str tmp1, [pmc, #AT91_PMC_SCDR]
>
> wfi @ Wait For Interrupt
> #else
> mcr p15, 0, tmp1, c7, c0, 4
> #endif
>
> .endm
>
> .text
>
I will send the new patch serials to support it in this week. Thanks.
> ENTRY(at91_slow_clock)
> (...)
> /* Wait for interrupt */
> _do_wfi
> (...)
>
>
>
>
> > + /* Turn on the main oscillator */
> > + ldr tmp1, [pmc, #AT91_CKGR_MOR]
> > + orr tmp1, tmp1, #AT91_PMC_MOSCEN
>
> at91sam9x5 and probably others need a key here:
> orr tmp1, tmp1, #AT91_PMC_KEY
>
> > + str tmp1, [pmc, #AT91_CKGR_MOR]
>
>
>
> What about the following parts which are also in linux-3.10-at91 branch but not in
> this rework, are they necessary ?
Yes, they are necessary, I will send the next patch serial in this week.
>
> sdr_sr_done:
> /* Disable MPDDRC Clock*/
> cmp ddrcid, #0
> beq 2f
> bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
> mov tmp1, #0x01
> mov tmp1, tmp1, lsl tmp2
>
> tst ddrcid, #0x20 /* > 32 ? */
> beq 1f
> str tmp1, [pmc, #AT91_PMC_PCDR1]
> b 2f
> 1:
> str tmp1, [pmc, #AT91_PMC_PCDR]
> 2:
>
> /* Disable DDR Clock */
> mov tmp1, #AT91_PMC_SYS_DDR
> str tmp1, [pmc, #AT91_PMC_SCDR]
>
>
>
>
> /* Enable MPDDRC Clock*/
> cmp ddrcid, #0
> beq 4f
> bic tmp2, ddrcid, #0xe0 /* fetch lowest 5 bits */
> mov tmp1, #0x01
> mov tmp1, tmp1, lsl tmp2
>
> tst ddrcid, #0x20 /* > 32 ? */
> beq 3f
> str tmp1, [pmc, #AT91_PMC_PCER1]
> b 4f
> 3:
> str tmp1, [pmc, #AT91_PMC_PCER]
> 4:
>
> /* Enable DDR clock */
> mov tmp1, #AT91_PMC_SYS_DDR
> str tmp1, [pmc, #AT91_PMC_SCER]
>
>
>
> Sylvain
Best Regards,
Wenyou Yang
^ permalink raw reply [flat|nested] 56+ messages in thread
* RE: [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-23 17:32 ` Sylvain Rochet
@ 2015-01-26 3:06 ` Yang, Wenyou
-1 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-26 3:06 UTC (permalink / raw)
To: Sylvain Rochet
Cc: Ferre, Nicolas, linux, linux-kernel, alexandre.belloni, peda,
linux-arm-kernel
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 3346 bytes --]
Hi Sylvain,
Thank you for review.
> -----Original Message-----
> From: Sylvain Rochet [mailto:sylvain.rochet@finsecur.com]
> Sent: Saturday, January 24, 2015 1:33 AM
> To: Yang, Wenyou
> Cc: Ferre, Nicolas; linux@arm.linux.org.uk; linux-kernel@vger.kernel.org;
> alexandre.belloni@free-electrons.com; peda@axentia.se; linux-arm-
> kernel@lists.infradead.org
> Subject: Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram
> function as the suspend to memory mode
>
> Hello Wenyou,
>
> On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c index
> > 691e6db..a1010f0 100644
> > --- a/arch/arm/mach-at91/pm.c
> > +++ b/arch/arm/mach-at91/pm.c
> (...)
> > static int at91_pm_enter(suspend_state_t state) {
> > at91_pinctrl_gpio_suspend();
> >
> > switch (state) {
> > + /*
> > + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> > + * drivers must suspend more deeply, the master clock switches
> > + * to the clk32k and turns off the main oscillator
> > + *
> > + * STANDBY mode has *all* drivers suspended; ignores irqs not
> > + * marked as 'wakeup' event sources; and reduces DRAM power.
> > + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> > + * nothing fancy done with main or cpu clocks.
> > + */
> > + case PM_SUSPEND_MEM:
> > + case PM_SUSPEND_STANDBY:
> (...)
> > - case PM_SUSPEND_MEM:
> > - /*
> > - * Ensure that clocks are in a valid state.
> > - */
> > - if (!at91_pm_verify_clocks())
> > - goto error;
> (...)
> > + if (!at91_pm_verify_clocks())
> > + goto error;
> >
> (...)
> > - case PM_SUSPEND_STANDBY:
> > - /*
> > - * NOTE: the Wait-for-Interrupt instruction needs to be
>
> By doing that at91_pm_verify_clocks() is now called for both MEM and STANDBY
> targets.
>
> In my opinion this function is misnamed and should be called
> at91_pm_verify_clocks_for_slow_clock_mode(). This function actually checks if
> we can safely switch to slow clock mode, if some peripherals are still using the
> master clock, we abort the suspend because we can't suspend in good condition.
> Hard unclocking peripherals which ask for a soft stop, like USB controllers, is
> something we should avoid doing.
>
> This function checks if USB PLL and PLL B are stopped, if PCK0..PCK3 are
> stopped too (or just using the 32k clock). If all drivers suspended correctly this is
> the state we expect and we can suspend in a deep state.
>
> Not this is currently not the case in linux-next, suspend/resume support to all Atmel
> USB drivers (ehci-atmel,ohci-at91,atmel_usba,at91_udc) are in my series:
> [PATCHv7 0/6] USB: host: Atmel OHCI and EHCI drivers improvements
> <1421761144-11767-1-git-send-email-sylvain.rochet@finsecur.com>
> [PATCHv6 0/5] USB: gadget: atmel_usba_udc: Driver improvements
> <1421945805-31129-1-git-send-email-sylvain.rochet@finsecur.com>
>
> We are not going to change any clock for STANDBY target, there is no clock to
> check, so we don't need to call at91_pm_verify_clocks() for this target.
I will change in the next version.
Thanks.
>
> Sylvain
Best Regards,
Wenyou Yang
ÿôèº{.nÇ+‰·Ÿ®‰†+%ŠËÿ±éݶ\x17¥Šwÿº{.nÇ+‰·¥Š{±þG«éÿŠ{ayº\x1dʇڙë,j\a¢f£¢·hšïêÿ‘êçz_è®\x03(階ŠÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?™¨èÚ&£ø§~á¶iO•æ¬z·švØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?–I¥
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-26 3:06 ` Yang, Wenyou
0 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-26 3:06 UTC (permalink / raw)
To: linux-arm-kernel
Hi Sylvain,
Thank you for review.
> -----Original Message-----
> From: Sylvain Rochet [mailto:sylvain.rochet at finsecur.com]
> Sent: Saturday, January 24, 2015 1:33 AM
> To: Yang, Wenyou
> Cc: Ferre, Nicolas; linux at arm.linux.org.uk; linux-kernel at vger.kernel.org;
> alexandre.belloni at free-electrons.com; peda at axentia.se; linux-arm-
> kernel at lists.infradead.org
> Subject: Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram
> function as the suspend to memory mode
>
> Hello Wenyou,
>
> On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> >
> > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c index
> > 691e6db..a1010f0 100644
> > --- a/arch/arm/mach-at91/pm.c
> > +++ b/arch/arm/mach-at91/pm.c
> (...)
> > static int at91_pm_enter(suspend_state_t state) {
> > at91_pinctrl_gpio_suspend();
> >
> > switch (state) {
> > + /*
> > + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> > + * drivers must suspend more deeply, the master clock switches
> > + * to the clk32k and turns off the main oscillator
> > + *
> > + * STANDBY mode has *all* drivers suspended; ignores irqs not
> > + * marked as 'wakeup' event sources; and reduces DRAM power.
> > + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> > + * nothing fancy done with main or cpu clocks.
> > + */
> > + case PM_SUSPEND_MEM:
> > + case PM_SUSPEND_STANDBY:
> (...)
> > - case PM_SUSPEND_MEM:
> > - /*
> > - * Ensure that clocks are in a valid state.
> > - */
> > - if (!at91_pm_verify_clocks())
> > - goto error;
> (...)
> > + if (!at91_pm_verify_clocks())
> > + goto error;
> >
> (...)
> > - case PM_SUSPEND_STANDBY:
> > - /*
> > - * NOTE: the Wait-for-Interrupt instruction needs to be
>
> By doing that at91_pm_verify_clocks() is now called for both MEM and STANDBY
> targets.
>
> In my opinion this function is misnamed and should be called
> at91_pm_verify_clocks_for_slow_clock_mode(). This function actually checks if
> we can safely switch to slow clock mode, if some peripherals are still using the
> master clock, we abort the suspend because we can't suspend in good condition.
> Hard unclocking peripherals which ask for a soft stop, like USB controllers, is
> something we should avoid doing.
>
> This function checks if USB PLL and PLL B are stopped, if PCK0..PCK3 are
> stopped too (or just using the 32k clock). If all drivers suspended correctly this is
> the state we expect and we can suspend in a deep state.
>
> Not this is currently not the case in linux-next, suspend/resume support to all Atmel
> USB drivers (ehci-atmel,ohci-at91,atmel_usba,at91_udc) are in my series:
> [PATCHv7 0/6] USB: host: Atmel OHCI and EHCI drivers improvements
> <1421761144-11767-1-git-send-email-sylvain.rochet@finsecur.com>
> [PATCHv6 0/5] USB: gadget: atmel_usba_udc: Driver improvements
> <1421945805-31129-1-git-send-email-sylvain.rochet@finsecur.com>
>
> We are not going to change any clock for STANDBY target, there is no clock to
> check, so we don't need to call at91_pm_verify_clocks() for this target.
I will change in the next version.
Thanks.
>
> Sylvain
Best Regards,
Wenyou Yang
^ permalink raw reply [flat|nested] 56+ messages in thread
* RE: [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-23 23:02 ` Alexandre Belloni
@ 2015-01-26 3:08 ` Yang, Wenyou
-1 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-26 3:08 UTC (permalink / raw)
To: Alexandre Belloni, Sylvain Rochet
Cc: Ferre, Nicolas, linux, linux-kernel, peda, linux-arm-kernel
Hi Alexandre,
Thank you for review.
> -----Original Message-----
> From: Alexandre Belloni [mailto:alexandre.belloni@free-electrons.com]
> Sent: Saturday, January 24, 2015 7:02 AM
> To: Sylvain Rochet
> Cc: Yang, Wenyou; Ferre, Nicolas; linux@arm.linux.org.uk; linux-
> kernel@vger.kernel.org; peda@axentia.se; linux-arm-kernel@lists.infradead.org
> Subject: Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram
> function as the suspend to memory mode
>
> On 23/01/2015 at 18:32:34 +0100, Sylvain Rochet wrote :
> > Hello Wenyou,
> >
> > On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> > >
> > > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c index
> > > 691e6db..a1010f0 100644
> > > --- a/arch/arm/mach-at91/pm.c
> > > +++ b/arch/arm/mach-at91/pm.c
> > (...)
> > > static int at91_pm_enter(suspend_state_t state) {
> > > at91_pinctrl_gpio_suspend();
> > >
> > > switch (state) {
> > > + /*
> > > + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> > > + * drivers must suspend more deeply, the master clock switches
> > > + * to the clk32k and turns off the main oscillator
> > > + *
> > > + * STANDBY mode has *all* drivers suspended; ignores irqs not
> > > + * marked as 'wakeup' event sources; and reduces DRAM power.
> > > + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> > > + * nothing fancy done with main or cpu clocks.
> > > + */
> > > + case PM_SUSPEND_MEM:
> > > + case PM_SUSPEND_STANDBY:
> > (...)
> > > - case PM_SUSPEND_MEM:
> > > - /*
> > > - * Ensure that clocks are in a valid state.
> > > - */
> > > - if (!at91_pm_verify_clocks())
> > > - goto error;
> > (...)
> > > + if (!at91_pm_verify_clocks())
> > > + goto error;
> > >
> > (...)
> > > - case PM_SUSPEND_STANDBY:
> > > - /*
> > > - * NOTE: the Wait-for-Interrupt instruction needs to be
> >
> > By doing that at91_pm_verify_clocks() is now called for both MEM and
> > STANDBY targets.
> >
> > In my opinion this function is misnamed and should be called
> > at91_pm_verify_clocks_for_slow_clock_mode(). This function actually
> > checks if we can safely switch to slow clock mode, if some peripherals
> > are still using the master clock, we abort the suspend because we
> > can't suspend in good condition. Hard unclocking peripherals which ask
> > for a soft stop, like USB controllers, is something we should avoid doing.
> >
> > This function checks if USB PLL and PLL B are stopped, if PCK0..PCK3
> > are stopped too (or just using the 32k clock). If all drivers
> > suspended correctly this is the state we expect and we can suspend in
> > a deep state.
> >
> > Not this is currently not the case in linux-next, suspend/resume
> > support to all Atmel USB drivers
> > (ehci-atmel,ohci-at91,atmel_usba,at91_udc) are in my series:
> > [PATCHv7 0/6] USB: host: Atmel OHCI and EHCI drivers improvements
> > <1421761144-11767-1-git-send-email-sylvain.rochet@finsecur.com>
> > [PATCHv6 0/5] USB: gadget: atmel_usba_udc: Driver improvements
> > <1421945805-31129-1-git-send-email-sylvain.rochet@finsecur.com>
> >
> > We are not going to change any clock for STANDBY target, there is no
> > clock to check, so we don't need to call at91_pm_verify_clocks() for
> > this target.
> >
>
> I think we should actually stop checking those clocks. In the meantime, you are
> right and at91_pm_verify_clocks must not be called unconditionally.
Thanks.
I will change, the at91_pm_verify_clocks is only called for suspend to memory mode, not for the standby.
>
> --
> Alexandre Belloni, Free Electrons
> Embedded Linux, Kernel and Android engineering http://free-electrons.com
Best Regards,
Wenyou Yang
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-26 3:08 ` Yang, Wenyou
0 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-26 3:08 UTC (permalink / raw)
To: linux-arm-kernel
Hi Alexandre,
Thank you for review.
> -----Original Message-----
> From: Alexandre Belloni [mailto:alexandre.belloni at free-electrons.com]
> Sent: Saturday, January 24, 2015 7:02 AM
> To: Sylvain Rochet
> Cc: Yang, Wenyou; Ferre, Nicolas; linux at arm.linux.org.uk; linux-
> kernel at vger.kernel.org; peda at axentia.se; linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram
> function as the suspend to memory mode
>
> On 23/01/2015 at 18:32:34 +0100, Sylvain Rochet wrote :
> > Hello Wenyou,
> >
> > On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> > >
> > > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c index
> > > 691e6db..a1010f0 100644
> > > --- a/arch/arm/mach-at91/pm.c
> > > +++ b/arch/arm/mach-at91/pm.c
> > (...)
> > > static int at91_pm_enter(suspend_state_t state) {
> > > at91_pinctrl_gpio_suspend();
> > >
> > > switch (state) {
> > > + /*
> > > + * Suspend-to-RAM is like STANDBY plus slow clock mode, so
> > > + * drivers must suspend more deeply, the master clock switches
> > > + * to the clk32k and turns off the main oscillator
> > > + *
> > > + * STANDBY mode has *all* drivers suspended; ignores irqs not
> > > + * marked as 'wakeup' event sources; and reduces DRAM power.
> > > + * But otherwise it's identical to PM_SUSPEND_ON: cpu idle, and
> > > + * nothing fancy done with main or cpu clocks.
> > > + */
> > > + case PM_SUSPEND_MEM:
> > > + case PM_SUSPEND_STANDBY:
> > (...)
> > > - case PM_SUSPEND_MEM:
> > > - /*
> > > - * Ensure that clocks are in a valid state.
> > > - */
> > > - if (!at91_pm_verify_clocks())
> > > - goto error;
> > (...)
> > > + if (!at91_pm_verify_clocks())
> > > + goto error;
> > >
> > (...)
> > > - case PM_SUSPEND_STANDBY:
> > > - /*
> > > - * NOTE: the Wait-for-Interrupt instruction needs to be
> >
> > By doing that at91_pm_verify_clocks() is now called for both MEM and
> > STANDBY targets.
> >
> > In my opinion this function is misnamed and should be called
> > at91_pm_verify_clocks_for_slow_clock_mode(). This function actually
> > checks if we can safely switch to slow clock mode, if some peripherals
> > are still using the master clock, we abort the suspend because we
> > can't suspend in good condition. Hard unclocking peripherals which ask
> > for a soft stop, like USB controllers, is something we should avoid doing.
> >
> > This function checks if USB PLL and PLL B are stopped, if PCK0..PCK3
> > are stopped too (or just using the 32k clock). If all drivers
> > suspended correctly this is the state we expect and we can suspend in
> > a deep state.
> >
> > Not this is currently not the case in linux-next, suspend/resume
> > support to all Atmel USB drivers
> > (ehci-atmel,ohci-at91,atmel_usba,at91_udc) are in my series:
> > [PATCHv7 0/6] USB: host: Atmel OHCI and EHCI drivers improvements
> > <1421761144-11767-1-git-send-email-sylvain.rochet@finsecur.com>
> > [PATCHv6 0/5] USB: gadget: atmel_usba_udc: Driver improvements
> > <1421945805-31129-1-git-send-email-sylvain.rochet@finsecur.com>
> >
> > We are not going to change any clock for STANDBY target, there is no
> > clock to check, so we don't need to call at91_pm_verify_clocks() for
> > this target.
> >
>
> I think we should actually stop checking those clocks. In the meantime, you are
> right and at91_pm_verify_clocks must not be called unconditionally.
Thanks.
I will change, the at91_pm_verify_clocks is only called for suspend to memory mode, not for the standby.
>
> --
> Alexandre Belloni, Free Electrons
> Embedded Linux, Kernel and Android engineering http://free-electrons.com
Best Regards,
Wenyou Yang
^ permalink raw reply [flat|nested] 56+ messages in thread
* RE: [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
2015-01-23 23:13 ` Alexandre Belloni
@ 2015-01-27 3:08 ` Yang, Wenyou
-1 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-27 3:08 UTC (permalink / raw)
To: Alexandre Belloni, Sylvain Rochet
Cc: Ferre, Nicolas, linux, linux-kernel, peda, linux-arm-kernel
Hi Alexandre,
> -----Original Message-----
> From: Alexandre Belloni [mailto:alexandre.belloni@free-electrons.com]
> Sent: Saturday, January 24, 2015 7:13 AM
> To: Sylvain Rochet
> Cc: Yang, Wenyou; Ferre, Nicolas; linux@arm.linux.org.uk; linux-
> kernel@vger.kernel.org; peda@axentia.se; linux-arm-kernel@lists.infradead.org
> Subject: Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram
> function as the suspend to memory mode
>
> On 23/01/2015 at 17:50:20 +0100, Sylvain Rochet wrote :
> > Hello Wenyou,
> >
> >
> > On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> > >
> > > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c index
> > > 691e6db..a1010f0 100644
> > > --- a/arch/arm/mach-at91/pm.c
> > > +++ b/arch/arm/mach-at91/pm.c
> >
> >
> > > @@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc,
> void __iomem *ramc0,
> > > void __iomem *ramc1, int memctrl); extern u32
> > > at91_slow_clock_sz;
> > >
> > > +static void at91_pm_suspend(suspend_state_t state) {
> > (...)
> > > + slow_clock(at91_pmc_base, at91_ramc_base[0],
> > > + at91_ramc_base[1], pm_data);
> > > +}
> >
> >
> > > - if (slow_clock) {
> > > - slow_clock(at91_pmc_base, at91_ramc_base[0],
> > > - at91_ramc_base[1],
> > > - at91_pm_data.memctrl);
> > (...)
> > > + at91_pm_suspend(state);
> >
> >
> > By doing that you removed the condition "if (slow_clock)".
> >
> > But slow_clock can still be NULL, see commit d2e4679, there are
> > multiple reasons which ends up with a NULL slow_clock.
> >
>
> I would fix that by not calling suspend_set_ops(&at91_pm_ops) when slow_clock
> is NULL in patch 6 (quick and easy) or copying the whole
> at91_pm_sram_init() in at91_pm_init() and handle failures from there.
Thank you for suggestion, I will fix it.
>
>
> --
> Alexandre Belloni, Free Electrons
> Embedded Linux, Kernel and Android engineering http://free-electrons.com
Best Regards,
Wenyou Yang
^ permalink raw reply [flat|nested] 56+ messages in thread
* [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode
@ 2015-01-27 3:08 ` Yang, Wenyou
0 siblings, 0 replies; 56+ messages in thread
From: Yang, Wenyou @ 2015-01-27 3:08 UTC (permalink / raw)
To: linux-arm-kernel
Hi Alexandre,
> -----Original Message-----
> From: Alexandre Belloni [mailto:alexandre.belloni at free-electrons.com]
> Sent: Saturday, January 24, 2015 7:13 AM
> To: Sylvain Rochet
> Cc: Yang, Wenyou; Ferre, Nicolas; linux at arm.linux.org.uk; linux-
> kernel at vger.kernel.org; peda at axentia.se; linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH 07/12] pm: at91: the standby mode uses the same sram
> function as the suspend to memory mode
>
> On 23/01/2015 at 17:50:20 +0100, Sylvain Rochet wrote :
> > Hello Wenyou,
> >
> >
> > On Tue, Jan 20, 2015 at 04:17:00PM +0800, Wenyou Yang wrote:
> > >
> > > diff --git a/arch/arm/mach-at91/pm.c b/arch/arm/mach-at91/pm.c index
> > > 691e6db..a1010f0 100644
> > > --- a/arch/arm/mach-at91/pm.c
> > > +++ b/arch/arm/mach-at91/pm.c
> >
> >
> > > @@ -145,62 +145,51 @@ extern void at91_slow_clock(void __iomem *pmc,
> void __iomem *ramc0,
> > > void __iomem *ramc1, int memctrl); extern u32
> > > at91_slow_clock_sz;
> > >
> > > +static void at91_pm_suspend(suspend_state_t state) {
> > (...)
> > > + slow_clock(at91_pmc_base, at91_ramc_base[0],
> > > + at91_ramc_base[1], pm_data);
> > > +}
> >
> >
> > > - if (slow_clock) {
> > > - slow_clock(at91_pmc_base, at91_ramc_base[0],
> > > - at91_ramc_base[1],
> > > - at91_pm_data.memctrl);
> > (...)
> > > + at91_pm_suspend(state);
> >
> >
> > By doing that you removed the condition "if (slow_clock)".
> >
> > But slow_clock can still be NULL, see commit d2e4679, there are
> > multiple reasons which ends up with a NULL slow_clock.
> >
>
> I would fix that by not calling suspend_set_ops(&at91_pm_ops) when slow_clock
> is NULL in patch 6 (quick and easy) or copying the whole
> at91_pm_sram_init() in at91_pm_init() and handle failures from there.
Thank you for suggestion, I will fix it.
>
>
> --
> Alexandre Belloni, Free Electrons
> Embedded Linux, Kernel and Android engineering http://free-electrons.com
Best Regards,
Wenyou Yang
^ permalink raw reply [flat|nested] 56+ messages in thread
end of thread, other threads:[~2015-01-27 3:10 UTC | newest]
Thread overview: 56+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-20 8:16 [PATCH 00/12] AT91 pm cleanup for 3.20 Wenyou Yang
2015-01-20 8:16 ` Wenyou Yang
2015-01-20 8:16 ` [PATCH 01/12] pm: at91: pm_slowclock: improve reliability of suspend/resume Wenyou Yang
2015-01-20 8:16 ` Wenyou Yang
2015-01-20 8:16 ` [PATCH 02/12] pm: at91: Workaround DDRSDRC self-refresh bug with LPDDR1 memories Wenyou Yang
2015-01-20 8:16 ` Wenyou Yang
2015-01-20 8:16 ` [PATCH 03/12] pm: at91: pm_slowclock: remove the unused code related with SLOWDOWN_MASTER_CLOCK Wenyou Yang
2015-01-20 8:16 ` Wenyou Yang
2015-01-20 8:16 ` [PATCH 04/12] pm: at91: move the copying the sram function to the sram initializationi phase Wenyou Yang
2015-01-20 8:16 ` Wenyou Yang
2015-01-20 8:16 ` [PATCH 05/12] ARM: at91: move select SRAM to ARCH_AT91 Wenyou Yang
2015-01-20 8:16 ` Wenyou Yang
2015-01-23 10:24 ` Alexandre Belloni
2015-01-23 10:24 ` Alexandre Belloni
2015-01-26 1:10 ` Yang, Wenyou
2015-01-26 1:10 ` Yang, Wenyou
2015-01-20 8:16 ` [PATCH 06/12] pm: at91: remove the config item CONFIG_AT91_SLOW_CLOCK Wenyou Yang
2015-01-20 8:16 ` Wenyou Yang
2015-01-20 8:17 ` [PATCH 07/12] pm: at91: the standby mode uses the same sram function as the suspend to memory mode Wenyou Yang
2015-01-20 8:17 ` Wenyou Yang
2015-01-23 10:30 ` Alexandre Belloni
2015-01-23 10:30 ` Alexandre Belloni
2015-01-23 16:50 ` Sylvain Rochet
2015-01-23 16:50 ` Sylvain Rochet
2015-01-23 23:13 ` Alexandre Belloni
2015-01-23 23:13 ` Alexandre Belloni
2015-01-27 3:08 ` Yang, Wenyou
2015-01-27 3:08 ` Yang, Wenyou
2015-01-23 17:32 ` Sylvain Rochet
2015-01-23 17:32 ` Sylvain Rochet
2015-01-23 23:02 ` Alexandre Belloni
2015-01-23 23:02 ` Alexandre Belloni
2015-01-26 3:08 ` Yang, Wenyou
2015-01-26 3:08 ` Yang, Wenyou
2015-01-26 3:06 ` Yang, Wenyou
2015-01-26 3:06 ` Yang, Wenyou
2015-01-20 8:17 ` [PATCH 08/12] pm: at91: rename file name: pm_slowclock.S -->pm_suspend.S Wenyou Yang
2015-01-20 8:17 ` Wenyou Yang
2015-01-23 19:17 ` Sylvain Rochet
2015-01-23 19:17 ` Sylvain Rochet
2015-01-23 23:17 ` Alexandre Belloni
2015-01-23 23:17 ` Alexandre Belloni
2015-01-25 13:30 ` Sylvain Rochet
2015-01-25 13:30 ` Sylvain Rochet
2015-01-26 1:25 ` Yang, Wenyou
2015-01-26 1:25 ` Yang, Wenyou
2015-01-20 8:17 ` [PATCH 09/12] pm: at91: rename function name: at91_slow_clock()-->at91_pm_suspend_sram_fn Wenyou Yang
2015-01-20 8:17 ` Wenyou Yang
2015-01-20 8:24 ` Wenyou Yang
2015-01-20 8:24 ` Wenyou Yang
2015-01-20 8:24 ` [PATCH 10/12] pm: at91: remove the at91_xxx_standby() function definitions in the pm.h Wenyou Yang
2015-01-20 8:24 ` Wenyou Yang
2015-01-20 8:25 ` [PATCH 11/12] pm: at91: remove the struct ramc_ids .data at91_xxx_standby members Wenyou Yang
2015-01-20 8:25 ` Wenyou Yang
2015-01-20 8:26 ` [PATCH 12/12] pm: at91: amend the pm_suspend entry for at91_cpuidle_device Wenyou Yang
2015-01-20 8:26 ` Wenyou Yang
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.