* Re: [edk2-devel] [PATCH 43/43] UefiPayloadPkg: Add option to follow BGRT spec
2022-01-31 20:56 ` [PATCH 43/43] UefiPayloadPkg: Add option to follow BGRT spec Sean Rhodes
@ 2022-02-01 22:51 ` Guo Dong
0 siblings, 0 replies; 2+ messages in thread
From: Guo Dong @ 2022-02-01 22:51 UTC (permalink / raw)
To: devel@edk2.groups.io, Rhodes, Sean
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+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [guo.dong@intel.com] -=-=-=-=-=-=
^ permalink raw reply related [flat|nested] 2+ messages in thread