From: Geert Uytterhoeven <geert@linux-m68k.org>
To: Simon Horman <horms@verge.net.au>
Cc: Geert Uytterhoeven <geert+renesas@glider.be>,
Magnus Damm <magnus.damm@gmail.com>,
Linux-sh list <linux-sh@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: Re: [PATCH 00/13] ARM: shmobile: dts: Add chosen/stdout-path
Date: Tue, 30 Sep 2014 09:04:16 +0200 [thread overview]
Message-ID: <CAMuHMdXdobOLhwNfDaZPPuzQHV+j67__bkLeprLOuEx5jts2XQ@mail.gmail.com> (raw)
In-Reply-To: <20140930063029.GA30269@verge.net.au>
Hi Simon,
On Tue, Sep 30, 2014 at 8:30 AM, Simon Horman <horms@verge.net.au> wrote:
> On Tue, Sep 30, 2014 at 11:07:56AM +0900, Simon Horman wrote:
>> On Thu, Sep 25, 2014 at 06:13:16PM +0200, Geert Uytterhoeven wrote:
>> > This patch series adds chosen/stdout-path to the various shmobile DTS files
>> > that are used for reference or multi-platform builds.
>> >
>> > This serves two purposes:
>> > 1. It makes automatic console selection work in the absence of a "console="
>> > parameter on the kernel command line, allowing to remove "console="
>> > parameters from chosen/bootargs on many platforms.
>> > Note that these "console=" parameters are still needed in some cases:
>> > a. If console output is wanted on two consoles devices (graphical and
>> > serial consoles), like on armadillo and kzm9g,
>> > b. When using a non-default baud for the serial console, like on
>> > henninger and alt,
>> > 2. It will allow the PM domain code to find the PM domain for the console
>> > device, to make "no_console_suspend" work in the DT case.
>> > For now, this will matter on armadillo.
>> >
>> > The first two patches are small cleanups for emev2 dtsi, on top of which the
>> > change for kzm9d is built.
>>
>> Thanks, I have queued these up for v3.19.
>
> I spoke a little too soon.
>
> While testing these patches I noticed that the koelsch patch
> cases there to be no kernel output to the console (during boot)
> when booting using legacy C.
>
> I assume that this is due to removing
> console=ttySC6,115200 from bootargs.
Sorry, in my mind koelsch legacy is already gone ;-)
> I also assume this affects all boards that can still
> be booted using legacy C.
Only the ones where the DTS is shared between legacy and
reference/multi-platform.
Armadillo is not affected, as the console parameters are kept to have
output on both graphical and serial console.
So that leaves us with lager and koelsch only.
> I have dropped this series for now.
Do you want me to resend with the console= parameter kept for koelsch
and lager?
Or should we delay this whole series until koelsch and lager legacy
have been dropped?
IIUC, this is going to happen soon for koelsch, but not for lager?
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
next prev parent reply other threads:[~2014-09-30 7:04 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-09-25 16:13 [PATCH 00/13] ARM: shmobile: dts: Add chosen/stdout-path Geert Uytterhoeven
[not found] ` <1411661609-4280-1-git-send-email-geert+renesas-gXvu3+zWzMSzQB+pC5nmwQ@public.gmane.org>
2014-09-25 16:13 ` [PATCH 01/13] ARM: shmobile: emev2 dtsi: Use generic names for device nodes Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 03/13] ARM: shmobile: kzm9d dts: Add chosen/stdout-path Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 05/13] ARM: shmobile: ape6evm-reference " Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 12/13] ARM: shmobile: alt " Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 02/13] ARM: shmobile: emev2 dtsi: Add uart* labels for easier referencing Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 04/13] ARM: shmobile: genmai dts: Add chosen/stdout-path Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 06/13] ARM: shmobile: armadillo800eva " Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 07/13] ARM: shmobile: bockw-reference " Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 08/13] ARM: shmobile: marzen " Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 09/13] ARM: shmobile: lager " Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 10/13] ARM: shmobile: henninger " Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 11/13] ARM: shmobile: koelsch " Geert Uytterhoeven
2014-09-25 16:13 ` [PATCH 13/13] ARM: shmobile: kzm9g-reference " Geert Uytterhoeven
2014-09-30 2:07 ` [PATCH 00/13] ARM: shmobile: " Simon Horman
2014-09-30 6:30 ` Simon Horman
2014-09-30 7:04 ` Geert Uytterhoeven [this message]
2014-09-30 8:05 ` Simon Horman
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=CAMuHMdXdobOLhwNfDaZPPuzQHV+j67__bkLeprLOuEx5jts2XQ@mail.gmail.com \
--to=geert@linux-m68k.org \
--cc=devicetree@vger.kernel.org \
--cc=geert+renesas@glider.be \
--cc=horms@verge.net.au \
--cc=linux-sh@vger.kernel.org \
--cc=magnus.damm@gmail.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;
as well as URLs for NNTP newsgroup(s).