From: "Maciej W. Rozycki" <macro@linux-mips.org>
To: Peter Hurley <peter@hurleysoftware.com>
Cc: Robert Schwebel <r.schwebel@pengutronix.de>,
linux-serial@vger.kernel.org, linux-kernel@vger.kernel.org,
kernel@pengutronix.de
Subject: Re: earlycon: no match?
Date: Thu, 7 May 2015 18:09:25 +0100 (BST) [thread overview]
Message-ID: <alpine.LFD.2.11.1505071749370.1538@eddie.linux-mips.org> (raw)
In-Reply-To: <5547E447.9020604@hurleysoftware.com>
On Mon, 4 May 2015, Peter Hurley wrote:
> >> Since 2007, 'console=' is a early param synonym for 'earlycon='; IOW,
> >> the message is new but not the behavior.
> >
> > "console=" had nothing to do with early param before.
>
> *Yes, it has* since the commit I referenced in the previous email and
> the email before that when I noted that 'console=' and 'earlycon=' are
> synonyms if CONFIG_SERIAL_EARLYCON=y, and it has been that way since
> 2007.
>
> The only thing that has changed is that I added a diagnostic; _to repeat_,
> the earlycon matching code has always run in this case, only the
> diagnostic is new.
What's the point of having two parameters as synonyms whose syntax is not
compatible to each other in the general case? I'd expect the following
cases to be handled:
1. Regular console only (no early console requested) => `console=foo...'.
2. Both early and regular console => `earlycon=blah... console=foo...'.
3. Early console handing over to regular console => `earlycon=blah...'.
Why do you want to support `console=blah...' too (a question to be asked
back in 2007, but better late than never)?
If you do need to support such aliasing, then I suggest that you complain
about no early console match only if `earlycon=blah...' has been truly
used and avoid the warning where `console=blah...' has been used instead
(a warning about the latter can be issued where no regular console match
happened).
Maciej
next prev parent reply other threads:[~2015-05-07 17:09 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-03 21:10 Robert Schwebel
2015-05-04 14:01 ` Peter Hurley
2015-05-04 16:52 ` Peter Hurley
2015-05-04 19:42 ` Robert Schwebel
2015-05-04 20:21 ` Peter Hurley
2015-05-04 20:52 ` Robert Schwebel
2015-05-04 21:27 ` Peter Hurley
2015-05-04 21:58 ` Robert Schwebel
2015-05-04 22:31 ` Peter Hurley
2015-05-05 4:51 ` Robert Schwebel
2015-05-05 10:39 ` Peter Hurley
2015-05-05 16:32 ` Robert Schwebel
2015-05-05 18:15 ` Peter Hurley
2015-05-05 17:06 ` Fabio Estevam
2015-05-07 17:09 ` Maciej W. Rozycki [this message]
2015-05-07 17:22 ` Peter Hurley
2015-05-07 20:14 ` Maciej W. Rozycki
2015-05-07 21:13 ` Peter Hurley
2015-05-07 22:25 ` Maciej W. Rozycki
2015-05-07 22:37 ` Peter Hurley
2015-05-08 16:11 ` Maciej W. Rozycki
2015-05-08 6:34 ` Sascha Hauer
2015-05-08 10:12 ` Peter Hurley
2015-05-04 20:22 ` Sascha Hauer
2015-05-04 20:34 ` 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=alpine.LFD.2.11.1505071749370.1538@eddie.linux-mips.org \
--to=macro@linux-mips.org \
--cc=kernel@pengutronix.de \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-serial@vger.kernel.org \
--cc=peter@hurleysoftware.com \
--cc=r.schwebel@pengutronix.de \
/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®