This patch has changes for MdeModulePkg which need review from MdeModulePkg maintainers. Pease separate this patch into different patches for different package.
And please CC the package maintainers when you send patches to edk2 devel list follow EDK2 process. Else the patch might be not reviewed in time. Thanks, Guo -----Original Message----- From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Sean Rhodes Sent: Monday, January 31, 2022 1:56 PM To: devel@edk2.groups.io Cc: Rhodes, Sean <sean@starlabs.systems> Subject: [edk2-devel] [PATCH 43/43] UefiPayloadPkg: Add option to follow BGRT spec Add option to centre the Boot Logo 38.2% from the top of screen, following the BGRT specification. Signed-off-by: Sean Rhodes <sean@starlabs.systems> --- MdeModulePkg/Library/BootLogoLib/BootLogoLib.c | 6 +++++- MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf | 3 +++ MdeModulePkg/MdeModulePkg.dec | 3 +++ UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++ 4 files changed, 13 insertions(+), 1 deletion(-) diff --git a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c index 478ec2d40e..f6d052eea2 100644 --- a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c +++ b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c @@ -176,7 +176,11 @@ BootLogoEnableLogo ( break; case EdkiiPlatformLogoDisplayAttributeCenter: DestX = (SizeOfX - Image.Width) / 2;- DestY = (SizeOfY - Image.Height) / 2;+ if (FixedPcdGetBool (PcdFollowBGRTSpec) == TRUE) {+ DestY = (SizeOfY * 382) / 1000 - Image.Height / 2;+ } else {+ DestY = (SizeOfY - Image.Height) / 2;+ } break; case EdkiiPlatformLogoDisplayAttributeCenterRight: DestX = SizeOfX - Image.Width;diff --git a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf index 7d50f2dfa3..03ff038f47 100644 --- a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf +++ b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf @@ -48,5 +48,8 @@ gEfiUserManagerProtocolGuid ## CONSUMES gEdkiiPlatformLogoProtocolGuid ## CONSUMES +[Pcd]+ gEfiMdeModulePkgTokenSpaceGuid.PcdFollowBGRTSpec+ [FeaturePcd] gEfiMdePkgTokenSpaceGuid.PcdUgaConsumeSupport ## CONSUMESdiff --git a/MdeModulePkg/MdeModulePkg.dec b/MdeModulePkg/MdeModulePkg.dec index 463e889e9a..fb3eb7ab7f 100644 --- a/MdeModulePkg/MdeModulePkg.dec +++ b/MdeModulePkg/MdeModulePkg.dec @@ -2079,6 +2079,9 @@ # @Prompt Enable PCIe Resizable BAR Capability support. gEfiMdeModulePkgTokenSpaceGuid.PcdPcieResizableBarSupport|FALSE|BOOLEAN|0x10000024 + # Follow BGRT Specifcation+ gEfiMdeModulePkgTokenSpaceGuid.PcdFollowBGRTSpec|TRUE|BOOLEAN|0x00000025+ [PcdsPatchableInModule] ## Specify memory size with page number for PEI code when # Loading Module at Fixed Address feature is enabled.diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc b/UefiPayloadPkg/UefiPayloadPkg.dsc index 1ce96a51c1..115111c037 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 FOLLOW_BGRT_SPEC = TRUE # # SBL: UEFI payload for Slim Bootloader # COREBOOT: UEFI payload for coreboot@@ -398,6 +399,7 @@ !if $(PERFORMANCE_MEASUREMENT_ENABLE) gEfiMdePkgTokenSpaceGuid.PcdPerformanceLibraryPropertyMask | 0x1 !endif+ gEfiMdeModulePkgTokenSpaceGuid.PcdFollowBGRTSpec|$(FOLLOW_BGRT_SPEC) [PcdsPatchableInModule.X64] gPcAtChipsetPkgTokenSpaceGuid.PcdRtcIndexRegister|$(RTC_INDEX_REGISTER)-- 2.32.0 -=-=-=-=-=-= Groups.io Links: You receive all messages sent to this group. View/Reply Online (#86263): https://edk2.groups.io/g/devel/message/86263 Mute This Topic: https://groups.io/mt/88818048/1781375 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [guo.d...@intel.com] -=-=-=-=-=-= -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#86320): https://edk2.groups.io/g/devel/message/86320 Mute This Topic: https://groups.io/mt/88818048/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-