Re: [RFC/PATCH 0/6] Add CPUIdle support for Renesas SoCs

[Daniel Lezcano <daniel.lezcano@linaro.org>]

On 03/04/2015 05:55 AM, Keita Kobayashi wrote:
> Hello
>
> These patches add CPUIdle support for Renesas SoCs.
> These patches have been tested on Renesas Gen2 series with
> following cases on renesas-devel-20150301-v4.0-rc1 tag.
>
> [Test case]
>     a. Lager board(CA15 * 4)
>     b. Lager board(CA7 * 4)
>     c. Lager board(CA15 * 4 + CA7 *4)
>     d. Koelsch board(CA15 * 2)
>
> These cases work fine.
>
> Keita Kobayashi (6):
>    cpuidle: renesas: Add CPUIdle Driver for Renesas SoCs
>    ARM: shmobile: Change CPUIdle InterFace
>    ARM: shmobile: APMU: Add Core-Standby function for CPUIdle
>    ARM: shmobile: r8a7791: Add Core-Standby CPUIdle support for r8a7791
>    ARM: shmobile: r8a7790: Add Core-Standby CPUIdle support for r8a7790
>    ARM: shmobile: Enable Renesas CPUIdle for shmobile_defconfig
>
>   arch/arm/configs/shmobile_defconfig           |   2 +
>   arch/arm/mach-shmobile/common.h               |   8 +-
>   arch/arm/mach-shmobile/cpuidle.c              |  31 ++++---
>   arch/arm/mach-shmobile/platsmp-apmu.c         |  20 ++++-
>   arch/arm/mach-shmobile/pm-r8a7790.c           |  55 ++++++++++++
>   arch/arm/mach-shmobile/pm-r8a7791.c           |  41 +++++++++
>   drivers/cpuidle/Kconfig.arm                   |   8 ++
>   drivers/cpuidle/Makefile                      |   1 +
>   drivers/cpuidle/cpuidle-renesas.c             | 118 ++++++++++++++++++++++++++
>   include/linux/platform_data/renesas-cpuidle.h |  24 ++++++
>   10 files changed, 294 insertions(+), 14 deletions(-)
>   create mode 100644 drivers/cpuidle/cpuidle-renesas.c
>   create mode 100644 include/linux/platform_data/renesas-cpuidle.h

Hi Keita,

the platform approach to pass private data from the arch specific to the 
cpuidle driver no longer accepted. Unfortunately, there is no alternate 
solution upstream.

But hopefully, there is a patchset [1] submitted for upstreaming which 
unify arm32/64 into a single generic cpuidle driver and use the DT to 
define the idle functions to be used.

I suggest you have a look at this patchset if it can fit your need.

   -- Daniel

[1] https://lkml.org/lkml/2015/3/3/271


-- 
  <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 |
<http://www.linaro.org/linaro-blog/> Blog