Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932628AbbDORfW (ORCPT ); Wed, 15 Apr 2015 13:35:22 -0400 Received: from foss.arm.com ([217.140.101.70]:36583 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753395AbbDORfS (ORCPT ); Wed, 15 Apr 2015 13:35:18 -0400 Date: Wed, 15 Apr 2015 18:35:13 +0100 From: Lorenzo Pieralisi To: Lina Iyer Cc: Kumar Gala , Catalin Marinas , Device Tree Mailing List , linux-arm-msm , Will Deacon , "linux-kernel@vger.kernel.org" , "arm@kernel.org" , Abhimanyu Kapur , "linux-arm-kernel@lists.infradead.org" Subject: Re: [RFC PATCH 0/5] Add smp booting support for Qualcomm ARMv8 SoCs Message-ID: <20150415173513.GE23758@red-moon> References: <1428601031-5366-1-git-send-email-galak@codeaurora.org> <20150410100529.GA6854@e104818-lin.cambridge.arm.com> <493B15F8-0EBE-4633-9604-671EF403F36E@codeaurora.org> <20150410161052.GF6854@e104818-lin.cambridge.arm.com> <20150413094117.GA2745@e104818-lin.cambridge.arm.com> <245B9FDD-E1B5-41E4-9F24-4D5BB86C450E@codeaurora.org> <20150414223242.GA18888@e107981-lin.cambridge.arm.com> <20150415161759.GB1111@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20150415161759.GB1111@linaro.org> User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3118 Lines: 73 On Wed, Apr 15, 2015 at 05:17:59PM +0100, Lina Iyer wrote: > On Tue, Apr 14 2015 at 16:32 -0600, Lorenzo Pieralisi wrote: > >On Tue, Apr 14, 2015 at 03:21:17PM +0100, Kumar Gala wrote: > > > >[...] > > > >> > Looking beyond this set of patches, I can foresee that you won't care > >> > about the generic arm64 cpuidle driver either, or more precisely the > >> > separation between cpuidle subsystem+driver and the SoC-specific > >> > back-end (cpu_operations). > >> > >> That's probably true for what I guess are a number of reasons. I'm guessing the arm64 cpuidle driver expects PSCI. > > > >Wrap lines sensibly please. > > > >The arm64 cpuidle driver, that is now arm generic cpuidle driver does > >not expect anything apart from an enable-method (and you pulled > >part of its back-end implementation for arm32 Qualcomm platforms, FYI). > > > The backend for this SoC would leverage the same platform code as ARM32. > The cpu_operations callbacks for init and suspend will call into the the > same platform functions used by arm32 QCOM SoCs. It is understood, but this does not mean we should merge this patchset, actually it is the other way around. It was extremely complicated to factor out some common CPUidle bits because of the prolification of power down interfaces in arm/mach code, each with its quirks du jour. If we had a standard interface (that encompasses what all ARM mach code power interfaces do, basically PSCI) when arm32 power management code was being pushed upstream we would not have that power management arm/mach code today. PSCI is there to solve that problem, and it predates v8, there is no reason to merge code that provides no added value (I am obviously talking about the pseudo-boot protocol this patchset is enabling, not the platforms themselves which we definitely want to support upstream, with some preconditions that are equal for everyone) and to leverage a legacy that does not exist for arm64. Thanks, Lorenzo > > Thanks, > Lina > > >It took years to consolidate it and the main reason was the lack of > >standard interfaces for power down/up sequences that this patchset of > >yours wants to promote in arm64 world. > > > >The lack of standard power interfaces may not have been an issue for you, > >who cares about Qualcomm code, it has been a sore issue for people > >trying to generalize things across ARM platforms in the kernel, which is > >the only sensible way forward. > > > >PSCI is a standard interface (and Qualcomm are already contributing to > >it, for the records) that can certainly be extended, and you are welcome > >to contribute to it, but certainly not ignored. > > > >Thanks, > >Lorenzo > >-- > >To unsubscribe from this list: send the line "unsubscribe linux-arm-msm" in > >the body of a message to majordomo@vger.kernel.org > >More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/