* [PATCH 0/2] Introduce and use new boot logo position
@ 2021-01-06 15:08 Patrick Rudolph
2021-01-06 15:08 ` [PATCH 1/2] UefiPayloadPkg: Enable boot logo Patrick Rudolph
2021-01-06 15:08 ` [PATCH 2/2] MdeModulePkg/BootLogoLib: Center logo 38.2% from top of screen Patrick Rudolph
0 siblings, 2 replies; 9+ messages in thread
From: Patrick Rudolph @ 2021-01-06 15:08 UTC (permalink / raw)
To: devel; +Cc: maurice.ma, guo.dong, benjamin.you
- Enable the boot logo for UefiPayloadPkg.
- Use Microsoft's recommended positioning for the boot logo as default
Tim Crawford (2):
UefiPayloadPkg: Enable boot logo
MdeModulePkg/BootLogoLib: Center logo 38.2% from top of screen
MdeModulePkg/Include/Protocol/PlatformLogo.h | 3 ++-
MdeModulePkg/Library/BootLogoLib/BootLogoLib.c | 4 ++++
MdeModulePkg/Logo/Logo.c | 2 +-
UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c | 3 +++
UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf | 1 +
UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++
UefiPayloadPkg/UefiPayloadPkg.fdf | 2 ++
7 files changed, 15 insertions(+), 2 deletions(-)
--
2.26.2
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/2] UefiPayloadPkg: Enable boot logo
2021-01-06 15:08 [PATCH 0/2] Introduce and use new boot logo position Patrick Rudolph
@ 2021-01-06 15:08 ` Patrick Rudolph
2021-01-06 16:27 ` [edk2-devel] " Ma, Maurice
2021-01-06 22:30 ` Guo Dong
2021-01-06 15:08 ` [PATCH 2/2] MdeModulePkg/BootLogoLib: Center logo 38.2% from top of screen Patrick Rudolph
1 sibling, 2 replies; 9+ messages in thread
From: Patrick Rudolph @ 2021-01-06 15:08 UTC (permalink / raw)
To: devel; +Cc: maurice.ma, guo.dong, benjamin.you
From: Tim Crawford <tcrawford@system76.com>
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
---
UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c | 3 +++
UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf | 1 +
UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++
UefiPayloadPkg/UefiPayloadPkg.fdf | 2 ++
4 files changed, 8 insertions(+)
diff --git a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
index c5c6af0abc..2df2d5a02d 100644
--- a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
+++ b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
@@ -216,6 +216,9 @@ PlatformBootManagerAfterConsole (
Black.Blue = Black.Green = Black.Red = Black.Reserved = 0;
White.Blue = White.Green = White.Red = White.Reserved = 0xFF;
+ gST->ConOut->ClearScreen (gST->ConOut);
+ BootLogoEnableLogo ();
+
EfiBootManagerConnectAll ();
EfiBootManagerRefreshAllBootOption ();
diff --git a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
index 1f5a0bcad0..beba973a70 100644
--- a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
+++ b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
@@ -39,6 +39,7 @@
UefiRuntimeServicesTableLib
UefiLib
UefiBootManagerLib
+ BootLogoLib
PcdLib
DxeServicesLib
MemoryAllocationLib
diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc b/UefiPayloadPkg/UefiPayloadPkg.dsc
index ae62a9c4d6..1a084c479e 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.dsc
+++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
@@ -169,6 +169,7 @@
CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
SecurityManagementLib|MdeModulePkg/Library/DxeSecurityManagementLib/DxeSecurityManagementLib.inf
UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
+ BootLogoLib|MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf
CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/CustomizedDisplayLib.inf
FrameBufferBltLib|MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.inf
@@ -389,6 +390,7 @@
MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf
UefiCpuPkg/CpuDxe/CpuDxe.inf
MdeModulePkg/Universal/BdsDxe/BdsDxe.inf
+ MdeModulePkg/Logo/LogoDxe.inf
MdeModulePkg/Application/UiApp/UiApp.inf {
<LibraryClasses>
NULL|MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf
diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf b/UefiPayloadPkg/UefiPayloadPkg.fdf
index a97ace7395..98d3fdea4a 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.fdf
+++ b/UefiPayloadPkg/UefiPayloadPkg.fdf
@@ -112,6 +112,8 @@ INF MdeModulePkg/Universal/DisplayEngineDxe/DisplayEngineDxe.inf
INF UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf
INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
+INF MdeModulePkg/Logo/LogoDxe.inf
+
#
# PCI Support
#
--
2.26.2
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 2/2] MdeModulePkg/BootLogoLib: Center logo 38.2% from top of screen
2021-01-06 15:08 [PATCH 0/2] Introduce and use new boot logo position Patrick Rudolph
2021-01-06 15:08 ` [PATCH 1/2] UefiPayloadPkg: Enable boot logo Patrick Rudolph
@ 2021-01-06 15:08 ` Patrick Rudolph
1 sibling, 0 replies; 9+ messages in thread
From: Patrick Rudolph @ 2021-01-06 15:08 UTC (permalink / raw)
To: devel; +Cc: maurice.ma, guo.dong, benjamin.you
From: Tim Crawford <tcrawford@system76.com>
Use Microsoft's recommended positioning [1] for the boot logo.
> We recommend that the logo is placed with its center at 38.2% from the
> screen's top edge. This positioning is based on the golden ratio's
> visual aesthetics and matches the Windows 10 design proportions.
[1]: https://docs.microsoft.com/en-us/windows-hardware/drivers/bringup/boot-screen-components#position-the-logo-during-post
Based on Tim Crawford <tcrawford@system76.com> initial commit.
Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
---
MdeModulePkg/Include/Protocol/PlatformLogo.h | 3 ++-
MdeModulePkg/Library/BootLogoLib/BootLogoLib.c | 4 ++++
MdeModulePkg/Logo/Logo.c | 2 +-
3 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/MdeModulePkg/Include/Protocol/PlatformLogo.h b/MdeModulePkg/Include/Protocol/PlatformLogo.h
index 55c9e08696..21a4c79e1d 100644
--- a/MdeModulePkg/Include/Protocol/PlatformLogo.h
+++ b/MdeModulePkg/Include/Protocol/PlatformLogo.h
@@ -29,7 +29,8 @@ typedef enum {
EdkiiPlatformLogoDisplayAttributeCenterBottom,
EdkiiPlatformLogoDisplayAttributeLeftBottom,
EdkiiPlatformLogoDisplayAttributeCenterLeft,
- EdkiiPlatformLogoDisplayAttributeCenter
+ EdkiiPlatformLogoDisplayAttributeCenter,
+ EdkiiPlatformLogoDisplayAttributeMicrosoftRecommended
} EDKII_PLATFORM_LOGO_DISPLAY_ATTRIBUTE;
/**
diff --git a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
index 134660f28d..d40c65b59f 100644
--- a/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
+++ b/MdeModulePkg/Library/BootLogoLib/BootLogoLib.c
@@ -173,6 +173,10 @@ BootLogoEnableLogo (
DestX = 0;
DestY = (SizeOfY - Image.Height) / 2;
break;
+ case EdkiiPlatformLogoDisplayAttributeMicrosoftRecommended:
+ DestX = (SizeOfX - Image.Width) / 2;
+ DestY = (SizeOfY * 382) / 1000 - Image.Height / 2;
+ break;
case EdkiiPlatformLogoDisplayAttributeCenter:
DestX = (SizeOfX - Image.Width) / 2;
DestY = (SizeOfY - Image.Height) / 2;
diff --git a/MdeModulePkg/Logo/Logo.c b/MdeModulePkg/Logo/Logo.c
index c647253ecd..131a1b456a 100644
--- a/MdeModulePkg/Logo/Logo.c
+++ b/MdeModulePkg/Logo/Logo.c
@@ -26,7 +26,7 @@ EFI_HII_HANDLE mHiiHandle;
LOGO_ENTRY mLogos[] = {
{
IMAGE_TOKEN (IMG_LOGO),
- EdkiiPlatformLogoDisplayAttributeCenter,
+ EdkiiPlatformLogoDisplayAttributeMicrosoftRecommended,
0,
0
}
--
2.26.2
^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [edk2-devel] [PATCH 1/2] UefiPayloadPkg: Enable boot logo
2021-01-06 15:08 ` [PATCH 1/2] UefiPayloadPkg: Enable boot logo Patrick Rudolph
@ 2021-01-06 16:27 ` Ma, Maurice
2021-01-06 22:30 ` Guo Dong
1 sibling, 0 replies; 9+ messages in thread
From: Ma, Maurice @ 2021-01-06 16:27 UTC (permalink / raw)
To: patrick.rudolph@9elements.com
Cc: Dong, Guo, You, Benjamin, devel@edk2.groups.io
Reviewed-by: Maurice Ma <maurice.ma@intel.com>
Regards
Maurice
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Patrick
> Rudolph
> Sent: Wednesday, January 6, 2021 7:08
> To: devel@edk2.groups.io
> Cc: Ma, Maurice <maurice.ma@intel.com>; Dong, Guo <guo.dong@intel.com>;
> You, Benjamin <benjamin.you@intel.com>
> Subject: [edk2-devel] [PATCH 1/2] UefiPayloadPkg: Enable boot logo
>
> From: Tim Crawford <tcrawford@system76.com>
>
> Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
> ---
> UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c |
> 3 +++
> UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
> | 1 +
> UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++
> UefiPayloadPkg/UefiPayloadPkg.fdf | 2 ++
> 4 files changed, 8 insertions(+)
>
> diff --git
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> index c5c6af0abc..2df2d5a02d 100644
> --- a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> +++
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> @@ -216,6 +216,9 @@ PlatformBootManagerAfterConsole (
> Black.Blue = Black.Green = Black.Red = Black.Reserved = 0;
>
> White.Blue = White.Green = White.Red = White.Reserved = 0xFF;
>
>
>
> + gST->ConOut->ClearScreen (gST->ConOut);
>
> + BootLogoEnableLogo ();
>
> +
>
> EfiBootManagerConnectAll ();
>
> EfiBootManagerRefreshAllBootOption ();
>
>
>
> diff --git
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf
> index 1f5a0bcad0..beba973a70 100644
> ---
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf
> +++
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf
> @@ -39,6 +39,7 @@
> UefiRuntimeServicesTableLib
>
> UefiLib
>
> UefiBootManagerLib
>
> + BootLogoLib
>
> PcdLib
>
> DxeServicesLib
>
> MemoryAllocationLib
>
> diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc
> b/UefiPayloadPkg/UefiPayloadPkg.dsc
> index ae62a9c4d6..1a084c479e 100644
> --- a/UefiPayloadPkg/UefiPayloadPkg.dsc
> +++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
> @@ -169,6 +169,7 @@
> CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
>
>
> SecurityManagementLib|MdeModulePkg/Library/DxeSecurityManagementLib/
> DxeSecurityManagementLib.inf
>
>
> UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBootM
> anagerLib.inf
>
> + BootLogoLib|MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf
>
>
> CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/Customiz
> edDisplayLib.inf
>
>
> FrameBufferBltLib|MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltL
> ib.inf
>
>
>
> @@ -389,6 +390,7 @@
> MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf
>
> UefiCpuPkg/CpuDxe/CpuDxe.inf
>
> MdeModulePkg/Universal/BdsDxe/BdsDxe.inf
>
> + MdeModulePkg/Logo/LogoDxe.inf
>
> MdeModulePkg/Application/UiApp/UiApp.inf {
>
> <LibraryClasses>
>
>
> NULL|MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf
>
> diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf
> b/UefiPayloadPkg/UefiPayloadPkg.fdf
> index a97ace7395..98d3fdea4a 100644
> --- a/UefiPayloadPkg/UefiPayloadPkg.fdf
> +++ b/UefiPayloadPkg/UefiPayloadPkg.fdf
> @@ -112,6 +112,8 @@ INF
> MdeModulePkg/Universal/DisplayEngineDxe/DisplayEngineDxe.inf
> INF UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf
>
>
>
> INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
>
> +INF MdeModulePkg/Logo/LogoDxe.inf
>
> +
>
> #
>
> # PCI Support
>
> #
>
> --
> 2.26.2
>
>
>
> -=-=-=-=-=-=
> Groups.io Links: You receive all messages sent to this group.
> View/Reply Online (#69834): https://edk2.groups.io/g/devel/message/69834
> Mute This Topic: https://groups.io/mt/79476244/1773972
> Group Owner: devel+owner@edk2.groups.io
> Unsubscribe: https://edk2.groups.io/g/devel/unsub [maurice.ma@intel.com]
> -=-=-=-=-=-=
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] UefiPayloadPkg: Enable boot logo
2021-01-06 15:08 ` [PATCH 1/2] UefiPayloadPkg: Enable boot logo Patrick Rudolph
2021-01-06 16:27 ` [edk2-devel] " Ma, Maurice
@ 2021-01-06 22:30 ` Guo Dong
2021-01-07 7:20 ` Patrick Rudolph
1 sibling, 1 reply; 9+ messages in thread
From: Guo Dong @ 2021-01-06 22:30 UTC (permalink / raw)
To: Patrick Rudolph, devel@edk2.groups.io; +Cc: Ma, Maurice, You, Benjamin
Hi Patrick,
I didn't look at the impl details. If the logo is shown from the bootloader, how it would be impacted in payload phase?
I saw you have clearscreen in this patch. Does it only move the logo to center of the screen?
Thanks,
Guo
> -----Original Message-----
> From: Patrick Rudolph <patrick.rudolph@9elements.com>
> Sent: Wednesday, January 6, 2021 8:08 AM
> To: devel@edk2.groups.io
> Cc: Ma, Maurice <maurice.ma@intel.com>; Dong, Guo <guo.dong@intel.com>;
> You, Benjamin <benjamin.you@intel.com>
> Subject: [PATCH 1/2] UefiPayloadPkg: Enable boot logo
>
> From: Tim Crawford <tcrawford@system76.com>
>
> Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
> ---
> UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> | 3 +++
>
> UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
> | 1 +
> UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++
> UefiPayloadPkg/UefiPayloadPkg.fdf | 2 ++
> 4 files changed, 8 insertions(+)
>
> diff --git
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> index c5c6af0abc..2df2d5a02d 100644
> ---
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> +++
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> @@ -216,6 +216,9 @@ PlatformBootManagerAfterConsole (
> Black.Blue = Black.Green = Black.Red = Black.Reserved = 0;
>
> White.Blue = White.Green = White.Red = White.Reserved = 0xFF;
>
>
>
> + gST->ConOut->ClearScreen (gST->ConOut);
>
> + BootLogoEnableLogo ();
>
> +
>
> EfiBootManagerConnectAll ();
>
> EfiBootManagerRefreshAllBootOption ();
>
>
>
> diff --git
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf
> index 1f5a0bcad0..beba973a70 100644
> ---
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf
> +++
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> nf
> @@ -39,6 +39,7 @@
> UefiRuntimeServicesTableLib
>
> UefiLib
>
> UefiBootManagerLib
>
> + BootLogoLib
>
> PcdLib
>
> DxeServicesLib
>
> MemoryAllocationLib
>
> diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc
> b/UefiPayloadPkg/UefiPayloadPkg.dsc
> index ae62a9c4d6..1a084c479e 100644
> --- a/UefiPayloadPkg/UefiPayloadPkg.dsc
> +++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
> @@ -169,6 +169,7 @@
> CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
>
>
> SecurityManagementLib|MdeModulePkg/Library/DxeSecurityManagementLib/
> DxeSecurityManagementLib.inf
>
>
> UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBootM
> anagerLib.inf
>
> + BootLogoLib|MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf
>
>
> CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/Customiz
> edDisplayLib.inf
>
>
> FrameBufferBltLib|MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBlt
> Lib.inf
>
>
>
> @@ -389,6 +390,7 @@
> MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf
>
> UefiCpuPkg/CpuDxe/CpuDxe.inf
>
> MdeModulePkg/Universal/BdsDxe/BdsDxe.inf
>
> + MdeModulePkg/Logo/LogoDxe.inf
>
> MdeModulePkg/Application/UiApp/UiApp.inf {
>
> <LibraryClasses>
>
>
> NULL|MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf
>
> diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf
> b/UefiPayloadPkg/UefiPayloadPkg.fdf
> index a97ace7395..98d3fdea4a 100644
> --- a/UefiPayloadPkg/UefiPayloadPkg.fdf
> +++ b/UefiPayloadPkg/UefiPayloadPkg.fdf
> @@ -112,6 +112,8 @@ INF
> MdeModulePkg/Universal/DisplayEngineDxe/DisplayEngineDxe.inf
> INF UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf
>
>
>
> INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
>
> +INF MdeModulePkg/Logo/LogoDxe.inf
>
> +
>
> #
>
> # PCI Support
>
> #
>
> --
> 2.26.2
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH 1/2] UefiPayloadPkg: Enable boot logo
2021-01-06 22:30 ` Guo Dong
@ 2021-01-07 7:20 ` Patrick Rudolph
2021-01-08 2:05 ` [edk2-devel] " Guo Dong
0 siblings, 1 reply; 9+ messages in thread
From: Patrick Rudolph @ 2021-01-07 7:20 UTC (permalink / raw)
To: Dong, Guo; +Cc: devel@edk2.groups.io, Ma, Maurice, You, Benjamin
Hi Guo,
Yes, the clear screen will remove the boot logo, if any, displayed by
the bootloader.
The EDK2 user that builds UefiPayloadPkg needs to add the bootloader
logo as part of the build process to keep this boot logo displayed.
It might be possible to keep the bootloader's boot logo by adding an
ACPI BGRT parser in the BlSupportDxe.
Regards,
Patrick Rudolph
On Wed, Jan 6, 2021 at 11:30 PM Dong, Guo <guo.dong@intel.com> wrote:
>
>
> Hi Patrick,
>
> I didn't look at the impl details. If the logo is shown from the bootloader, how it would be impacted in payload phase?
> I saw you have clearscreen in this patch. Does it only move the logo to center of the screen?
>
> Thanks,
> Guo
>
> > -----Original Message-----
> > From: Patrick Rudolph <patrick.rudolph@9elements.com>
> > Sent: Wednesday, January 6, 2021 8:08 AM
> > To: devel@edk2.groups.io
> > Cc: Ma, Maurice <maurice.ma@intel.com>; Dong, Guo <guo.dong@intel.com>;
> > You, Benjamin <benjamin.you@intel.com>
> > Subject: [PATCH 1/2] UefiPayloadPkg: Enable boot logo
> >
> > From: Tim Crawford <tcrawford@system76.com>
> >
> > Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
> > ---
> > UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > | 3 +++
> >
> > UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
> > | 1 +
> > UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++
> > UefiPayloadPkg/UefiPayloadPkg.fdf | 2 ++
> > 4 files changed, 8 insertions(+)
> >
> > diff --git
> > a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > index c5c6af0abc..2df2d5a02d 100644
> > ---
> > a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > +++
> > b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > @@ -216,6 +216,9 @@ PlatformBootManagerAfterConsole (
> > Black.Blue = Black.Green = Black.Red = Black.Reserved = 0;
> >
> > White.Blue = White.Green = White.Red = White.Reserved = 0xFF;
> >
> >
> >
> > + gST->ConOut->ClearScreen (gST->ConOut);
> >
> > + BootLogoEnableLogo ();
> >
> > +
> >
> > EfiBootManagerConnectAll ();
> >
> > EfiBootManagerRefreshAllBootOption ();
> >
> >
> >
> > diff --git
> > a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> > nf
> > b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> > nf
> > index 1f5a0bcad0..beba973a70 100644
> > ---
> > a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> > nf
> > +++
> > b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> > nf
> > @@ -39,6 +39,7 @@
> > UefiRuntimeServicesTableLib
> >
> > UefiLib
> >
> > UefiBootManagerLib
> >
> > + BootLogoLib
> >
> > PcdLib
> >
> > DxeServicesLib
> >
> > MemoryAllocationLib
> >
> > diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc
> > b/UefiPayloadPkg/UefiPayloadPkg.dsc
> > index ae62a9c4d6..1a084c479e 100644
> > --- a/UefiPayloadPkg/UefiPayloadPkg.dsc
> > +++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
> > @@ -169,6 +169,7 @@
> > CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
> >
> >
> > SecurityManagementLib|MdeModulePkg/Library/DxeSecurityManagementLib/
> > DxeSecurityManagementLib.inf
> >
> >
> > UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBootM
> > anagerLib.inf
> >
> > + BootLogoLib|MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf
> >
> >
> > CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/Customiz
> > edDisplayLib.inf
> >
> >
> > FrameBufferBltLib|MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBlt
> > Lib.inf
> >
> >
> >
> > @@ -389,6 +390,7 @@
> > MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf
> >
> > UefiCpuPkg/CpuDxe/CpuDxe.inf
> >
> > MdeModulePkg/Universal/BdsDxe/BdsDxe.inf
> >
> > + MdeModulePkg/Logo/LogoDxe.inf
> >
> > MdeModulePkg/Application/UiApp/UiApp.inf {
> >
> > <LibraryClasses>
> >
> >
> > NULL|MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf
> >
> > diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf
> > b/UefiPayloadPkg/UefiPayloadPkg.fdf
> > index a97ace7395..98d3fdea4a 100644
> > --- a/UefiPayloadPkg/UefiPayloadPkg.fdf
> > +++ b/UefiPayloadPkg/UefiPayloadPkg.fdf
> > @@ -112,6 +112,8 @@ INF
> > MdeModulePkg/Universal/DisplayEngineDxe/DisplayEngineDxe.inf
> > INF UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf
> >
> >
> >
> > INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
> >
> > +INF MdeModulePkg/Logo/LogoDxe.inf
> >
> > +
> >
> > #
> >
> > # PCI Support
> >
> > #
> >
> > --
> > 2.26.2
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [edk2-devel] [PATCH 1/2] UefiPayloadPkg: Enable boot logo
2021-01-07 7:20 ` Patrick Rudolph
@ 2021-01-08 2:05 ` Guo Dong
0 siblings, 0 replies; 9+ messages in thread
From: Guo Dong @ 2021-01-08 2:05 UTC (permalink / raw)
To: devel@edk2.groups.io, patrick.rudolph@9elements.com
Cc: Ma, Maurice, You, Benjamin
Thanks Patrick Rudolph for this info.
Reviewed-by: Guo Dong <guo.dong@intel.com>
> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Patrick
> Rudolph
> Sent: Thursday, January 7, 2021 12:20 AM
> To: Dong, Guo <guo.dong@intel.com>
> Cc: devel@edk2.groups.io; Ma, Maurice <maurice.ma@intel.com>; You,
> Benjamin <benjamin.you@intel.com>
> Subject: Re: [edk2-devel] [PATCH 1/2] UefiPayloadPkg: Enable boot logo
>
> Hi Guo,
> Yes, the clear screen will remove the boot logo, if any, displayed by
> the bootloader.
> The EDK2 user that builds UefiPayloadPkg needs to add the bootloader
> logo as part of the build process to keep this boot logo displayed.
>
> It might be possible to keep the bootloader's boot logo by adding an
> ACPI BGRT parser in the BlSupportDxe.
>
> Regards,
> Patrick Rudolph
>
> On Wed, Jan 6, 2021 at 11:30 PM Dong, Guo <guo.dong@intel.com> wrote:
> >
> >
> > Hi Patrick,
> >
> > I didn't look at the impl details. If the logo is shown from the bootloader, how
> it would be impacted in payload phase?
> > I saw you have clearscreen in this patch. Does it only move the logo to center
> of the screen?
> >
> > Thanks,
> > Guo
> >
> > > -----Original Message-----
> > > From: Patrick Rudolph <patrick.rudolph@9elements.com>
> > > Sent: Wednesday, January 6, 2021 8:08 AM
> > > To: devel@edk2.groups.io
> > > Cc: Ma, Maurice <maurice.ma@intel.com>; Dong, Guo
> <guo.dong@intel.com>;
> > > You, Benjamin <benjamin.you@intel.com>
> > > Subject: [PATCH 1/2] UefiPayloadPkg: Enable boot logo
> > >
> > > From: Tim Crawford <tcrawford@system76.com>
> > >
> > > Signed-off-by: Patrick Rudolph <patrick.rudolph@9elements.com>
> > > ---
> > > UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > > | 3 +++
> > >
> > >
> UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
> > > | 1 +
> > > UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++
> > > UefiPayloadPkg/UefiPayloadPkg.fdf | 2 ++
> > > 4 files changed, 8 insertions(+)
> > >
> > > diff --git
> > >
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > >
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > > index c5c6af0abc..2df2d5a02d 100644
> > > ---
> > >
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > > +++
> > >
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
> > > @@ -216,6 +216,9 @@ PlatformBootManagerAfterConsole (
> > > Black.Blue = Black.Green = Black.Red = Black.Reserved = 0;
> > >
> > > White.Blue = White.Green = White.Red = White.Reserved = 0xFF;
> > >
> > >
> > >
> > > + gST->ConOut->ClearScreen (gST->ConOut);
> > >
> > > + BootLogoEnableLogo ();
> > >
> > > +
> > >
> > > EfiBootManagerConnectAll ();
> > >
> > > EfiBootManagerRefreshAllBootOption ();
> > >
> > >
> > >
> > > diff --git
> > >
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> > > nf
> > >
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> > > nf
> > > index 1f5a0bcad0..beba973a70 100644
> > > ---
> > >
> a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> > > nf
> > > +++
> > >
> b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.i
> > > nf
> > > @@ -39,6 +39,7 @@
> > > UefiRuntimeServicesTableLib
> > >
> > > UefiLib
> > >
> > > UefiBootManagerLib
> > >
> > > + BootLogoLib
> > >
> > > PcdLib
> > >
> > > DxeServicesLib
> > >
> > > MemoryAllocationLib
> > >
> > > diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc
> > > b/UefiPayloadPkg/UefiPayloadPkg.dsc
> > > index ae62a9c4d6..1a084c479e 100644
> > > --- a/UefiPayloadPkg/UefiPayloadPkg.dsc
> > > +++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
> > > @@ -169,6 +169,7 @@
> > >
> CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
> > >
> > >
> > >
> SecurityManagementLib|MdeModulePkg/Library/DxeSecurityManagementLib/
> > > DxeSecurityManagementLib.inf
> > >
> > >
> > >
> UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBootM
> > > anagerLib.inf
> > >
> > > + BootLogoLib|MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf
> > >
> > >
> > >
> CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/Customiz
> > > edDisplayLib.inf
> > >
> > >
> > >
> FrameBufferBltLib|MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBlt
> > > Lib.inf
> > >
> > >
> > >
> > > @@ -389,6 +390,7 @@
> > > MdeModulePkg/Universal/SecurityStubDxe/SecurityStubDxe.inf
> > >
> > > UefiCpuPkg/CpuDxe/CpuDxe.inf
> > >
> > > MdeModulePkg/Universal/BdsDxe/BdsDxe.inf
> > >
> > > + MdeModulePkg/Logo/LogoDxe.inf
> > >
> > > MdeModulePkg/Application/UiApp/UiApp.inf {
> > >
> > > <LibraryClasses>
> > >
> > >
> > >
> NULL|MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf
> > >
> > > diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf
> > > b/UefiPayloadPkg/UefiPayloadPkg.fdf
> > > index a97ace7395..98d3fdea4a 100644
> > > --- a/UefiPayloadPkg/UefiPayloadPkg.fdf
> > > +++ b/UefiPayloadPkg/UefiPayloadPkg.fdf
> > > @@ -112,6 +112,8 @@ INF
> > > MdeModulePkg/Universal/DisplayEngineDxe/DisplayEngineDxe.inf
> > > INF UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf
> > >
> > >
> > >
> > > INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
> > >
> > > +INF MdeModulePkg/Logo/LogoDxe.inf
> > >
> > > +
> > >
> > > #
> > >
> > > # PCI Support
> > >
> > > #
> > >
> > > --
> > > 2.26.2
> >
>
>
>
>
^ permalink raw reply [flat|nested] 9+ messages in thread
* [PATCH 1/2] UefiPayloadPkg: Enable Boot Logo
2022-07-10 19:13 [PATCH 0/2] Boot Logo Support Sean Rhodes
@ 2022-07-10 19:13 ` Sean Rhodes
0 siblings, 0 replies; 9+ messages in thread
From: Sean Rhodes @ 2022-07-10 19:13 UTC (permalink / raw)
To: devel; +Cc: Sean Rhodes, Guo Dong, Ray Ni, Maurice Ma, Benjamin You
Cc: Guo Dong <guo.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Cc: Maurice Ma <maurice.ma@intel.com>
Cc: Benjamin You <benjamin.you@intel.com>
Signed-off-by: Sean Rhodes <sean@starlabs.systems>
---
.../Library/PlatformBootManagerLib/PlatformBootManager.c | 3 +++
.../Library/PlatformBootManagerLib/PlatformBootManagerLib.inf | 1 +
UefiPayloadPkg/UefiPayloadPkg.dsc | 2 ++
UefiPayloadPkg/UefiPayloadPkg.fdf | 2 ++
4 files changed, 8 insertions(+)
diff --git a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
index 0eb577313a..93f7776fbc 100644
--- a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
+++ b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
@@ -252,6 +252,9 @@ PlatformBootManagerAfterConsole (
Black.Blue = Black.Green = Black.Red = Black.Reserved = 0;
White.Blue = White.Green = White.Red = White.Reserved = 0xFF;
+ gST->ConOut->ClearScreen (gST->ConOut);
+ BootLogoEnableLogo ();
+
EfiBootManagerConnectAll ();
EfiBootManagerRefreshAllBootOption ();
diff --git a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
index 9f58c460cd..7ec93420f2 100644
--- a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
+++ b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
@@ -39,6 +39,7 @@
UefiRuntimeServicesTableLib
UefiLib
UefiBootManagerLib
+ BootLogoLib
PcdLib
DxeServicesLib
MemoryAllocationLib
diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc b/UefiPayloadPkg/UefiPayloadPkg.dsc
index cfcf38578d..7d390ac8eb 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.dsc
+++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
@@ -212,6 +212,7 @@
CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
SecurityManagementLib|MdeModulePkg/Library/DxeSecurityManagementLib/DxeSecurityManagementLib.inf
UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
+ BootLogoLib|MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf
CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/CustomizedDisplayLib.inf
FrameBufferBltLib|MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.inf
@@ -561,6 +562,7 @@
!endif
UefiCpuPkg/CpuDxe/CpuDxe.inf
MdeModulePkg/Universal/BdsDxe/BdsDxe.inf
+ MdeModulePkg/Logo/LogoDxe.inf
MdeModulePkg/Application/UiApp/UiApp.inf {
<LibraryClasses>
NULL|MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf
diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf b/UefiPayloadPkg/UefiPayloadPkg.fdf
index c7b04978ad..2beef7ba10 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.fdf
+++ b/UefiPayloadPkg/UefiPayloadPkg.fdf
@@ -158,6 +158,8 @@ INF MdeModulePkg/Universal/EbcDxe/EbcDxe.inf
INF UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf
INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
+INF MdeModulePkg/Logo/LogoDxe.inf
+
#
# PCI Support
#
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [PATCH 1/2] UefiPayloadPkg: Enable Boot Logo
@ 2022-07-20 9:09 Sean Rhodes
0 siblings, 0 replies; 9+ messages in thread
From: Sean Rhodes @ 2022-07-20 9:09 UTC (permalink / raw)
To: devel; +Cc: Sean Rhodes, Guo Dong, Ray Ni
Cc: Guo Dong <guo.dong@intel.com>
Cc: Ray Ni <ray.ni@intel.com>
Signed-off-by: Sean Rhodes <sean@starlabs.systems>
---
.../Library/PlatformBootManagerLib/PlatformBootManager.c | 5 +++++
.../PlatformBootManagerLib/PlatformBootManagerLib.inf | 1 +
UefiPayloadPkg/UefiPayloadPkg.dsc | 6 ++++++
UefiPayloadPkg/UefiPayloadPkg.fdf | 4 ++++
4 files changed, 16 insertions(+)
diff --git a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
index b360e29dfe..2fadd3f130 100644
--- a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
+++ b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManager.c
@@ -252,6 +252,11 @@ PlatformBootManagerAfterConsole (
Black.Blue = Black.Green = Black.Red = Black.Reserved = 0;
White.Blue = White.Green = White.Red = White.Reserved = 0xFF;
+ if (&gEdkiiPlatformLogoProtocolGuid) {
+ gST->ConOut->ClearScreen (gST->ConOut);
+ BootLogoEnableLogo ();
+ }
+
EfiBootManagerConnectAll ();
EfiBootManagerRefreshAllBootOption ();
diff --git a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
index 9f58c460cd..7ec93420f2 100644
--- a/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
+++ b/UefiPayloadPkg/Library/PlatformBootManagerLib/PlatformBootManagerLib.inf
@@ -39,6 +39,7 @@
UefiRuntimeServicesTableLib
UefiLib
UefiBootManagerLib
+ BootLogoLib
PcdLib
DxeServicesLib
MemoryAllocationLib
diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc b/UefiPayloadPkg/UefiPayloadPkg.dsc
index 5e947526b7..bdafe8ae99 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.dsc
+++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
@@ -40,6 +40,7 @@
DEFINE SD_ENABLE = TRUE
DEFINE PS2_MOUSE_ENABLE = TRUE
DEFINE SD_MMC_TIMEOUT = 1000000
+ DEFINE BOOTSPLASH_IMAGE = FALSE
#
# NULL: NullMemoryTestDxe
@@ -221,6 +222,7 @@
CapsuleLib|MdeModulePkg/Library/DxeCapsuleLibNull/DxeCapsuleLibNull.inf
SecurityManagementLib|MdeModulePkg/Library/DxeSecurityManagementLib/DxeSecurityManagementLib.inf
UefiBootManagerLib|MdeModulePkg/Library/UefiBootManagerLib/UefiBootManagerLib.inf
+ BootLogoLib|MdeModulePkg/Library/BootLogoLib/BootLogoLib.inf
CustomizedDisplayLib|MdeModulePkg/Library/CustomizedDisplayLib/CustomizedDisplayLib.inf
FrameBufferBltLib|MdeModulePkg/Library/FrameBufferBltLib/FrameBufferBltLib.inf
@@ -419,6 +421,7 @@
gUefiPayloadPkgTokenSpaceGuid.PcdDispatchModuleAbove4GMemory|$(ABOVE_4G_MEMORY)
gUefiPayloadPkgTokenSpaceGuid.PcdBootManagerEscape|$(BOOT_MANAGER_ESCAPE)
+
gEfiMdePkgTokenSpaceGuid.PcdMaximumUnicodeStringLength|1800000
gEfiCryptoPkgTokenSpaceGuid.PcdCryptoServiceFamilyEnable.HmacSha256.Family | PCD_CRYPTO_SERVICE_ENABLE_FAMILY
@@ -593,6 +596,9 @@
!endif
UefiCpuPkg/CpuDxe/CpuDxe.inf
MdeModulePkg/Universal/BdsDxe/BdsDxe.inf
+!if $(BOOTSPLASH_IMAGE)
+ MdeModulePkg/Logo/LogoDxe.inf
+!endif
MdeModulePkg/Application/UiApp/UiApp.inf {
<LibraryClasses>
NULL|MdeModulePkg/Library/DeviceManagerUiLib/DeviceManagerUiLib.inf
diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf b/UefiPayloadPkg/UefiPayloadPkg.fdf
index 5c055e61b3..6d54bb0e13 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.fdf
+++ b/UefiPayloadPkg/UefiPayloadPkg.fdf
@@ -163,6 +163,10 @@ INF MdeModulePkg/Universal/EbcDxe/EbcDxe.inf
INF UefiPayloadPkg/BlSupportDxe/BlSupportDxe.inf
INF MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
+!if $(BOOTSPLASH_IMAGE)
+INF MdeModulePkg/Logo/LogoDxe.inf
+!endif
+
#
# PCI Support
#
--
2.34.1
^ permalink raw reply related [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-07-20 9:09 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-01-06 15:08 [PATCH 0/2] Introduce and use new boot logo position Patrick Rudolph
2021-01-06 15:08 ` [PATCH 1/2] UefiPayloadPkg: Enable boot logo Patrick Rudolph
2021-01-06 16:27 ` [edk2-devel] " Ma, Maurice
2021-01-06 22:30 ` Guo Dong
2021-01-07 7:20 ` Patrick Rudolph
2021-01-08 2:05 ` [edk2-devel] " Guo Dong
2021-01-06 15:08 ` [PATCH 2/2] MdeModulePkg/BootLogoLib: Center logo 38.2% from top of screen Patrick Rudolph
-- strict thread matches above, loose matches on Subject: below --
2022-07-10 19:13 [PATCH 0/2] Boot Logo Support Sean Rhodes
2022-07-10 19:13 ` [PATCH 1/2] UefiPayloadPkg: Enable Boot Logo Sean Rhodes
2022-07-20 9:09 Sean Rhodes
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox