From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [RFC/PATCH 0/6] Add CPUIdle support for Renesas SoCs Date: Thu, 05 Mar 2015 19:02:11 +0100 Message-ID: <54F89A23.1060008@linaro.org> References: <1425444946-3084-1-git-send-email-keita.kobayashi.ym@renesas.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mail-wi0-f172.google.com ([209.85.212.172]:46195 "EHLO mail-wi0-f172.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758562AbbCESCO (ORCPT ); Thu, 5 Mar 2015 13:02:14 -0500 Received: by widem10 with SMTP id em10so40351803wid.5 for ; Thu, 05 Mar 2015 10:02:13 -0800 (PST) In-Reply-To: <1425444946-3084-1-git-send-email-keita.kobayashi.ym@renesas.com> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org 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 r8a77= 91 > ARM: shmobile: r8a7790: Add Core-Standby CPUIdle support for r8a77= 90 > 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 th= e=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 fo= r ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog