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 <[email protected]> > Sent: Thursday, February 17, 2022 23:24 > To: [email protected] > Cc: Dong, Guo <[email protected]>; Rhodes, Sean <[email protected]>; > Ni, Ray <[email protected]>; Ma, Maurice <[email protected]>; You, > Benjamin <[email protected]> > Subject: [PATCH 2/2] UefiPayloadPkg: Hookup SD Card timeout > > Hook SD_CARD_TIMEOUT build option to SdMmcGenericTimeoutValue PCD. > > Cc: Guo Dong <[email protected]> > Cc: Ray Ni <[email protected]> > Cc: Maurice Ma <[email protected]> > Cc: Benjamin You <[email protected]> > Signed-off-by: Sean Rhodes <[email protected]> > --- > 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 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#86758): https://edk2.groups.io/g/devel/message/86758 Mute This Topic: https://groups.io/mt/89228248/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
