[edk2-devel] [PATCH v4 1/1] ArmPkg/SmbiosMiscDxe: Adjust the priority of getting firmware version
Leif Lindholm
quic_llindhol at quicinc.com
Tue Mar 28 15:10:28 UTC 2023
On Mon, Mar 27, 2023 at 01:35:43 +0700, Tinh Nguyen wrote:
> The BIOS Firmware Version in the SMBIOS Type 0 can be fetched from
> the fixed PcdFirmwareVersionString or platform specific OemMiscLib.
> In fact, the support from OemMiscLib comes into play when the firmware
> version may be modified at boot time for extended information.
> Therefore, the priority of getting the version from OemMiscLib should
> be higher.
>
> In case there is no modification in the OemMiscLib, we have to keep
> HII string STR_MISC_BIOS_VERSION empty or 'Not Specified'
> to indicate that the firmware version should be fetched from
> the PcdFirmwareVersionString.
>
> Signed-off-by: Tinh Nguyen <tinhnguyen at os.amperecomputing.com>
> Reviewed-by: Rebecca Cran <rebecca at bsdio.com>
> Reviewed-by: Oliver Smith-Denny <osd at smith-denny.com>
> Reviewed-by: Leif Lindholm <quic_llindhol at quicinc.com>
Thanks, merged as 07e17188df90.
/
Leif
> ---
>
> Changes since v1:
> + Change GetBiosVersion () to SetBiosVersion () and move the selection logic
> fully into SetBiosVersion ().
> Changes since v2:
> + Add R-B by Rebecca and remove @retval
> VOID
> Changes since v3:
> + Change the function's comment. Add R-B by Leif and Oliver.
>
> ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c | 58 ++++++++++++--------
> 1 file changed, 36 insertions(+), 22 deletions(-)
>
> diff --git a/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c b/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c
> index 66ead22a6e2c..c9f25b750254 100644
> --- a/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c
> +++ b/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c
> @@ -1,6 +1,6 @@
> /** @file
>
> - Copyright (c) 2022, Ampere Computing LLC. All rights reserved.<BR>
> + Copyright (c) 2022 - 2023, Ampere Computing LLC. All rights reserved.<BR>
> Copyright (c) 2021, NUVIA Inc. All rights reserved.<BR>
> Copyright (c) 2009, Intel Corporation. All rights reserved.<BR>
> Copyright (c) 2015, Hisilicon Limited. All rights reserved.<BR>
> @@ -124,22 +124,47 @@ GetBiosReleaseDate (
> return ReleaseDate;
> }
>
> -/**
> - Fetches the firmware ('BIOS') version from the
> - FirmwareVersionInfo HOB.
> +/** Fetches the Firmware version string for SMBIOS type 0
> +
> + This function first acquires the firmware version string from OemMiscLib;
> + if it is invalid, PcdFirmwareVersionString is used as a fallback,
> + and then sets it in SMBIOS type 0.
>
> - @return The version as a UTF-16 string
> **/
> -CHAR16 *
> -GetBiosVersion (
> +VOID
> +SetBiosVersion (
> VOID
> )
> {
> - CHAR16 *ReleaseString;
> + CHAR16 *DefaultVersionString;
> + CHAR16 *Version;
> + EFI_STRING_ID TokenToUpdate;
>
> - ReleaseString = (CHAR16 *)FixedPcdGetPtr (PcdFirmwareVersionString);
> + DefaultVersionString = HiiGetString (
> + mSmbiosMiscHiiHandle,
> + STRING_TOKEN (STR_MISC_BIOS_VERSION),
> + NULL
> + );
>
> - return ReleaseString;
> + OemUpdateSmbiosInfo (
> + mSmbiosMiscHiiHandle,
> + STRING_TOKEN (STR_MISC_BIOS_VERSION),
> + BiosVersionType00
> + );
> +
> + Version = HiiGetString (
> + mSmbiosMiscHiiHandle,
> + STRING_TOKEN (STR_MISC_BIOS_VERSION),
> + NULL
> + );
> +
> + if (((StrCmp (Version, DefaultVersionString) == 0) || (StrLen (Version) == 0))) {
> + Version = (CHAR16 *)FixedPcdGetPtr (PcdFirmwareVersionString);
> + if (StrLen (Version) > 0) {
> + TokenToUpdate = STRING_TOKEN (STR_MISC_BIOS_VERSION);
> + HiiSetString (mSmbiosMiscHiiHandle, TokenToUpdate, Version, NULL);
> + }
> + }
> }
>
> /**
> @@ -187,18 +212,7 @@ SMBIOS_MISC_TABLE_FUNCTION (MiscBiosVendor) {
> HiiSetString (mSmbiosMiscHiiHandle, TokenToUpdate, Vendor, NULL);
> }
>
> - Version = GetBiosVersion ();
> -
> - if (StrLen (Version) > 0) {
> - TokenToUpdate = STRING_TOKEN (STR_MISC_BIOS_VERSION);
> - HiiSetString (mSmbiosMiscHiiHandle, TokenToUpdate, Version, NULL);
> - } else {
> - OemUpdateSmbiosInfo (
> - mSmbiosMiscHiiHandle,
> - STRING_TOKEN (STR_MISC_BIOS_VERSION),
> - BiosVersionType00
> - );
> - }
> + SetBiosVersion ();
>
> Char16String = GetBiosReleaseDate ();
> if (StrLen (Char16String) > 0) {
> --
> 2.39.2
-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#102035): https://edk2.groups.io/g/devel/message/102035
Mute This Topic: https://groups.io/mt/97867123/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/leave/3943202/1813853/130120423/xyzzy [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-
More information about the edk2-devel-archive
mailing list