From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Lezcano Subject: Re: [PATCH v2 0/2] Enabling PSCI based idle on ARM 32-bit platforms Date: Tue, 13 Oct 2015 10:32:05 +0200 Message-ID: <561CC185.60001@linaro.org> References: <1444648628-24790-1-git-send-email-lorenzo.pieralisi@arm.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-f173.google.com ([209.85.212.173]:38087 "EHLO mail-wi0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751680AbbJMIcI (ORCPT ); Tue, 13 Oct 2015 04:32:08 -0400 Received: by wieq12 with SMTP id q12so20130539wie.1 for ; Tue, 13 Oct 2015 01:32:07 -0700 (PDT) In-Reply-To: <1444648628-24790-1-git-send-email-lorenzo.pieralisi@arm.com> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Lorenzo Pieralisi , linux-arm-kernel@lists.infradead.org, linux-pm@vger.kernel.org Cc: Will Deacon , Sudeep Holla , Russell King , Catalin Marinas , Mark Rutland , Jisheng Zhang , Lina Iyer Hi Lorenzo, I would like to take this patch through my tree but it does not apply. Mind to refresh it against linux-pm or v4.3-rc5 ? -- Daniel On 10/12/2015 01:17 PM, Lorenzo Pieralisi wrote: > This patch series is v2 of a previous posting: > > http://lists.infradead.org/pipermail/linux-arm-kernel/2015-October/37= 3756.html > > v1->v2: > > - Refactored patch 1 to remove cpu parameter from cpuidle_ops > suspend hook > - Refactored psci_cpu_init_idle to stub out dt parsing function and > make it usable on both ARM/ARM64 with no additional changes > - Updated ARM cpuidle_ops to new interfaces > - Fixed PSCI enable method string in ARM cpuidle_ops struct > > PSCI firmware provides a kernel API that, through a standard interfac= e, > allows to manage power states transitions in a seamless manner for > ARM and ARM64 systems. > > Current PSCI code that initializes CPUidle states on PSCI based > systems lives in arch/arm64 directory but it is not ARM64 specific > and can be shared with ARM 32-bit systems so that the generic > ARM CPUidle driver can leverage the common PSCI interface. > > This patch series moves PSCI CPUidle management code to > drivers/firmware directory so that ARM and ARM64 architecture > can actually share the code. > > It is made up of two patches: > > Patch 1 refactors ARM 32-bit generic idle implementation to remove > the cpu parameter from the cpuidle_ops suspend hook, in preparation > for a common PSCI implementation for ARM/ARM64 PSCI idle. > > Patch 2 moves ARM64 PSCI CPUidle functions implementation to > drivers/firmware so that it can be shared with ARM 32-bit platforms > code. This patch also adds a PSCI entry section on ARM 32-bit systems > so that the PSCI CPUidle back-end can be enabled when the enable-meth= od > corresponds to PSCI. > > Patches apply on top of current patch stack to enable PSCI 1.0: > > git://git.kernel.org/pub/scm/linux/kernel/git/lpieralisi/linux.git ta= gs/firmware/psci-1.0 > > Tested on Juno board (ARM64), compile tested only on ARM 32-bit syste= ms. > > Lorenzo Pieralisi (2): > ARM: cpuidle: remove cpu parameter from the cpuidle_ops suspend ho= ok > ARM64: kernel: PSCI: move PSCI idle management code to > drivers/firmware > > arch/arm/include/asm/cpuidle.h | 2 +- > arch/arm/kernel/cpuidle.c | 2 +- > arch/arm64/kernel/psci.c | 99 +----------------------------- > drivers/firmware/Makefile | 2 +- > drivers/firmware/psci_cpuops.c | 133 ++++++++++++++++++++++++++++++= +++++++++++ > drivers/soc/qcom/spm.c | 10 ++-- > include/linux/psci.h | 3 + > 7 files changed, 146 insertions(+), 105 deletions(-) > create mode 100644 drivers/firmware/psci_cpuops.c > --=20 Linaro.org =E2=94=82 Open source software fo= r ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog