On Monday, November 26, 2018 4:10:28 PM CET Andy Shevchenko wrote: > On Sun, Nov 25, 2018 at 09:28:44AM -0500, Prarit Bhargava wrote: > > On 11/21/2018 02:17 PM, Andy Shevchenko wrote: > > > On Wed, Nov 21, 2018 at 12:51:28PM -0500, Prarit Bhargava wrote: > > >> On 11/21/2018 08:43 AM, Andy Shevchenko wrote: > > >>> Since SPCR 1.04 [1] the baud rate of 0 means a preconfigured state of UART. > > >>> Assume firmware or bootloader configures console correctly. > > >>> > > >>> [1]: https://docs.microsoft.com/en-us/windows-hardware/drivers/serports/serial-port-console-redirection-table > > > > > >>> + if (!baud_rate) { > > >>> snprintf(opts, sizeof(opts), "%s,%s,0x%llx", uart, iotype, > > >>> table->serial_port.address); > > >>> } else { > > > > > >> OOC what happens if you pass in opts="uart,io,0x3f8,0" to setup_earlycon() and > > >> add_preferred_console() ? > > > > > > Nothing critical for 8250, though > > > a) we have a plenty of UART drivers and I don't check them; > > > b) the code above (avoidance of 0 to be baud rate parameter) is already in > > > upstream by some reason (see a) as well). > > > > > > So, I wouldn't touch that logic. > > > > > > > Agreed. I was thinking that if ",0" was accepted in that code it would > > lead towards a further cleanup. > > > > Reviewed-by: Prarit Bhargava <prarit@xxxxxxxxxx> > > Thanks for review! > > Rafael, I guess we are okay with this version if you have no objections. Applied, thanks!
