Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 8:32 PM, Peter Hurley wrote: > On 04/03/2015 11:24 PM, Yinghai Lu wrote: >> On Fri, Apr 3, 2015 at 8:15 PM, Peter Hurley >> wrote: >>> On 04/03/2015 11:09 PM, Yinghai Lu wrote: On Fri, Apr 3, 2015 at 7:19 PM, Yinghai Lu wrote: > On Fri, Apr 3, 2015 at 5:04 PM, Pe

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Peter Hurley
On 04/03/2015 11:37 PM, Yinghai Lu wrote: > What is NVM here? nvm == nevermind -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 8:32 PM, Peter Hurley wrote: Just now noticed, this patch only fix partial problem. Kernel console is ok. But later initrd/init scripts revert back to 9600 again. >>> >>> Please share the init scripts. >> >> I am using opensuse 13.1 rescue disk as r

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Peter Hurley
On 04/03/2015 11:24 PM, Yinghai Lu wrote: > On Fri, Apr 3, 2015 at 8:15 PM, Peter Hurley wrote: >> On 04/03/2015 11:09 PM, Yinghai Lu wrote: >>> On Fri, Apr 3, 2015 at 7:19 PM, Yinghai Lu wrote: On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley wrote: > Restore undocumented behavior of

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 8:24 PM, Yinghai Lu wrote: > On Fri, Apr 3, 2015 at 8:15 PM, Peter Hurley wrote: >> On 04/03/2015 11:09 PM, Yinghai Lu wrote: >>> On Fri, Apr 3, 2015 at 7:19 PM, Yinghai Lu wrote: On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley wrote: > Restore undocumented be

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 8:09 PM, Peter Hurley wrote: > > All this automatic behavior and aliases for the same things just > adds unnecessary complexity and maintenance burden, without measurable > gain. > > The real tragedy is that all this was totally unnecessary; what was > wrong with "earlycon=

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 8:15 PM, Peter Hurley wrote: > On 04/03/2015 11:09 PM, Yinghai Lu wrote: >> On Fri, Apr 3, 2015 at 7:19 PM, Yinghai Lu wrote: >>> On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley >>> wrote: Restore undocumented behavior of kernel command line parameters of the forms

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Peter Hurley
On 04/03/2015 11:09 PM, Yinghai Lu wrote: > On Fri, Apr 3, 2015 at 7:19 PM, Yinghai Lu wrote: >> On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley >> wrote: >>> Restore undocumented behavior of kernel command line parameters of >>> the forms: >>> console=uart[8250],io|mmio|mmio32,[,options] >>>

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Peter Hurley
On 04/03/2015 10:56 PM, Yinghai Lu wrote: > On Fri, Apr 3, 2015 at 7:29 PM, Peter Hurley wrote: >> On 04/03/2015 10:19 PM, Yinghai Lu wrote: >>> On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley >>> wrote: Restore undocumented behavior of kernel command line parameters of the forms:

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 7:19 PM, Yinghai Lu wrote: > On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley wrote: >> Restore undocumented behavior of kernel command line parameters of >> the forms: >> console=uart[8250],io|mmio|mmio32,[,options] >> console=uart[8250],[,options] >> where 'options' h

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 7:50 PM, Peter Hurley wrote: > On 04/03/2015 10:29 PM, Peter Hurley wrote: >> On 04/03/2015 10:19 PM, Yinghai Lu wrote: >>> Another regression. >>> when user have >>> console=uart8250,io,0x3f8 console=uart8250,io,0x2f8 >>> >>> before your patchset: >>> port_0x3f8 is early co

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 7:29 PM, Peter Hurley wrote: > On 04/03/2015 10:19 PM, Yinghai Lu wrote: >> On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley >> wrote: >>> Restore undocumented behavior of kernel command line parameters of >>> the forms: >>> console=uart[8250],io|mmio|mmio32,[,options] >>>

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Peter Hurley
On 04/03/2015 10:29 PM, Peter Hurley wrote: > On 04/03/2015 10:19 PM, Yinghai Lu wrote: >> Another regression. >> when user have >> console=uart8250,io,0x3f8 console=uart8250,io,0x2f8 >> >> before your patchset: >> port_0x3f8 is early console, and will be console later. >> and port_0x2f8 is ignored

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Peter Hurley
On 04/03/2015 10:19 PM, Yinghai Lu wrote: > On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley wrote: >> Restore undocumented behavior of kernel command line parameters of >> the forms: >> console=uart[8250],io|mmio|mmio32,[,options] >> console=uart[8250],[,options] >> where 'options' have not b

Re: [PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Yinghai Lu
On Fri, Apr 3, 2015 at 5:04 PM, Peter Hurley wrote: > Restore undocumented behavior of kernel command line parameters of > the forms: > console=uart[8250],io|mmio|mmio32,[,options] > console=uart[8250],[,options] > where 'options' have not been specified; in this case, the hardware > is as

[PATCH v2] earlycon: 8250: Fix command line regression

2015-04-03 Thread Peter Hurley
Restore undocumented behavior of kernel command line parameters of the forms: console=uart[8250],io|mmio|mmio32,[,options] console=uart[8250],[,options] where 'options' have not been specified; in this case, the hardware is assumed to be initialized. Document the required behavior of the o