Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965002AbbDJQH3 (ORCPT ); Fri, 10 Apr 2015 12:07:29 -0400 Received: from foss.arm.com ([217.140.101.70]:55904 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933577AbbDJQHZ (ORCPT ); Fri, 10 Apr 2015 12:07:25 -0400 Date: Fri, 10 Apr 2015 17:07:20 +0100 From: Lorenzo Pieralisi To: Kumar Gala Cc: "linux-arm-msm@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , "arm@kernel.org" , "devicetree@vger.kernel.org" , Catalin Marinas , Will Deacon , "abhimany@codeaurora.org" Subject: Re: [RFC PATCH 0/5] Add smp booting support for Qualcomm ARMv8 SoCs Message-ID: <20150410160720.GC17828@red-moon> References: <1428601031-5366-1-git-send-email-galak@codeaurora.org> <20150410110328.GC25921@red-moon> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: 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: 2026 Lines: 44 On Fri, Apr 10, 2015 at 04:25:54PM +0100, Kumar Gala wrote: > > On Apr 10, 2015, at 6:03 AM, Lorenzo Pieralisi wrote: > > > On Thu, Apr 09, 2015 at 06:37:06PM +0100, Kumar Gala wrote: > >> This patch set adds support for SMP boot on the MSM8x16 family of Qualcomm SoCs. > >> > >> To support SMP on the MSM8x16 SoCs we need to add ARMv8/64-bit SCM interfaces to > >> setup the boot/release addresses for the secondary CPUs. In addition we need > >> a uniquie set of cpu ops. I'm aware the desired methods for booting secondary > >> CPUs is either via spintable or PSCI. However, these SoCs are shipping with a > >> firmware that does not support those methods. > > > > Why ? Do not tell me you were not aware of those standard methods, > > because I can't believe you. > > > > If there were additional features to add to spin-table and PSCI, > > you were, you are and you will always be welcome to debate them. > > > > There is no justification for this patchset, honestly. > > > > Lorenzo > > The justification for this patchset is support for a hardware platform that exists in the world. The kernel usual is willing to accept such things as long as the code is reasonable. You are telling me you add a linker section for cpu_ops just for "a platform that exists", sorry I do not believe that (and that's *not* reasonable). I do not like this line of reasoning, at all, because you were aware of PSCI and ignored it, deliberately. Start by pushing code for platforms upstream that support PSCI, "they are coming up next" does not cut it. I can see power management mach code coming next, just no way. Implement PSCI and the kernel will be willing to accept it, as it stands as far as I am concerned that's a NAK on the series. 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/