Re: [PATCH] serial: Set baudrate on boot
> This looks right to me. > > Can you please update the docs for setbrg() in serial.h to mention > that this method is always called after probing, so there is now no > need to set the baud rate in the probe() method? > Sure. Will do that in v2.
Re: [PATCH] serial: Set baudrate on boot
Hi Sean, On Sun, 2 Feb 2020 at 11:15, Sean Anderson wrote: > > Currently, the baud rate is never set on boot. This works ok when a previous > bootloader has configured the baudrate properly, or when the baudrate is set > to > a reasonable default in the serial driver's probe(). However, when this is not > the case, we could be using a different baud rate than what was configured. > > Signed-off-by: Sean Anderson > --- > drivers/serial/serial-uclass.c | 1 + > 1 file changed, 1 insertion(+) This looks right to me. Can you please update the docs for setbrg() in serial.h to mention that this method is always called after probing, so there is now no need to set the baud rate in the probe() method?
[PATCH] serial: Set baudrate on boot
Currently, the baud rate is never set on boot. This works ok when a previous bootloader has configured the baudrate properly, or when the baudrate is set to a reasonable default in the serial driver's probe(). However, when this is not the case, we could be using a different baud rate than what was configured. Signed-off-by: Sean Anderson --- drivers/serial/serial-uclass.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/serial/serial-uclass.c b/drivers/serial/serial-uclass.c index 0f5f1fa406..8aa542c32f 100644 --- a/drivers/serial/serial-uclass.c +++ b/drivers/serial/serial-uclass.c @@ -161,6 +161,7 @@ int serial_init(void) #if CONFIG_IS_ENABLED(SERIAL_PRESENT) serial_find_console_or_panic(); gd->flags |= GD_FLG_SERIAL_READY; + serial_setbrg(); #endif return 0; -- 2.25.0
