From: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org> To: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> Cc: Sudeep Holla <sudeep.holla-5wv7dgnIgG8@public.gmane.org>, linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>, Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, Jon Hunter <jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> Subject: Re: [PATCH 8/9] arm64: tegra: Enable PSCI on P3310 Date: Thu, 17 Nov 2016 17:21:34 +0000 [thread overview] Message-ID: <786f354d-a1f2-f2c6-fde7-7b1af3df756c@arm.com> (raw) In-Reply-To: <20161117171131.20062-8-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> On 17/11/16 17:11, Thierry Reding wrote: > From: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> > > The P3310 processor module comes ships with a firmware that implements > PSCI 1.0. Enable and use it to bring up all CPUs. > > Signed-off-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> > --- > arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi | 36 ++++++++++++++++++++++++++ > 1 file changed, 36 insertions(+) > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > index 807af7b68761..2c158c6809a5 100644 > --- a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > @@ -26,7 +26,43 @@ > status = "okay"; > }; > > + cpus { > + cpu@0 { > + enable-method = "psci"; > + }; > + > + cpu@1 { > + enable-method = "psci"; > + }; > + > + cpu@2 { > + enable-method = "psci"; > + }; > + > + cpu@3 { > + enable-method = "psci"; > + }; > + > + cpu@4 { > + enable-method = "psci"; > + }; > + > + cpu@5 { > + enable-method = "psci"; > + }; > + }; > + > bpmp { > status = "okay"; > }; > + > + psci { > + compatible = "arm,psci-1.0"; > + status = "okay"; > + method = "smc"; [...] > + > + cpu_off = <0x84000002>; > + cpu_on = <0xc4000003>; > + cpu_suspend = <0xc4000001>; These are applicable only for "arm,psci"(i.e. PSCI v0.1), so you need to drop them. -- Regards, Sudeep
WARNING: multiple messages have this Message-ID (diff)
From: sudeep.holla@arm.com (Sudeep Holla) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 8/9] arm64: tegra: Enable PSCI on P3310 Date: Thu, 17 Nov 2016 17:21:34 +0000 [thread overview] Message-ID: <786f354d-a1f2-f2c6-fde7-7b1af3df756c@arm.com> (raw) In-Reply-To: <20161117171131.20062-8-thierry.reding@gmail.com> On 17/11/16 17:11, Thierry Reding wrote: > From: Thierry Reding <treding@nvidia.com> > > The P3310 processor module comes ships with a firmware that implements > PSCI 1.0. Enable and use it to bring up all CPUs. > > Signed-off-by: Thierry Reding <treding@nvidia.com> > --- > arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi | 36 ++++++++++++++++++++++++++ > 1 file changed, 36 insertions(+) > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > index 807af7b68761..2c158c6809a5 100644 > --- a/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3310.dtsi > @@ -26,7 +26,43 @@ > status = "okay"; > }; > > + cpus { > + cpu at 0 { > + enable-method = "psci"; > + }; > + > + cpu at 1 { > + enable-method = "psci"; > + }; > + > + cpu at 2 { > + enable-method = "psci"; > + }; > + > + cpu at 3 { > + enable-method = "psci"; > + }; > + > + cpu at 4 { > + enable-method = "psci"; > + }; > + > + cpu at 5 { > + enable-method = "psci"; > + }; > + }; > + > bpmp { > status = "okay"; > }; > + > + psci { > + compatible = "arm,psci-1.0"; > + status = "okay"; > + method = "smc"; [...] > + > + cpu_off = <0x84000002>; > + cpu_on = <0xc4000003>; > + cpu_suspend = <0xc4000001>; These are applicable only for "arm,psci"(i.e. PSCI v0.1), so you need to drop them. -- Regards, Sudeep
next prev parent reply other threads:[~2016-11-17 17:21 UTC|newest] Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-11-17 17:11 [PATCH 1/9] arm64: tegra: Add Tegra186 support Thierry Reding 2016-11-17 17:11 ` Thierry Reding [not found] ` <20161117171131.20062-1-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2016-11-17 17:11 ` [PATCH 2/9] arm64: tegra: Add CPU nodes for Tegra186 Thierry Reding 2016-11-17 17:11 ` Thierry Reding 2016-11-17 17:11 ` [PATCH 3/9] arm64: tegra: Add serial ports on Tegra186 Thierry Reding 2016-11-17 17:11 ` Thierry Reding 2016-11-17 17:11 ` [PATCH 4/9] arm64: tegra: Add I2C controllers " Thierry Reding 2016-11-17 17:11 ` Thierry Reding 2016-11-17 17:11 ` [PATCH 5/9] arm64: tegra: Add SDHCI " Thierry Reding 2016-11-17 17:11 ` Thierry Reding 2016-11-17 17:11 ` [PATCH 6/9] arm64: tegra: Add GPIO " Thierry Reding 2016-11-17 17:11 ` Thierry Reding 2016-11-17 17:11 ` [PATCH 7/9] arm64: tegra: Add NVIDIA P3310 processor module support Thierry Reding 2016-11-17 17:11 ` Thierry Reding 2016-11-17 17:11 ` [PATCH 8/9] arm64: tegra: Enable PSCI on P3310 Thierry Reding 2016-11-17 17:11 ` Thierry Reding [not found] ` <20161117171131.20062-8-thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> 2016-11-17 17:21 ` Sudeep Holla [this message] 2016-11-17 17:21 ` Sudeep Holla [not found] ` <786f354d-a1f2-f2c6-fde7-7b1af3df756c-5wv7dgnIgG8@public.gmane.org> 2016-11-17 17:33 ` Thierry Reding 2016-11-17 17:33 ` Thierry Reding 2016-11-17 17:11 ` [PATCH 9/9] arm64: tegra: Add NVIDIA P2771 board support Thierry Reding 2016-11-17 17:11 ` Thierry Reding
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=786f354d-a1f2-f2c6-fde7-7b1af3df756c@arm.com \ --to=sudeep.holla-5wv7dgnigg8@public.gmane.org \ --cc=acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \ --cc=jonathanh-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \ --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \ --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \ --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \ --cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.