From: ahs3@redhat.com (Al Stone)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 2/2] ARM64: kernel: unify ACPI and DT cpus initialization
Date: Wed, 13 May 2015 16:00:58 -0600 [thread overview]
Message-ID: <5553C99A.5020706@redhat.com> (raw)
In-Reply-To: <1431522769-11276-3-git-send-email-lorenzo.pieralisi@arm.com>
On 05/13/2015 07:12 AM, Lorenzo Pieralisi wrote:
> The code that initializes cpus on arm64 is currently split in two
> different code paths that carry out DT and ACPI cpus initialization.
>
> Most of the code executing SMP initialization is common and should
> be merged to reduce discrepancies between ACPI and DT initialization
> and to have code initializing cpus in a single common place in the
> kernel.
>
> This patch refactors arm64 SMP cpus initialization code to merge
> ACPI and DT boot paths in a common file and to create sanity
> checks that can be reused by both boot methods.
>
> Current code assumes PSCI is the only available boot method
> when arm64 boots with ACPI; this can be easily extended if/when
> the ACPI parking protocol is merged into the kernel.
>
> Signed-off-by: Sudeep Holla <sudeep.holla@arm.com>
> Signed-off-by: Lorenzo Pieralisi <lorenzo.pieralisi@arm.com>
> Acked-by: Hanjun Guo <hanjun.guo@linaro.org>
> Acked-by: Mark Rutland <mark.rutland@arm.com>
> Tested-by: Hanjun Guo <hanjun.guo@linaro.org>
> Tested-by: Mark Rutland <mark.rutland@arm.com> [DT]
> Cc: Will Deacon <will.deacon@arm.com>
> Cc: Catalin Marinas <catalin.marinas@arm.com>
[snip...]
These patches look pretty good so far, but I've only just started
reviewing them. This one raised the question: do we know if there
is someone already working on parking protocol? I've seen a form
of it working on AMD Seattles but I don't recall seeing any patches
being submitted -- I'll search archives again, just in case I missed
it the first time, but if there is someone working on it, that would
be good to know.
Thanks.
--
ciao,
al
-----------------------------------
Al Stone
Software Engineer
Red Hat, Inc.
ahs3 at redhat.com
-----------------------------------
next prev parent reply other threads:[~2015-05-13 22:00 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-13 13:12 [PATCH v2 0/2] ARM64: consolidate ACPI/DT SMP initialization Lorenzo Pieralisi
2015-05-13 13:12 ` [PATCH v2 1/2] ARM64: kernel: make cpu_ops hooks DT agnostic Lorenzo Pieralisi
2015-05-13 13:12 ` [PATCH v2 2/2] ARM64: kernel: unify ACPI and DT cpus initialization Lorenzo Pieralisi
2015-05-13 22:00 ` Al Stone [this message]
2015-05-15 8:58 ` Lorenzo Pieralisi
2015-07-13 15:00 ` Lorenzo Pieralisi
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=5553C99A.5020706@redhat.com \
--to=ahs3@redhat.com \
--cc=linux-arm-kernel@lists.infradead.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: link
Be 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.