[PATCH v2] x86: serial: ns16550: Allow the UART to be silently disabled

Bin Meng bmeng.cn at gmail.com
Sun Nov 5 15:11:52 CET 2023


Hi Simon,

On Thu, Nov 2, 2023 at 2:04 AM Simon Glass <sjg at chromium.org> wrote:
>
> U-Boot normally requires a UART. When booting from coreboot it is
> sometimes just not available, e.g. when no sysinfo or DBG2 information
> is provided.
>
> In this case we need to continue running, since the display can be used.
> Add a flag to disable serial for this case.
>
> This allows U-Boot to start up and operation from the display, instead
> of hanging on start-up.
>
> This could perhaps be hidden behind a Kconfig option to reduce code
> size.

I think using a generic Kconfig option and filtering this out in the
generic serial probe code is better than what is proposed in this
patch.

>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> Changes in v2:
> - Drop RFC tag since there were no comments
>
>  drivers/serial/ns16550.c         | 17 +++++++++++++++--
>  drivers/serial/serial_coreboot.c |  1 +
>  include/ns16550.h                |  1 +
>  3 files changed, 17 insertions(+), 2 deletions(-)
>

Regards,
Bin


More information about the U-Boot mailing list