[edk2-devel] [PATCH v4 1/1] OvmfPkg: Fix Hii form name mismatch with EFI variable

[edk2-devel] [PATCH v4 1/1] OvmfPkg: Fix Hii form name mismatch with EFI variable

[Wang, Yin]

The Hii form is named "MainFormState" while the EFI variable is named "PlatformConfig".
This discrepancy in names causes the following SCT cases to fail on RiscVVirtQemu:
ExtractConfigConformance
ExtractConfigFunction
ExtractConfig_Func
Previous Commit 16acacf addressed two of these issues, and this patch fixes the remaining one.

Signed-off-by: Yin Wang <yin.wang@intel.com>

v4: Only update commit message about version history

- aefcc918: Enhance ExtractConfig() to match UEFI specification, adding check of whether "<ConfigHdr> matches the GUID and name"
- 16acacf2: Fix Hii form name mismatch in ExtractConfig()
- This patch: Add one more fix for Hii form name mismatch

v3: Gerd comments on v2 that caused issues may related to RISC-V specific config. To address this separately, the patch was restored to v1, and details were added to the commit message.

v2: Add two more changes which caused other two SCT Hii cases to fail.

v1: Initial version
---
 OvmfPkg/PlatformDxe/Platform.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/OvmfPkg/PlatformDxe/Platform.c b/OvmfPkg/PlatformDxe/Platform.c
index cd3bfd554c..c32439683d 100644
--- a/OvmfPkg/PlatformDxe/Platform.c
+++ b/OvmfPkg/PlatformDxe/Platform.c
@@ -279,7 +279,7 @@ ExtractConfig (
     //
     ConfigRequestHdr = HiiConstructConfigHdr (
                          &gOvmfPlatformConfigGuid,
-                         mVariableName,
+                         mHiiFormName,
                          mImageHandle
                          );
     if (ConfigRequestHdr == NULL) {
-- 
2.34.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#108223): https://edk2.groups.io/g/devel/message/108223
Mute This Topic: https://groups.io/mt/101088203/7686176
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io]
-=-=-=-=-=-=-=-=-=-=-=-