From: Catalin Marinas <catalin.marinas@arm.com>
To: G Gregory <graeme.gregory@linaro.org>
Cc: Arnd Bergmann <arnd@arndb.de>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
"hanjun.guo@linaro.org" <hanjun.guo@linaro.org>,
Mark Rutland <Mark.Rutland@arm.com>,
"linaro-acpi@lists.linaro.org" <linaro-acpi@lists.linaro.org>,
Will Deacon <Will.Deacon@arm.com>, Lv Zheng <lv.zheng@intel.com>,
Rob Herring <robh@kernel.org>,
Lorenzo Pieralisi <Lorenzo.Pieralisi@arm.com>,
Al Stone <al.stone@linaro.org>,
Daniel Lezcano <daniel.lezcano@linaro.org>,
Robert Moore <robert.moore@intel.com>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
"jcm@redhat.com" <jcm@redhat.com>,
"grant.likely@linaro.org" <grant.likely@linaro.org>,
Charles Garcia-Tobin <Charles.Garcia-Tobin@arm.com>,
Robert Richter <rric@kernel.org>,
Jason Cooper <jason@lakedaemon.net>,
Marc Zyngier <Marc.Zyngier@arm.com>,
Liviu Dudau <Liviu.Dudau@arm.com>,
Mark Brown <broonie@kernel.org>,
Bjorn Helgaas <bhelgaas@google.com>,
"Kangkang.Shen@huawei.com" <Kangkang.Shen@huawei.com>,
Randy Dunlap <rdunlap@infradead.org>,
"Rafael J. Wysocki" <rjw@rjwysocki.net>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Sudeep Holla <Sudeep.Holla@arm.com>,
Olof Johansson <olof@lixom.net>
Subject: Re: [PATCH v5 18/18] Documentation: ACPI for ARM64
Date: Tue, 6 Jan 2015 14:03:06 +0000 [thread overview]
Message-ID: <20150106140305.GE8829@e104818-lin.cambridge.arm.com> (raw)
In-Reply-To: <CAGCh1+kYRPg8QJ57bamH7Ctb1vJKmn2rirMUpbXcrU=Qw_VF2A@mail.gmail.com>
On Tue, Jan 06, 2015 at 01:51:27PM +0000, G Gregory wrote:
> On 6 January 2015 at 11:20, Catalin Marinas <catalin.marinas@arm.com> wrote:
> > On Mon, Jan 05, 2015 at 08:16:30PM +0000, Arnd Bergmann wrote:
> >> On Monday 05 January 2015 13:13:02 Catalin Marinas wrote:
> >> > > since passing no DT tables to OS but
> >> > > acpi=force is missing is a corner case, we can do a follow up patch to
> >> > > fix that, does it make sense?
> >> >
> >> > Not entirely. Why would no dtb and no acpi=force be a corner case? I
> >> > thought this should be the default when only ACPI tables are passed, no
> >> > need for an additional acpi=force argument.
> >>
> >> We don't really support the case of only ACPI tables for now. The expectation
> >> is that you always have working DT support, at least for the next few years
> >> as ACPI features are ramping up, and without acpi=force it should not try
> >> to use ACPI at all.
> >
> > So if both DT and ACPI are present, just use DT unless acpi=force is
> > passed. So far I think we agree but what I want to avoid is always
> > mandating acpi=force even when the DT tables are missing (in the long
> > run).
> >
> > Now, what's preventing a vendor firmware from providing only ACPI
> > tables? Do we enforce it in some way (arm-acpi.txt, kernel warning etc.)
> > that both DT and ACPI are supported, or at least that dts files are
> > merged in the kernel first?
>
> How do we tell the difference between a DT passed purely for booting purposes
> ie a skeleton DT. And one which actually has hardware description as this needs
> to be done before unpacking the DT.
You do the check in EFI_STUB (part of the kernel). An ACPI-only firmware
should never pass a DT to the kernel, not even for booting purposes
(EFI_STUB builds one on the fly to pass parameters to the kernel but
that's purely an internal kernel decision).
There are other ways as well, but if we decide to enforce this, I think
EFI_STUB is the best option.
--
Catalin
next prev parent reply other threads:[~2015-01-06 14:03 UTC|newest]
Thread overview: 109+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-10-17 13:36 [PATCH v5 00/18] Introduce ACPI for ARM64 based on ACPI 5.1 Hanjun Guo
2014-10-17 13:36 ` [PATCH v5 01/18] ARM64: Move the init of cpu_logical_map(0) before unflatten_device_tree() Hanjun Guo
2014-11-18 13:45 ` Hanjun Guo
2014-11-18 16:43 ` Catalin Marinas
2014-11-18 16:57 ` Will Deacon
2014-11-18 17:02 ` Sudeep Holla
2014-11-18 17:03 ` Will Deacon
2014-11-19 0:29 ` Hanjun Guo
2014-10-17 13:36 ` [PATCH v5 02/18] ACPI / table: Add new function to get table entries Hanjun Guo
2014-11-24 1:27 ` Rafael J. Wysocki
2014-11-24 11:03 ` Hanjun Guo
2014-11-24 14:51 ` Rafael J. Wysocki
2014-11-25 3:38 ` Hanjun Guo
2014-11-25 21:20 ` Rafael J. Wysocki
2014-11-26 1:42 ` Hanjun Guo
2014-10-17 13:36 ` [PATCH v5 03/18] ACPI / table: Count matched and successfully parsed entries without specifying max entries Hanjun Guo
2014-11-18 13:51 ` Hanjun Guo
2014-11-18 20:15 ` Rafael J. Wysocki
2014-11-19 0:34 ` Hanjun Guo
2014-11-24 1:45 ` Rafael J. Wysocki
2014-11-24 8:34 ` Tomasz Nowicki
2014-11-24 15:16 ` Rafael J. Wysocki
2014-11-24 15:01 ` Tomasz Nowicki
2014-11-24 15:37 ` Rafael J. Wysocki
2014-11-24 15:18 ` Tomasz Nowicki
2014-10-17 13:37 ` [PATCH v5 04/18] ARM64 / ACPI: Get RSDP and ACPI boot-time tables Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 05/18] ARM64 / ACPI: Introduce sleep-arm.c Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 06/18] ARM64 / ACPI: Introduce early_param for "acpi" and pass acpi=force to enable ACPI Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 07/18] ARM64 / ACPI: If we chose to boot from acpi then disable FDT Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 08/18] ARM64 / ACPI: Make PCI optional for ACPI on ARM64 Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 09/18] ARM64 / ACPI: Parse FADT table to get PSCI flags for PSCI init Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 10/18] ACPI / table: Print GIC information when MADT is parsed Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 11/18] ARM64 / ACPI: Parse MADT for SMP initialization Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 12/18] ACPI / processor: Make it possible to get CPU hardware ID via GICC Hanjun Guo
2014-10-24 17:39 ` Lorenzo Pieralisi
2014-10-27 9:58 ` Hanjun Guo
2014-10-29 10:43 ` Lorenzo Pieralisi
2014-10-30 8:27 ` Hanjun Guo
2014-10-29 21:33 ` Rafael J. Wysocki
2014-10-30 8:30 ` Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 13/18] ARM64 / ACPI: Introduce ACPI_IRQ_MODEL_GIC and register device's gsi Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 14/18] ARM64 / ACPI: Add GICv2 specific ACPI boot support Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 15/18] ARM64 / ACPI: Parse GTDT to initialize arch timer Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 16/18] ARM64 / ACPI: Select ACPI_REDUCED_HARDWARE_ONLY if ACPI is enabled on ARM64 Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 17/18] ARM64 / ACPI: Enable ARM64 in Kconfig Hanjun Guo
2014-10-17 13:37 ` [PATCH v5 18/18] Documentation: ACPI for ARM64 Hanjun Guo
2014-12-18 20:01 ` Suravee Suthikulanit
2014-12-19 13:04 ` Hanjun Guo
2014-12-18 20:04 ` Timur Tabi
2014-12-19 13:53 ` Hanjun Guo
2014-12-24 17:18 ` Catalin Marinas
2014-12-24 19:33 ` Jon Masters
2014-12-26 13:23 ` Mark Brown
2014-12-30 11:23 ` Hanjun Guo
2015-01-05 13:13 ` Catalin Marinas
2015-01-05 20:16 ` Arnd Bergmann
2015-01-06 11:20 ` Catalin Marinas
2015-01-06 13:51 ` G Gregory
2015-01-06 14:03 ` Catalin Marinas [this message]
2015-01-06 13:59 ` [Linaro-acpi] " Arnd Bergmann
2015-01-06 14:11 ` Catalin Marinas
2015-01-06 19:30 ` Arnd Bergmann
2015-01-15 14:10 ` Grant Likely
2015-01-15 15:51 ` Jon Masters
2015-01-15 16:52 ` Arnd Bergmann
2015-01-15 17:22 ` Al Stone
2015-01-16 16:35 ` Arnd Bergmann
2015-01-15 18:00 ` Mark Brown
2015-01-06 16:24 ` Jon Masters
2015-01-06 19:21 ` [Linaro-acpi] " Arnd Bergmann
2015-01-06 22:06 ` Jon Masters
2015-01-07 4:55 ` Jon Masters
2015-01-07 10:36 ` Arnd Bergmann
2015-01-07 11:50 ` Catalin Marinas
2015-01-07 13:06 ` Arnd Bergmann
2015-01-07 17:27 ` Mark Brown
2015-01-07 17:44 ` Jon Masters
2015-01-07 19:48 ` Arnd Bergmann
2015-01-07 20:05 ` Mark Brown
2015-01-07 20:14 ` Jon Masters
2015-01-09 10:33 ` Catalin Marinas
2015-01-09 10:55 ` Arnd Bergmann
2015-01-09 15:13 ` Catalin Marinas
2015-01-07 18:41 ` Jason Cooper
2015-01-07 19:58 ` Jon Masters
2015-01-07 20:05 ` Jon Masters
2015-01-07 22:59 ` Jason Cooper
2015-01-08 11:26 ` Arnd Bergmann
2015-01-08 19:59 ` Kangkang Shen
2015-01-07 21:40 ` Jason Cooper
2015-01-07 22:10 ` Jon Masters
2015-01-04 9:39 ` Hanjun Guo
2015-01-05 11:05 ` Catalin Marinas
2015-01-06 11:11 ` Hanjun Guo
2015-01-06 11:29 ` Catalin Marinas
2015-01-06 13:50 ` Hanjun Guo
2015-01-06 13:54 ` G Gregory
2015-01-06 13:59 ` Hanjun Guo
2015-01-06 14:05 ` Arnd Bergmann
2015-01-06 14:16 ` Catalin Marinas
2015-01-06 14:37 ` Charles Garcia-Tobin
2015-01-06 16:37 ` Jon Masters
2015-01-09 23:12 ` Arnd Bergmann
[not found] ` <CAJ5Y-eZ5cu9_OhG24yAv+CZq7zKg0vU+eVGekyN+9dDzaz1OhQ@mail.gmail.com>
2014-12-30 20:13 ` ashwinc
2014-12-31 8:34 ` Hanjun Guo
2014-12-31 15:08 ` ashwinc
2015-01-01 20:04 ` Graeme Gregory
2015-01-02 9:28 ` Hanjun Guo
2015-01-02 16:47 ` Catalin Marinas
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=20150106140305.GE8829@e104818-lin.cambridge.arm.com \
--to=catalin.marinas@arm.com \
--cc=Charles.Garcia-Tobin@arm.com \
--cc=Kangkang.Shen@huawei.com \
--cc=Liviu.Dudau@arm.com \
--cc=Lorenzo.Pieralisi@arm.com \
--cc=Marc.Zyngier@arm.com \
--cc=Mark.Rutland@arm.com \
--cc=Sudeep.Holla@arm.com \
--cc=Will.Deacon@arm.com \
--cc=al.stone@linaro.org \
--cc=arnd@arndb.de \
--cc=bhelgaas@google.com \
--cc=broonie@kernel.org \
--cc=daniel.lezcano@linaro.org \
--cc=graeme.gregory@linaro.org \
--cc=grant.likely@linaro.org \
--cc=hanjun.guo@linaro.org \
--cc=jason@lakedaemon.net \
--cc=jcm@redhat.com \
--cc=linaro-acpi@lists.linaro.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lv.zheng@intel.com \
--cc=olof@lixom.net \
--cc=rdunlap@infradead.org \
--cc=rjw@rjwysocki.net \
--cc=robert.moore@intel.com \
--cc=robh@kernel.org \
--cc=rric@kernel.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 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®