From: Daniel Thompson <daniel.thompson@linaro.org>
To: Mark Rutland <mark.rutland@arm.com>
Cc: Liviu Dudau <liviu.dudau@arm.com>,
Sudeep Holla <sudeep.holla@arm.com>,
Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>,
Rob Herring <robh+dt@kernel.org>,
Catalin Marinas <catalin.marinas@arm.com>,
Will Deacon <will.deacon@arm.com>,
linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
linux-kernel@vger.kernel.org, patches@linaro.org
Subject: Re: [PATCH] arm64: dts: foundation-v8: Enable PSCI mode
Date: Tue, 19 Sep 2017 16:59:15 +0100 [thread overview]
Message-ID: <f0710af4-d414-1f2a-7d86-dd559a50171f@linaro.org> (raw)
In-Reply-To: <20170918161248.GB5106@leverpostej>
On 18/09/17 17:12, Mark Rutland wrote:
> Hi Daniel,
>
> On Mon, Sep 18, 2017 at 04:38:32PM +0100, Daniel Thompson wrote:
>> Currently if the Foundation model is running ARM Trusted Firmware then
>> the kernel, which is configured to use spin tables, cannot start secondary
>> processors or "power off" the simulation.
>>
>> Add a couple of labels to the include file, and introduce a new .dts
>> file that uses these to override the enable-method.
>>
>> Signed-off-by: Daniel Thompson <daniel.thompson@linaro.org>
>
> This looks good, but has the unfortunate effect of leaving the
> (irrelevant) cpu-release-addr property in the PSCI dts files, as that's
> in the underlying dtsi file.
>
> Could we split spin-table / PSCI parts into separate dtsi files?
>
> e.g. have:
>
> * foundation-v8.dtsi
> * foundation-v8-gicv{2,3}.dtsi
> * foundation-v8-{psci,spin-table}.dtsi
>
> ... and then combine those to build the dts files we want.
Will do.
>
> FWIW, with that:
>
> Acked-by: Mark Rutland <mark.rutland@arm.com>
Talking about what we want... if it's all split out I might as well add
a gicv2+psci DT as well. Will that still retain your Acked-by or do you
want to see it first ;-) ?
Daniel.
next prev parent reply other threads:[~2017-09-19 15:59 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-09-18 15:38 Daniel Thompson
2017-09-18 16:12 ` Mark Rutland
2017-09-19 15:59 ` Daniel Thompson [this message]
2017-09-19 16:20 ` Mark Rutland
2017-09-19 18:32 ` [PATCH v2] " Daniel Thompson
2017-09-20 9:42 ` Sudeep Holla
2017-09-20 11:17 ` Daniel Thompson
2017-10-02 17:26 ` Sudeep Holla
2017-10-03 9:12 ` Daniel Thompson
2017-10-03 10:15 ` Ard Biesheuvel
2017-10-03 14:09 ` Sudeep Holla
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=f0710af4-d414-1f2a-7d86-dd559a50171f@linaro.org \
--to=daniel.thompson@linaro.org \
--cc=catalin.marinas@arm.com \
--cc=devicetree@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=liviu.dudau@arm.com \
--cc=lorenzo.pieralisi@arm.com \
--cc=mark.rutland@arm.com \
--cc=patches@linaro.org \
--cc=robh+dt@kernel.org \
--cc=sudeep.holla@arm.com \
--cc=will.deacon@arm.com \
/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: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®