From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sudeep Holla Subject: Re: [PATCH v2] drivers: firmware: psci: unify enable-method binding on ARM {64,32}-bit systems Date: Tue, 26 Apr 2016 11:42:58 +0100 Message-ID: <571F4632.6090003@arm.com> References: <1461338282-19509-1-git-send-email-sudeep.holla@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1461338282-19509-1-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: arm-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, Kevin Hilman , Arnd Bergmann Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Sudeep Holla , Maxime Ripard , Mark Rutland , Lorenzo Pieralisi , Olof Johansson , Rob Herring List-Id: devicetree@vger.kernel.org On 22/04/16 16:18, Sudeep Holla wrote: > Currently ARM CPUs DT bindings allows different enable-method value for > PSCI based systems. On ARM 64-bit this property is required and must be > "psci" while on ARM 32-bit systems this property is optional and must > be "arm,psci" if present. > > However, "arm,psci" has always been the compatible string for the PSCI > node, and was never intended to be the enable-method. So this is a bug > in the binding and not a deliberate attempt at specifying 32-bit > differently. > > This is problematic if 32-bit OS is run on 64-bit system which has > "psci" as enable-method rather than the expected "arm,psci". > > So let's unify the value into "psci" and remove support for "arm,psci" > before it finds any users. > > Reported-by: Soby Mathew > Cc: Rob Herring > Acked-by: Mark Rutland > Acked-by: Lorenzo Pieralisi > Signed-off-by: Sudeep Holla > --- > Documentation/devicetree/bindings/arm/cpus.txt | 1 - > drivers/firmware/psci.c | 2 +- > 2 files changed, 1 insertion(+), 2 deletions(-) > > v2->v1: > - Added Mark and Lorenzo's ACKs > - Updated commit message to specify that it also conflicts with > PSCI compatible(from Mark's reply on the thread) > > Hi ARM-SoC guys, > > Can you please pick up this single fix for PSCI binding/driver for v4.6 ? > Gentle ping to include this in next round of arm-soc fixes. It would be good to get this fixed before v4.6 as we don't want to have user of the binding which we want to remove in the release. -- Regards, Sudeep -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html