From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lorenzo Pieralisi Subject: Re: [PATCH] drivers: firmware: psci: unify enable-method binding on ARM {64,32}-bit systems Date: Fri, 22 Apr 2016 15:59:21 +0100 Message-ID: <20160422145921.GA14353@red-moon> References: <1461335911-13233-1-git-send-email-sudeep.holla@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1461335911-13233-1-git-send-email-sudeep.holla-5wv7dgnIgG8@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Sudeep Holla Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Maxime Ripard , Soby Mathew , Rob Herring , Mark Rutland List-Id: devicetree@vger.kernel.org On Fri, Apr 22, 2016 at 03:38:31PM +0100, 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". > > 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 > Cc: Mark Rutland > Cc: 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(-) Acked-by: Lorenzo Pieralisi > diff --git a/Documentation/devicetree/bindings/arm/cpus.txt b/Documentation/devicetree/bindings/arm/cpus.txt > index ccc62f145306..3f0cbbb8395f 100644 > --- a/Documentation/devicetree/bindings/arm/cpus.txt > +++ b/Documentation/devicetree/bindings/arm/cpus.txt > @@ -192,7 +192,6 @@ nodes to be present and contain the properties described below. > can be one of: > "allwinner,sun6i-a31" > "allwinner,sun8i-a23" > - "arm,psci" > "arm,realview-smp" > "brcm,bcm-nsp-smp" > "brcm,brahma-b15" > diff --git a/drivers/firmware/psci.c b/drivers/firmware/psci.c > index 11bfee8b79a9..b5d05807e6ec 100644 > --- a/drivers/firmware/psci.c > +++ b/drivers/firmware/psci.c > @@ -360,7 +360,7 @@ static struct cpuidle_ops psci_cpuidle_ops __initdata = { > .init = psci_dt_cpu_init_idle, > }; > > -CPUIDLE_METHOD_OF_DECLARE(psci, "arm,psci", &psci_cpuidle_ops); > +CPUIDLE_METHOD_OF_DECLARE(psci, "psci", &psci_cpuidle_ops); > #endif > #endif > > -- > 1.9.1 > -- 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