Re: [PATCH v4 2/2] efi_loader: Set default console colors on efi_cout_clear_screen if needed

[Heinrich Schuchardt]

Am 18. Januar 2023 21:44:59 MEZ schrieb Jan Kiszka <jan.kiszka at siemens.com>:
>From: Jan Kiszka <jan.kiszka at siemens.com>
>
>Ensures a consistent background color of the whole screen for succeeding
>outputs as both demanded by the spec and implemented in EDK2 as well.
>
>Signed-off-by: Jan Kiszka <jan.kiszka at siemens.com>


Reviewed-by: Heinrich Schuchard <xypron.glpk at gmx.de>

>---
> lib/efi_loader/efi_console.c | 6 ++++++
> 1 file changed, 6 insertions(+)
>
>diff --git a/lib/efi_loader/efi_console.c b/lib/efi_loader/efi_console.c
>index e40c129d1b2..84b5f60b321 100644
>--- a/lib/efi_loader/efi_console.c
>+++ b/lib/efi_loader/efi_console.c
>@@ -497,6 +497,12 @@ static efi_status_t EFIAPI efi_cout_clear_screen(
> {
> 	EFI_ENTRY("%p", this);
> 
>+	/* Set default colors if not done yet */
>+	if (efi_con_mode.attribute == 0) {
>+		efi_con_mode.attribute = 0x07;
>+		printf(ESC "[0;37;40m");
>+	}
>+
> 	efi_clear_screen();
> 
> 	return EFI_EXIT(EFI_SUCCESS);