[U-Boot] [PATCH 1/1][for V2018.03] efi_loader: do not use 2.0.5 as UEFI revision number

Mark Kettenis mark.kettenis at xs4all.nl
Mon Feb 5 19:24:10 UTC 2018


> From: Heinrich Schuchardt <xypron.glpk at gmx.de>
> Date: Mon,  5 Feb 2018 18:04:21 +0100
> 
> Currently the UEFI revision number in the system table header is set to
> 2.0.5. This version number does not refer to any existing version of the
> UEFI standard.
> 
> Set the revision number to 2.7.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> ---
>  lib/efi_loader/efi_boottime.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

Heh.  I noticed this when I added code to OpenBSD/arm64 to print the
version number.  Didn't get around to submitting a diff yet.

Doesn't look like 2.7 has anything new in it that is mandatory, so I
guess it is as good as any version number out there.

Reviewed-by: Mark Kettenis <kettenis at openbsd.org>

> diff --git a/lib/efi_loader/efi_boottime.c b/lib/efi_loader/efi_boottime.c
> index 0fd272253d2..3b9184b3e24 100644
> --- a/lib/efi_loader/efi_boottime.c
> +++ b/lib/efi_loader/efi_boottime.c
> @@ -2893,7 +2893,7 @@ static uint16_t __efi_runtime_data firmware_vendor[] = L"Das U-Boot";
>  struct efi_system_table __efi_runtime_data systab = {
>  	.hdr = {
>  		.signature = EFI_SYSTEM_TABLE_SIGNATURE,
> -		.revision = 0x20005, /* 2.5 */
> +		.revision = 2 << 16 | 70, /* 2.7 */
>  		.headersize = sizeof(struct efi_table_hdr),
>  	},
>  	.fw_vendor = (long)firmware_vendor,
> -- 
> 2.14.2
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
> 


More information about the U-Boot mailing list