Re: earlycon: no match?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux