From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Date: Thu, 05 Mar 2015 18:02:11 +0000 Subject: Re: [RFC/PATCH 0/6] Add CPUIdle support for Renesas SoCs Message-Id: <54F89A23.1060008@linaro.org> List-Id: References: <1425444946-3084-1-git-send-email-keita.kobayashi.ym@renesas.com> In-Reply-To: <1425444946-3084-1-git-send-email-keita.kobayashi.ym@renesas.com> MIME-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable To: Keita Kobayashi , horms@verge.net.au, rjw@rjwysocki.net Cc: linux-sh@vger.kernel.org, linux-pm@vger.kernel.org, magnus.damm@gmail.com 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=20 cpuidle driver no longer accepted. Unfortunately, there is no alternate=20 solution upstream. But hopefully, there is a patchset [1] submitted for upstreaming which=20 unify arm32/64 into a single generic cpuidle driver and use the DT to=20 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 --=20 Linaro.org =E2=94=82 Open source software for AR= M SoCs Follow Linaro: Facebook | Twitter | Blog