Re: [PATCH 2/2] UefiPayloadPkg: Hookup SD Card timeout


Ma, Maurice <maurice.ma@...>
 

This patch depends on other changes in MdeModulePkg to add a new SdMmcGenericTimeoutValue PCD.
The changes for UefiPayloadPkg look fine to me.

One minor comment is about the macro name. Is this new introduced timeout PCD applicable to both SD and MMC device?
If so, "SD_CARD_TIMEOUT" might not be accurate to cover the MMC device. Maybe use "SD_MMC_TIMEOUT" ?

Thanks,
Maurice

-----Original Message-----
From: Sean Rhodes <sean@...>
Sent: Thursday, February 17, 2022 23:24
To: devel@edk2.groups.io
Cc: Dong, Guo <guo.dong@...>; Rhodes, Sean <sean@...>;
Ni, Ray <ray.ni@...>; Ma, Maurice <maurice.ma@...>; You,
Benjamin <benjamin.you@...>
Subject: [PATCH 2/2] UefiPayloadPkg: Hookup SD Card timeout

Hook SD_CARD_TIMEOUT build option to SdMmcGenericTimeoutValue PCD.

Cc: Guo Dong <guo.dong@...>
Cc: Ray Ni <ray.ni@...>
Cc: Maurice Ma <maurice.ma@...>
Cc: Benjamin You <benjamin.you@...>
Signed-off-by: Sean Rhodes <sean@...>
---
UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++
1 file changed, 2 insertions(+)

diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc
b/UefiPayloadPkg/UefiPayloadPkg.dsc
index 1ce96a51c1..d75fe26426 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.dsc
+++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
@@ -33,6 +33,7 @@
DEFINE UNIVERSAL_PAYLOAD = FALSE

DEFINE SECURITY_STUB_ENABLE = TRUE

DEFINE SMM_SUPPORT = FALSE

+ DEFINE SD_CARD_TIMEOUT = 1000000

#

# SBL: UEFI payload for Slim Bootloader

# COREBOOT: UEFI payload for coreboot

@@ -398,6 +399,7 @@
!if $(PERFORMANCE_MEASUREMENT_ENABLE)

gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask | 0x1

!endif

+
gEfiMdeModulePkgTokenSpaceGuid.PcdSdMmcGenericTimeoutValue|$(SD_CAR
D_TIMEOUT)



[PcdsPatchableInModule.X64]


gPcAtChipsetPkgTokenSpaceGuid.PcdRtcIndexRegister|$(RTC_INDEX_REGISTER)

--
2.32.0

Join devel@edk2.groups.io to automatically receive all group messages.