[PATCH V5 00/20] ARM: exynos: cpuidle: Move the driver to drivers/cpuidle

Daniel Lezcano
2014-04-14T09:01:03Z

Hi Kukjin,

I believe I addressed all the comments. Is it possible to take this
patchset for next ?

Regards

-- Daniel

On 04/11/2014 12:39 PM, Daniel Lezcano wrote:
> Changelog:
>
> V5:
> * Removed patch : "Move clock setup to pm.c"
> * Added patch : "Move arm core power down clock to exynos5250 common code"
> V4:
> * Took into account Tomasz's comments
> * Fixed missing call in for central suspend
> * Passed parameter to the wakeup mask function
> * Moved wakeup mask, boot vector and aftr state into a single
> function
> * Used this function as callback for platform data
> * Moved S5P_CHECK_AFTR/S5P_CHECK_SLEEP into pm.c
> * Set boot vector only one time
> * Splitted some patches to make them more readable
> V3:
> * Added patch : "ARM: exynos: cpuidle: Disable cpuidle for 5440"
> * Removed patch : "ARM: exynos: config: Enable cpuidle"
> * Removed default ARM_EXYNOS4210_CPUIDLE=y
> * Added comment about bug fix side effect 'for_each_possible_cpu'
> V2:
> * Added comment in changelog for calls order (5/17)
> * Call the powerdown only for cpu0 in the pm notifier
> * Set the pm notifier for all boards
>
> V1: initial post
>
> This patchset relies on the cpm_pm notifier to initiate the powerdown sequence
> operations from pm.c instead cpuidle.c. Thus the cpuidle driver is no longer
> dependent from arch specific code as everything is called from the pm.c file.
>
> The patchset applies on top of linux-samsung/for-next.
>
> Tested on exynos4: 4210
> Tested on exynos5: 5250 (without AFTR)
>
> Amit Daniel Kachhap (1):
> ARM: EXYNOS: Move arm core power down clock to exynos5250 common
> clock
>
> Daniel Lezcano (19):
> ARM: exynos: cpuidle: Prevent forward declaration
> ARM: exynos: cpuidle: Use cpuidle_register
> ARM: exynos: cpuidle: Change function name prefix
> ARM: exynos: cpuidle: Encapsulate register access inside a function
> ARM: exynos: cpuidle: Move some code inside the idle_finisher
> ARM: exynos: cpuidle: Fix S5P_WAKEUP_STAT call
> ARM: exynos: cpuidle: Use the cpu_pm notifier
> ARM: exynos: cpuidle: Move scu_enable in the cpu_pm notifier
> ARM: exynos: cpuidle: Remove ifdef for scu_enable
> ARM: exynos: cpuidle: Pass wakeup mask parameter to function
> ARM: exynos: cpuidle: Encapsulate boot vector code into a function
> ARM: exynos: cpuidle: Disable cpuidle for 5440
> ARM: exynos: cpuidle: Encapsulate the AFTR code into a function
> ARM: exynos: cpuidle: Move the AFTR state function into pm.c
> ARM: exynos: cpuidle: Move the power sequence call in the cpu_pm
> notifier
> ARM: exynos: cpuidle: Move S5P_CHECK_SLEEP into pm.c
> ARM: exynos: cpuidle: Pass the AFTR callback to the platform_data
> ARM: exynos: cpuidle: Cleanup all unneeded headers from cpuidle.c
> ARM: exynos: cpuidle: Move the driver to drivers/cpuidle directory
>
> arch/arm/mach-exynos/Makefile | 1 -
> arch/arm/mach-exynos/common.h | 1 +
> arch/arm/mach-exynos/cpuidle.c | 255 ----------------------------------
> arch/arm/mach-exynos/exynos.c | 8 +-
> arch/arm/mach-exynos/pm.c | 152 ++++++++++++++++----
> arch/arm/mach-exynos/regs-pmu.h | 2 -
> drivers/clk/samsung/clk-exynos5250.c | 42 ++++++
> drivers/cpuidle/Kconfig.arm | 6 +
> drivers/cpuidle/Makefile | 1 +
> drivers/cpuidle/cpuidle-exynos.c | 98 +++++++++++++
> 10 files changed, 277 insertions(+), 289 deletions(-)
> delete mode 100644 arch/arm/mach-exynos/cpuidle.c
> create mode 100644 drivers/cpuidle/cpuidle-exynos.c
>

--
<http://www.linaro.org/> Linaro.org ? Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |