Re: [PATCH] xen/arm: debug-pl011.inc: Use macros instead of hardcoded values

[Julien Grall <julien@xxxxxxx>]

On 16/11/2022 14:45, Michal Orzel wrote:
> Hi Julien,
Hi Michal,

> > > and use it in the pl011-debug files (+ there is a question whether we should 
> > > define WLEN_7-5 for completeness).
> > I would not define WLEN_7-5. That said, I wonder if we really need to
> > set the baud rate & co here?
> >
> > AFAICT the runtime driver never touch them. The reasoning is the
> > firmware is responsible to configure the serial. Therefore, I would
> > consider to drop the code (setting UARTCR might still be necessary).
> I do not really agree because the current behavior was done on purpose.
EARLY_UART_PL011_BAUD_RATE is only used for very early debugging (this 
is protected by CONFIG_DEBUG and CONFIG_EXPERT). This is not a 
production ready code.
> At the moment early_uart_init is called only if EARLY_UART_PL011_BAUD_RATE is 
> set to a value != 0.
> This is done in order to have flexibility to either stick to what 
> firmware/bootloader configured or to change this
> configuration by specifying the EARLY_UART_PL011_BAUD_RATE (useful when you do 
> not know what
> the firmware configured).
The chances are that you want to use the baud rate that was configured 
by the firmware. Otherwise, you would need to change the configuration 
of minicom (or whatever you used) to get proper output for the firmware 
and then Xen.
Furthermore, as I wrote before, the runtime driver doesn't configure the 
baud rate. This was removed in Xen 4.7 (see commit 2048e17ca9df 
"drivers/pl011: Don't configure baudrate") because it was buggy and this 
code is not simple.
So it makes no sense to configure the baud rate when using early printk 
but not the runtime driver.
So we have two choices:
 1) Remove the baud rate setting in the early uart code
 2) Support the baud rate in the runtime driver

I strongly prefer 1 so far because there are not any practical use to 
have a different baud rate for Xen and the firmware.
Cheers,

--
Julien Grall