Reviewed-by: Maurice Ma <maurice...@intel.com>

> -----Original Message-----
> From: Marcello Sylvester Bauer <marcello.ba...@9elements.com>
> Sent: Wednesday, June 24, 2020 2:58
> To: devel@edk2.groups.io
> Cc: Patrick Rudolph <patrick.rudo...@9elements.com>; Christian Walter
> <christian.wal...@9elements.com>; Ma, Maurice <maurice...@intel.com>;
> Dong, Guo <guo.d...@intel.com>; You, Benjamin <benjamin....@intel.com>
> Subject: [PATCH v2 1/1] UefiPayloadPkg: Make TerminalDxe build time
> configurable
> 
> From: Patrick Rudolph <patrick.rudo...@9elements.com>
> 
> As the TerminalDxe significantly slows down the boot menu rendering, add the
> DISABLE_SERIAL_TERMINAL option to disable it at build time.
> 
> Signed-off-by: Patrick Rudolph <patrick.rudo...@9elements.com>
> Signed-off-by: Marcello Sylvester Bauer <marcello.ba...@9elements.com>
> Cc: Patrick Rudolph <patrick.rudo...@9elements.com>
> Cc: Christian Walter <christian.wal...@9elements.com>
> Cc: Maurice Ma <maurice...@intel.com>
> Cc: Guo Dong <guo.d...@intel.com>
> Cc: Benjamin You <benjamin....@intel.com>
> ---
>  UefiPayloadPkg/UefiPayloadPkgIa32.dsc    | 5 +++++
>  UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc | 5 +++++
>  UefiPayloadPkg/UefiPayloadPkg.fdf        | 2 ++
>  3 files changed, 12 insertions(+)
> 
> diff --git a/UefiPayloadPkg/UefiPayloadPkgIa32.dsc
> b/UefiPayloadPkg/UefiPayloadPkgIa32.dsc
> index c6c47833871b..49f42285882d 100644
> --- a/UefiPayloadPkg/UefiPayloadPkgIa32.dsc
> +++ b/UefiPayloadPkg/UefiPayloadPkgIa32.dsc
> @@ -58,6 +58,9 @@ [Defines]
>    DEFINE UART_DEFAULT_STOP_BITS       = 1   DEFINE
> DEFAULT_TERMINAL_TYPE        = 0 +  # Enabling the serial terminal will slow
> down the boot menu rendering!+  DEFINE DISABLE_SERIAL_TERMINAL      =
> FALSE+   #   #  typedef struct {   #    UINT16  VendorId;          ///< 
> Vendor ID to
> match the PCI device.  The value 0xFFFF terminates the list of entries.@@ -
> 502,7 +505,9 @@ [Components.IA32]
>    MdeModulePkg/Universal/Console/ConPlatformDxe/ConPlatformDxe.inf
> MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitterDxe.inf
> MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsoleDxe.in
> f+!if $(DISABLE_SERIAL_TERMINAL) == FALSE
> MdeModulePkg/Universal/Console/TerminalDxe/TerminalDxe.inf+!endif
> UefiPayloadPkg/GraphicsOutputDxe/GraphicsOutputDxe.inf    
> #--------------------
> ----------diff --git a/UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc
> b/UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc
> index 5559b1258521..33805902f705 100644
> --- a/UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc
> +++ b/UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc
> @@ -59,6 +59,9 @@ [Defines]
>    DEFINE UART_DEFAULT_STOP_BITS       = 1   DEFINE
> DEFAULT_TERMINAL_TYPE        = 0 +  # Enabling the serial terminal will slow
> down the boot menu redering!+  DEFINE DISABLE_SERIAL_TERMINAL      =
> FALSE+   #   #  typedef struct {   #    UINT16  VendorId;          ///< 
> Vendor ID to
> match the PCI device.  The value 0xFFFF terminates the list of entries.@@ -
> 504,7 +507,9 @@ [Components.X64]
>    MdeModulePkg/Universal/Console/ConPlatformDxe/ConPlatformDxe.inf
> MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitterDxe.inf
> MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsoleDxe.in
> f+!if $(DISABLE_SERIAL_TERMINAL) == FALSE
> MdeModulePkg/Universal/Console/TerminalDxe/TerminalDxe.inf+!endif
> UefiPayloadPkg/GraphicsOutputDxe/GraphicsOutputDxe.inf    
> #--------------------
> ----------diff --git a/UefiPayloadPkg/UefiPayloadPkg.fdf
> b/UefiPayloadPkg/UefiPayloadPkg.fdf
> index dfbcde566849..03ff6a9595d1 100644
> --- a/UefiPayloadPkg/UefiPayloadPkg.fdf
> +++ b/UefiPayloadPkg/UefiPayloadPkg.fdf
> @@ -131,7 +131,9 @@ [FV.DXEFV]
>  INF MdeModulePkg/Universal/Console/ConPlatformDxe/ConPlatformDxe.inf
> INF MdeModulePkg/Universal/Console/ConSplitterDxe/ConSplitterDxe.inf INF
> MdeModulePkg/Universal/Console/GraphicsConsoleDxe/GraphicsConsoleDxe.in
> f+!if $(DISABLE_SERIAL_TERMINAL) == FALSE INF
> MdeModulePkg/Universal/Console/TerminalDxe/TerminalDxe.inf+!endif INF
> UefiPayloadPkg/GraphicsOutputDxe/GraphicsOutputDxe.inf  #--
> 2.25.4


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#61679): https://edk2.groups.io/g/devel/message/61679
Mute This Topic: https://groups.io/mt/75080106/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to