From: Aleksey Makarov <aleksey.makarov@linaro.org>
To: Rob Herring <robh+dt@kernel.org>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
"Rafael J . Wysocki" <rjw@rjwysocki.net>,
Len Brown <lenb@kernel.org>,
"linux-serial@vger.kernel.org" <linux-serial@vger.kernel.org>,
"linux-acpi@vger.kernel.org" <linux-acpi@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>,
Russell King <linux@arm.linux.org.uk>,
Leif Lindholm <leif.lindholm@linaro.org>,
Graeme Gregory <graeme.gregory@linaro.org>,
Al Stone <ahs3@redhat.com>,
Christopher Covington <cov@codeaurora.org>,
Yury Norov <ynorov@caviumnetworks.com>,
Peter Hurley <peter@hurleysoftware.com>,
"Zheng, Lv" <lv.zheng@intel.com>,
Frank Rowand <frowand.list@gmail.com>,
Grant Likely <grant.likely@linaro.org>,
Jiri Slaby <jslaby@suse.com>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [PATCH v5 1/6] of/serial: move earlycon early_param handling to serial
Date: Tue, 22 Mar 2016 19:55:32 +0300 [thread overview]
Message-ID: <56F17904.3070202@linaro.org> (raw)
In-Reply-To: <CAL_JsqL+YU7qSze40c+M5AaOvpZDFgvg=SsRGmpT3k_=L6P8JQ@mail.gmail.com>
On 03/22/2016 02:15 PM, Rob Herring wrote:
> On Tue, Mar 22, 2016 at 5:46 AM, Aleksey Makarov
> <aleksey.makarov@linaro.org> wrote:
>> From: Leif Lindholm <leif.lindholm@linaro.org>
>>
>> We have multiple "earlycon" early_param handlers - merge the DT one into
>> the main earlycon one. It's a cleanup that also will be useful
>> to decide if ACPI SPCR earlycon should be set up.
>
> How so? Isn't that determined by whether we have ACPI tables or a DT?
Oops, I missed this. This patch was not only to do parsing in one place,
but also to allow addition of code that decide which of ACPI/DT should be used
for earlycon initialization based on acpi_disabled.
I will fix this, thank you.
> This goes against trying to limit places of FDT parsing and keeping
> the early scanning all within fdt.c. At least it is not in the arch
> code. That said, I'm okay with this change if it helps.
>
> Rob
>
WARNING: multiple messages have this Message-ID (diff)
From: aleksey.makarov@linaro.org (Aleksey Makarov)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 1/6] of/serial: move earlycon early_param handling to serial
Date: Tue, 22 Mar 2016 19:55:32 +0300 [thread overview]
Message-ID: <56F17904.3070202@linaro.org> (raw)
In-Reply-To: <CAL_JsqL+YU7qSze40c+M5AaOvpZDFgvg=SsRGmpT3k_=L6P8JQ@mail.gmail.com>
On 03/22/2016 02:15 PM, Rob Herring wrote:
> On Tue, Mar 22, 2016 at 5:46 AM, Aleksey Makarov
> <aleksey.makarov@linaro.org> wrote:
>> From: Leif Lindholm <leif.lindholm@linaro.org>
>>
>> We have multiple "earlycon" early_param handlers - merge the DT one into
>> the main earlycon one. It's a cleanup that also will be useful
>> to decide if ACPI SPCR earlycon should be set up.
>
> How so? Isn't that determined by whether we have ACPI tables or a DT?
Oops, I missed this. This patch was not only to do parsing in one place,
but also to allow addition of code that decide which of ACPI/DT should be used
for earlycon initialization based on acpi_disabled.
I will fix this, thank you.
> This goes against trying to limit places of FDT parsing and keeping
> the early scanning all within fdt.c. At least it is not in the arch
> code. That said, I'm okay with this change if it helps.
>
> Rob
>
next prev parent reply other threads:[~2016-03-22 16:57 UTC|newest]
Thread overview: 49+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-03-22 10:46 [PATCH v5 0/6] ACPI: parse the SPCR table Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
[not found] ` <1458643595-14719-1-git-send-email-aleksey.makarov-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2016-03-22 10:46 ` [PATCH v5 1/6] of/serial: move earlycon early_param handling to serial Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
2016-03-22 11:15 ` Rob Herring
2016-03-22 11:15 ` Rob Herring
2016-03-22 16:55 ` Aleksey Makarov [this message]
2016-03-22 16:55 ` Aleksey Makarov
[not found] ` <1458643595-14719-2-git-send-email-aleksey.makarov-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>
2016-03-22 12:28 ` kbuild test robot
2016-03-22 12:28 ` kbuild test robot
2016-03-22 12:28 ` kbuild test robot
2016-03-22 10:46 ` [PATCH v5 2/6] ACPI: add definitions of DBG2 subtypes Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
2016-03-22 10:46 ` [PATCH v5 3/6] ACPI: parse SPCR and enable matching console Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
2016-03-22 11:09 ` Andy Shevchenko
2016-03-22 11:09 ` Andy Shevchenko
2016-03-22 16:07 ` Peter Hurley
2016-03-22 16:07 ` Peter Hurley
2016-03-22 17:04 ` Aleksey Makarov
2016-03-22 17:04 ` Aleksey Makarov
2016-03-22 17:21 ` Peter Hurley
2016-03-22 17:21 ` Peter Hurley
2016-03-22 12:26 ` Yury Norov
2016-03-22 12:26 ` Yury Norov
2016-03-22 12:26 ` Yury Norov
2016-03-22 14:57 ` Peter Hurley
2016-03-22 14:57 ` Peter Hurley
2016-03-22 16:51 ` Yury Norov
2016-03-22 16:51 ` Yury Norov
2016-03-22 16:51 ` Yury Norov
2016-03-22 17:08 ` Aleksey Makarov
2016-03-22 17:08 ` Aleksey Makarov
2016-03-22 17:32 ` Peter Hurley
2016-03-22 17:32 ` Peter Hurley
2016-03-22 10:46 ` [PATCH v5 4/6] ACPI: enable ACPI_SPCR_TABLE on ARM64 Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
2016-03-22 10:46 ` [PATCH v5 5/6] serial: pl011: add console matching function Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
2016-03-22 10:46 ` [PATCH v5 6/6] serial: pl011: add EARLYCON_DECLARE Aleksey Makarov
2016-03-22 10:46 ` Aleksey Makarov
2016-03-22 16:15 ` Peter Hurley
2016-03-22 16:15 ` Peter Hurley
2016-03-22 17:09 ` Aleksey Makarov
2016-03-22 17:09 ` Aleksey Makarov
2016-03-22 17:41 ` Peter Hurley
2016-03-22 17:41 ` Peter Hurley
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=56F17904.3070202@linaro.org \
--to=aleksey.makarov@linaro.org \
--cc=ahs3@redhat.com \
--cc=cov@codeaurora.org \
--cc=devicetree@vger.kernel.org \
--cc=frowand.list@gmail.com \
--cc=graeme.gregory@linaro.org \
--cc=grant.likely@linaro.org \
--cc=gregkh@linuxfoundation.org \
--cc=jslaby@suse.com \
--cc=leif.lindholm@linaro.org \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=lv.zheng@intel.com \
--cc=peter@hurleysoftware.com \
--cc=rjw@rjwysocki.net \
--cc=robh+dt@kernel.org \
--cc=ynorov@caviumnetworks.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 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.