Re: [PATCH 2/3] earlycon: Print a notice when uartclk is unknown

From: Petr Mladek
Date: Fri Sep 13 2024 - 11:36:03 EST


On Thu 2024-09-12 11:36:20, Raul E Rangel wrote:
> When trying to construct an earlycon=uart parameter it's hard to debug
> why it's not working. In my specific case it was because the default
> uartclk earlycon assumes doesn't match my hardware. This change adds a
> notice so that the user is made aware of that this assumption is being
> made. This should hopefully lead to them adding a <uartclk> option to
> their earlycon parameter.
>
> Booting with `console=uart,mmio32,0xfedc9000,115200n8`:
> [ 0.000000] earlycon: uart: Unknown uartclk, assuming 1843200hz
> [ 0.000000] earlycon: uart0 at MMIO32 0x00000000fedc9000 (options '115200n8')
>
> Signed-off-by: Raul E Rangel <rrangel@xxxxxxxxxxxx>

Looks good to me:

Reviewed-by: Petr Mladek <pmladek@xxxxxxxx>

Best Regards,
Petr