[PATCH 05/10] smbios: Set BIOS release version

Heinrich Schuchardt xypron.glpk at gmx.de
Sat Jan 9 20:27:51 CET 2021


On 1/7/21 5:21 AM, Simon Glass wrote:
> We may as well include the U-Boot release information in the type-0 table
> since it is designed for that purpose.
>
> U-Boot uses release versions based on the year and month. The year cannot
> fit in a byte, so drop the century.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
>   lib/smbios.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/lib/smbios.c b/lib/smbios.c
> index b1171f544a8..8e9c002dec2 100644
> --- a/lib/smbios.c
> +++ b/lib/smbios.c
> @@ -143,8 +143,8 @@ static int smbios_write_type0(ulong *current, int handle, ofnode node)
>   #endif
>   	t->bios_characteristics_ext2 = BIOS_CHARACTERISTICS_EXT2_TARGET;
>
> -	t->bios_major_release = 0xff;
> -	t->bios_minor_release = 0xff;
> +	t->bios_major_release = U_BOOT_VERSION_NUM % 100;

As seen in the commit message this needs explanation. Please add a
comment for future developers:

/* bios_major_release has only one byte, drop century */

Best regards

Heinrich

> +	t->bios_minor_release = U_BOOT_VERSION_NUM_PATCH;
>   	t->ec_major_release = 0xff;
>   	t->ec_minor_release = 0xff;
>
>



More information about the U-Boot mailing list