From mboxrd@z Thu Jan 1 00:00:00 1970 From: Niklas Cassel Subject: Re: [PATCHv1 1/8] arm64: dts: Fix various entry-method properties to reflect documentation Date: Tue, 14 May 2019 18:11:35 +0200 Message-ID: <20190514161135.GA1824@centauri.ideon.se> References: Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Amit Kucheria Cc: linux-kernel@vger.kernel.org, linux-arm-msm@vger.kernel.org, bjorn.andersson@linaro.org, andy.gross@linaro.org, David Brown , Li Yang , Shawn Guo , Sudeep Holla , linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org List-Id: devicetree@vger.kernel.org On Fri, May 10, 2019 at 04:59:39PM +0530, Amit Kucheria wrote: > The idle-states binding documentation[1] mentions that the This [1] reference is a null pointer ;) Other than that: Reviewed-by: Niklas Cassel > 'entry-method' property is required on 64-bit platforms and must be set > to "psci". > > We fixed up all uses of the entry-method property in > commit e9880240e4f4 ("arm64: dts: Fix various entry-method properties to > reflect documentation"). But a new one has appeared. Fix it up. > > Cc: Sudeep Holla > Signed-off-by: Amit Kucheria > --- > arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > index 2896bbcfa3bb..42e7822a0227 100644 > --- a/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > +++ b/arch/arm64/boot/dts/freescale/fsl-ls1028a.dtsi > @@ -51,7 +51,7 @@ > * PSCI node is not added default, U-boot will add missing > * parts if it determines to use PSCI. > */ > - entry-method = "arm,psci"; > + entry-method = "psci"; > > CPU_PH20: cpu-ph20 { > compatible = "arm,idle-state"; > -- > 2.17.1 >