Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932377AbbDNWdA (ORCPT ); Tue, 14 Apr 2015 18:33:00 -0400 Received: from foss.arm.com ([217.140.101.70]:34909 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754188AbbDNWcv (ORCPT ); Tue, 14 Apr 2015 18:32:51 -0400 Date: Tue, 14 Apr 2015 23:32:43 +0100 From: Lorenzo Pieralisi To: Kumar Gala Cc: 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: <20150414223242.GA18888@e107981-lin.cambridge.arm.com> 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> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <245B9FDD-E1B5-41E4-9F24-4D5BB86C450E@codeaurora.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: 1551 Lines: 37 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). 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-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/