From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (NAM10-DM6-obe.outbound.protection.outlook.com [40.107.93.121]) by mx.groups.io with SMTP id smtpd.web12.13330.1661359907615871676 for ; Wed, 24 Aug 2022 09:51:47 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@os.amperecomputing.com header.s=selector2 header.b=sOaUHmDM; spf=pass (domain: os.amperecomputing.com, ip: 40.107.93.121, mailfrom: nhi@os.amperecomputing.com) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Zkbi80xUzcsQb1yKdUEXcBTGpiUCg4AjjnmZDBoxzFMKA94YtXr8Yl9jA4qS+7TqXec+IGIVXUJ1fU6lmmaPYaFJNvpsEvYaKFZPWqPDZNojtDihpAYv4VhUqbVQqNTJ+nl22qt2e4z+6Ux0rSyL7oh4xnuQTk+DW5cr4/42qJkO0WpAWZhJ0awXhhZjEyHM/gB9x/8cUN70BehxntKtDNQpbZgL3wATRjl22YNXjHd+Q1JWALIjy8sdeC2Ytg5gkv18kr5K++KepqZV/gFIpgQhMpihiIV4a7/qP3MDZVDmFEg8lF8EDQVSdXQDaVuAvoqgkKWwt7w4OtmfXNCjug== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=+IztdDMjl4z0hhuwSpFyAecqzg9yDkjPFCjs5vrsJSo=; b=T1OZmTnyfbEBWvtqZBFQfa4U8TKfauuUABLKzBUtKYxZF7xIHNplhDRXT3fntNJav+C8+x+NXJ9uooJ6okuMjQPLwjf+V0WAjhmHy9cPHLeTPXDF8nH9uubHnteobN+4gAQdblxpD/bqU1GqMw6X+CIsqBQpb4+ptEWVsXtSGNUXBpAwscEsvvrkW/lD32a1krdKzblOwMzY2uwY8j6ceIkgFjANKmqz40MY4CrDtE6j+DQ9Q7PR7d2793EiD+PZi98hFQfyRuIAVkEF5PWOTzdXulQ9R6DkbrykndSf/lndinlUXG0pvJasiNnPScmW+fBgESYVwJUt7R5b3xyuKA== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=os.amperecomputing.com; dmarc=pass action=none header.from=os.amperecomputing.com; dkim=pass header.d=os.amperecomputing.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=os.amperecomputing.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=+IztdDMjl4z0hhuwSpFyAecqzg9yDkjPFCjs5vrsJSo=; b=sOaUHmDMCNNKc3QLxrZvL6LBxHpdQxAhP8xM4g+TfbrED/PzpQUY/b8/kxHBb84zsclcwd+B1sVtuvi6Hkvrfq1DIpJNaviCUyCBtwLKvhAaJ+NpHF5HHc8D2V+Rw9W9xUAUfYwLD3I8ETVJ/vP7WoLUzF7nZitq/7FL47QtFz0= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=os.amperecomputing.com; Received: from PH0PR01MB7287.prod.exchangelabs.com (2603:10b6:510:10a::21) by DM4PR01MB7642.prod.exchangelabs.com (2603:10b6:8:62::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5566.15; Wed, 24 Aug 2022 16:51:46 +0000 Received: from PH0PR01MB7287.prod.exchangelabs.com ([fe80::bceb:a82f:b3ef:53b9]) by PH0PR01MB7287.prod.exchangelabs.com ([fe80::bceb:a82f:b3ef:53b9%5]) with mapi id 15.20.5546.024; Wed, 24 Aug 2022 16:51:46 +0000 From: "Nhi Pham" To: devel@edk2.groups.io, quic_llindhol@quicinc.com, ardb+tianocore@kernel.org, sami.mujawar@arm.com, quic_rcran@quicinc.com Cc: patches@amperecomputing.com, Minh Nguyen , Nhi Pham , Rebecca Cran Subject: [PATCH v2 6/6] ArmPkg/SmbiosMiscDxe: Get SMBIOS information from OemMiscLib Date: Wed, 24 Aug 2022 23:50:32 +0700 Message-Id: <20220824165032.1784305-7-nhi@os.amperecomputing.com> X-Mailer: git-send-email 2.25.1 In-Reply-To: <20220824165032.1784305-1-nhi@os.amperecomputing.com> References: <20220824165032.1784305-1-nhi@os.amperecomputing.com> X-ClientProxiedBy: SGXP274CA0001.SGPP274.PROD.OUTLOOK.COM (2603:1096:4:b8::13) To PH0PR01MB7287.prod.exchangelabs.com (2603:10b6:510:10a::21) Return-Path: nhi@os.amperecomputing.com MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 0db9076a-7e90-43ea-ec51-08da85f0edee X-MS-TrafficTypeDiagnostic: DM4PR01MB7642:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ahN7o1QmBskEWAkMzwgJq9dwcoTk/clD7NHSRaUSQTBV437Cisi985MeRi745JM/eWkfUG5hfBseV74/2ka9PItjoCbuDaWLRopEaUvscB6cDbvCiP4B+Y55QEjngZg0ZwcZLMp9AdJZRMAh1v1pncbbcSnBz/H2qaSsS1ahxt2Dzy3nVgMAra44ogstunqMJoMCH1ZueuTHpB9LGbQHIoqCwFtLFJXL4KTDyorsriPZBOYIDdDzHg0/rPD3z7uSVN72fR/mVxvqrPdy1e/E2KXDVu7DncWjhcLI7XTZWzNv3aQ9XqyuuNvoq1P3KruiKk7KEtfGRcB4GgZr+d2Dn3ertic6HAaPXpNj84U6AO/lUd8rat/kBABawtW1B1bpUBdK4ClxiJTwjM0CL1/050bguxauxQpbL/8TDQ4E832ns89UOveS5O9dGb61b8iaOekN+kNdX8SuFuDtpT++vP/8rdSm9BUrEJd7e5tAb2UL6I9clWpokN2ETwiRz1JhriVGqqQE4qlgB9NpP+gizz11/JG912MQEMW0Xm1vz9++j/XkDH8b6sSPf3X/Tftj7dK1a7bR/WY73mnxjxDK0c4kUPDl+xc3/oV4XBITwvVJMgvgWiQX74QNzdAU1F0DCbczdLD8yfvAa//lzA+Lh14pT9/aN0pe3PhBFRkPhONNs4K9hunpW8193FjW9JAXgCgnT3Nyk3PLCwHWBJfBVq/H4ntvc8qkjnrejM9Hii5O/ouRRi10VZNBKOKIQIODMLgxpv0f4ok4gQWHFlo13bR8/XDveSGSmuwJOS9TBvg= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH0PR01MB7287.prod.exchangelabs.com;PTR:;CAT:NONE;SFS:(13230016)(4636009)(396003)(346002)(376002)(136003)(366004)(39850400004)(41300700001)(6506007)(6666004)(6512007)(52116002)(26005)(86362001)(186003)(83380400001)(2616005)(1076003)(6486002)(478600001)(316002)(66556008)(54906003)(4326008)(8676002)(66476007)(66946007)(8936002)(38350700002)(38100700002)(5660300002)(2906002)(44824005);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?qngot25AqwdFiO3jYedjOW95ytjJX1Hj6uZmbrwMvqAgiHqW4Zl1RzJvySYy?= =?us-ascii?Q?/Ybx1ZyQgDAYg8Dta6S8z3kvMGAyHtH3qICfV8gE3F+9WMsqUfr7DDkmIqCC?= =?us-ascii?Q?jrTfYo60jI9KDN9B/asb0vHjJWusrNpaBWRScm/Wj7EEbg4XE1QZVsXQ3bWq?= =?us-ascii?Q?7zty4rCxKDD+2g74obxVejK+Z+QKdcYbyr+5Oj20CCJiVa4o5GcVuS5Sqn/Y?= =?us-ascii?Q?2PRIOJDLcSNP1CJjumhk+2L1Hl1mqDBwpZGG1y3WlQJ8SABT52/5xL2tBKQc?= =?us-ascii?Q?a34R1nEP7xFTFhlWFAyRViufetMD2LtAAK0N1K+248HTQ2EmeWAaoF7XFFfl?= =?us-ascii?Q?E8eTIXAlhUB0uHHFV0OqSc1syQ0ObXiM7DGQums5yUo3uyY7WidYwei7+oGv?= =?us-ascii?Q?xi9Qjn27ITcK3OCn6gUjgixfn29pTurdWMa1eMYPL3REJULkVoly098ZAXxP?= =?us-ascii?Q?6uCT1JY2x2vbARR7Bivjd5Geoc/v+la6luM7iZHIKWB+zzrx0OuQM4pgwqZN?= =?us-ascii?Q?oEuhFIzLqaVpPIXVDlPEw0c8WA9wfHkcgntD8Xr69lW6Vb39obL3RIaVAcw1?= =?us-ascii?Q?Mw3f8hJ5EeYpexhJQaDn/R6pdkJv+7PiP1pFrxfIftohAmA4bwqYpSWM1RBl?= =?us-ascii?Q?fOuGEm070pcOGeHtnwS9fWIpusrClA34Ad/6Ony9FGkixlRZskcsxOh+MuYW?= =?us-ascii?Q?PA2Bb2/T7gJ452T4yeyFTRqTj5fKkW1k0DdUFGsbfE8mhwX0Q5MknenDbwpv?= =?us-ascii?Q?969Qb0+kd0i09VSpTMCbxtZNcO7BjFwl5HIGMyBoNC/C3ZI2b1jlezMo+HR9?= =?us-ascii?Q?hKcHBN7qesh1LY/BKnMnCEsfvCRROcLwEmPx3XFWOoAlIhqVxR71XixyKkOX?= =?us-ascii?Q?HBvcX7PDRr6utNNDPkZ2f5xjb2P1vbOcLYQ2nJJeRpqsmkGBV/4piPihSOkJ?= =?us-ascii?Q?qhC/cOCZWVRVT6MZ7TIlrlBThEPE1Km07IEGUrHXF7tmkVFrUHYMOXX7N8yw?= =?us-ascii?Q?afRkRZ9H9jp0CBELoUNNT0FFNGM/hj/cKML3SupZurOogfj4SefKBp+AcoUZ?= =?us-ascii?Q?pSCi9X24nz3TL4LBBtJsTCB/5gwGXdNBnYrVNjB6xf7cWlBF0f1yF/fBf1XU?= =?us-ascii?Q?zzMCMJ4rusIEZ8imP0TeE9oItpAS5DSG64c6Hn6nVe8Z1Tuu8YBmUBqkT7xz?= =?us-ascii?Q?BP6EbBnTyA3XgjdTY042Yk6vcziRpYepoRpf7AcHfkRtP3YLHfSJ5cS/MX8z?= =?us-ascii?Q?c5r+WtQ1Ft+SvJZv0TQQysdG9kW9iAPM0bfdoylhpEKNdSDiupXS1+eL6l8W?= =?us-ascii?Q?S/JEQgNK0eYpjoVK5aJKZOkPpDDijJZW+Y3quKMkWp7+a3l34cwV5A7vll2h?= =?us-ascii?Q?JyZU56H80iK2kX989QLDCZRP7/V9TqyE/jBAzqerS80XY1cnDuvLqbt8B7N5?= =?us-ascii?Q?v7eK9xEvrdgrJX/GjHLugnEaIY+9gH/G3cUfqXNUD3jdAZqbH0nYKOkOBG1a?= =?us-ascii?Q?EuSfSS4SCHUsyjifCUoi2nhm3TmGMIKnaU/W5p3rSK4xAhzRD+NsjUwKBH8v?= =?us-ascii?Q?ZJyXd6y19OjP3Njq68FOqrmWs/I0F0KpLU6r09EGJoHKmd73i83qRaJkx8v7?= =?us-ascii?Q?YlEGjTsndwQzN6FeQOJ0NRA=3D?= X-OriginatorOrg: os.amperecomputing.com X-MS-Exchange-CrossTenant-Network-Message-Id: 0db9076a-7e90-43ea-ec51-08da85f0edee X-MS-Exchange-CrossTenant-AuthSource: PH0PR01MB7287.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Aug 2022 16:51:45.9349 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: JmcOCpwo6qkqEfm8cJ3hmeJcevSD2uaQIbD21pcoSBB4LReHTuNP2BhsMFUfjf+mHWjtBMG6h+1PDh+jxEJXf4T2s6yhfS9naMf2mvU5Ge8= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM4PR01MB7642 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain From: Minh Nguyen In some scenarios, the information of Bios Version, Bios Release and Embedded Controller Firmware Release are fetched during UEFI booting. This patch supports updating those fields dynamically when the PCDs are empty. Signed-off-by: Nhi Pham Reviewed-by: Rebecca Cran Reviewed-by: Sami Mujawar --- ArmPkg/Include/Library/OemMiscLib.h | 21= +++++++++++++ ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c | 28= +++++++++++++++++ ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c | 32= +++++++++++++------- 3 files changed, 70 insertions(+), 11 deletions(-) diff --git a/ArmPkg/Include/Library/OemMiscLib.h b/ArmPkg/Include/Library/O= emMiscLib.h index 1936619d9b5b..541274999e5c 100644 --- a/ArmPkg/Include/Library/OemMiscLib.h +++ b/ArmPkg/Include/Library/OemMiscLib.h @@ -37,6 +37,7 @@ typedef struct { } OEM_MISC_PROCESSOR_DATA;=0D =0D typedef enum {=0D + BiosVersionType00,=0D ProductNameType01,=0D SerialNumType01,=0D UuidType01,=0D @@ -247,4 +248,24 @@ OemGetSystemUuid ( OUT GUID *SystemUuid=0D );=0D =0D +/** Fetches the BIOS release.=0D +=0D + @return The BIOS release.=0D +**/=0D +UINT16=0D +EFIAPI=0D +OemGetBiosRelease (=0D + VOID=0D + );=0D +=0D +/** Fetches the embedded controller firmware release.=0D +=0D + @return The embedded controller firmware release.=0D +**/=0D +UINT16=0D +EFIAPI=0D +OemGetEmbeddedControllerFirmwareRelease (=0D + VOID=0D + );=0D +=0D #endif // OEM_MISC_LIB_H_=0D diff --git a/ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c b/ArmPkg/U= niversal/Smbios/OemMiscLibNull/OemMiscLib.c index 32f6d55c1a9a..788ccab9e8c1 100644 --- a/ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c +++ b/ArmPkg/Universal/Smbios/OemMiscLibNull/OemMiscLib.c @@ -254,3 +254,31 @@ OemGetSystemUuid ( ASSERT (FALSE);=0D CopyGuid (SystemUuid, &gZeroGuid);=0D }=0D +=0D +/** Fetches the BIOS release.=0D +=0D + @return The BIOS release.=0D +**/=0D +UINT16=0D +EFIAPI=0D +OemGetBiosRelease (=0D + VOID=0D + )=0D +{=0D + ASSERT (FALSE);=0D + return 0xFFFF;=0D +}=0D +=0D +/** Fetches the embedded controller firmware release.=0D +=0D + @return The embedded controller firmware release.=0D +**/=0D +UINT16=0D +EFIAPI=0D +OemGetEmbeddedControllerFirmwareRelease (=0D + VOID=0D + )=0D +{=0D + ASSERT (FALSE);=0D + return 0xFFFF;=0D +}=0D diff --git a/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFun= ction.c b/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFuncti= on.c index b49c4b754cab..e9106a8a2fec 100644 --- a/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c +++ b/ArmPkg/Universal/Smbios/SmbiosMiscDxe/Type00/MiscBiosVendorFunction.c @@ -1,5 +1,6 @@ /** @file=0D =0D + Copyright (c) 2022, Ampere Computing LLC. All rights reserved.
=0D Copyright (c) 2021, NUVIA Inc. All rights reserved.
=0D Copyright (c) 2009, Intel Corporation. All rights reserved.
=0D Copyright (c) 2015, Hisilicon Limited. All rights reserved.
=0D @@ -13,6 +14,7 @@ #include =0D #include =0D #include =0D +#include =0D #include =0D #include =0D =0D @@ -191,11 +193,11 @@ SMBIOS_MISC_TABLE_FUNCTION (MiscBiosVendor) { TokenToUpdate =3D STRING_TOKEN (STR_MISC_BIOS_VERSION);=0D HiiSetString (mSmbiosMiscHiiHandle, TokenToUpdate, Version, NULL);=0D } else {=0D - Version =3D (CHAR16 *)PcdGetPtr (PcdFirmwareVersionString);=0D - if (StrLen (Version) > 0) {=0D - TokenToUpdate =3D STRING_TOKEN (STR_MISC_BIOS_VERSION);=0D - HiiSetString (mSmbiosMiscHiiHandle, TokenToUpdate, Version, NULL);=0D - }=0D + OemUpdateSmbiosInfo (=0D + mSmbiosMiscHiiHandle,=0D + STRING_TOKEN (STR_MISC_BIOS_VERSION),=0D + BiosVersionType00=0D + );=0D }=0D =0D Char16String =3D GetBiosReleaseDate ();=0D @@ -251,13 +253,21 @@ SMBIOS_MISC_TABLE_FUNCTION (MiscBiosVendor) { }=0D }=0D =0D - SmbiosRecord->SystemBiosMajorRelease =3D (UINT8)(PcdGet16 (PcdSystemBios= Release) >> 8);=0D - SmbiosRecord->SystemBiosMinorRelease =3D (UINT8)(PcdGet16 (PcdSystemBios= Release) & 0xFF);=0D + if (PcdGet16 (PcdSystemBiosRelease) !=3D 0xFFFF) {=0D + SmbiosRecord->SystemBiosMajorRelease =3D (UINT8)(PcdGet16 (PcdSystemBi= osRelease) >> 8);=0D + SmbiosRecord->SystemBiosMinorRelease =3D (UINT8)(PcdGet16 (PcdSystemBi= osRelease) & 0xFF);=0D + } else {=0D + SmbiosRecord->SystemBiosMajorRelease =3D (UINT8)(OemGetBiosRelease () = >> 8);=0D + SmbiosRecord->SystemBiosMinorRelease =3D (UINT8)(OemGetBiosRelease () = & 0xFF);=0D + }=0D =0D - SmbiosRecord->EmbeddedControllerFirmwareMajorRelease =3D (UINT16)=0D - (PcdGet16 (PcdEmb= eddedControllerFirmwareRelease) >> 8);=0D - SmbiosRecord->EmbeddedControllerFirmwareMinorRelease =3D (UINT16)=0D - (PcdGet16 (PcdEmb= eddedControllerFirmwareRelease) & 0xFF);=0D + if (PcdGet16 (PcdEmbeddedControllerFirmwareRelease) !=3D 0xFFFF) {=0D + SmbiosRecord->EmbeddedControllerFirmwareMajorRelease =3D (UINT16)(PcdG= et16 (PcdEmbeddedControllerFirmwareRelease) >> 8);=0D + SmbiosRecord->EmbeddedControllerFirmwareMinorRelease =3D (UINT16)(PcdG= et16 (PcdEmbeddedControllerFirmwareRelease) & 0xFF);=0D + } else {=0D + SmbiosRecord->EmbeddedControllerFirmwareMajorRelease =3D (UINT16)(OemG= etEmbeddedControllerFirmwareRelease () >> 8);=0D + SmbiosRecord->EmbeddedControllerFirmwareMinorRelease =3D (UINT16)(OemG= etEmbeddedControllerFirmwareRelease () & 0xFF);=0D + }=0D =0D OptionalStrStart =3D (CHAR8 *)(SmbiosRecord + 1);=0D UnicodeStrToAsciiStrS (Vendor, OptionalStrStart, VendorStrLen + 1);=0D --=20 2.25.1