Re: [PATCH v2 30/39] efi: Show the vendor in helloworld

[Heinrich Schuchardt <xypron.glpk@gmx.de>]

On 06.08.24 14:58, Simon Glass wrote:
> Show the vendor name so it is clear that this app has been built by
> U-Boot.

As described in the UEFI specification the system table is provided by
the UEFI firmware to the entry-point of EFI binaries. So on a Laptop you
might see "Xiaomi" or "Huawei" displayed when running through the new
lines. "U-Boot" will only show up when running on U-Boot.

>
> Signed-off-by: Simon Glass <sjg@chromium.org>
> ---
>
> (no changes since v1)
>
>   lib/efi_loader/helloworld.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/lib/efi_loader/helloworld.c b/lib/efi_loader/helloworld.c
> index 586177de0c8..f120249126d 100644
> --- a/lib/efi_loader/helloworld.c
> +++ b/lib/efi_loader/helloworld.c
> @@ -237,6 +237,9 @@ efi_status_t EFIAPI efi_main(efi_handle_t handle,
>   			(con_out, u"Missing device path for device handle\r\n");
>   		goto out;
>   	}
> +	con_out->output_string(con_out, u"Vendor: ");

%s/Vendor/Firmware vendor/

Best regards

Heinrich

> +	con_out->output_string(con_out, systab->fw_vendor);
> +	con_out->output_string(con_out, u"\n");
>   	con_out->output_string(con_out, u"Boot device: ");
>   	ret = print_device_path(device_path, device_path_to_text);
>   	if (ret != EFI_SUCCESS)