The usage is not clear from the MACRO name DEBUG_REAL_INSTANCE_SUPPORT
Not sure if we could have a better name (e.g., MULTIPLE_DEBUG_PORT_SUPPORT?)

Thanks,
Guo
-----Original Message-----
From: Sravanthi, K KavyaX <k.kavyax.sravan...@intel.com> 
Sent: Monday, August 29, 2022 10:29 PM
To: devel@edk2.groups.io
Cc: Sravanthi, K KavyaX <k.kavyax.sravan...@intel.com>; Dong, Guo 
<guo.d...@intel.com>; Ni, Ray <ray...@intel.com>; Rhodes, Sean 
<sean@starlabs.systems>; Guo, Gua <gua....@intel.com>
Subject: [PATCH v5 2/2] UefiPayloadPkg: Allow DxeMain.inf to consume the new 
SerialPortLib

Let DxeMain.inf consume new SerialPortLib to support multiple channel debug 
message on early DXE if DEBUG_REAL_INSTANCE_SUPPORT is true.

Cc: Guo Dong <guo.d...@intel.com>
Cc: Ray Ni <ray...@intel.com>
Cc: Sean Rhodes <sean@starlabs.systems>
Cc: Gua Guo <gua....@intel.com>
Signed-off-by: Kavya <k.kavyax.sravan...@intel.com>
---
 UefiPayloadPkg/UefiPayloadPkg.dsc | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/UefiPayloadPkg/UefiPayloadPkg.dsc 
b/UefiPayloadPkg/UefiPayloadPkg.dsc
index 1b23ff3dbc..24aa66006f 100644
--- a/UefiPayloadPkg/UefiPayloadPkg.dsc
+++ b/UefiPayloadPkg/UefiPayloadPkg.dsc
@@ -132,6 +132,8 @@
   # Note: for emulation platform such as QEMU, this may not work and should 
set it as FALSE
   DEFINE CPU_TIMER_LIB_ENABLE  = TRUE
 
+  DEFINE DEBUG_REAL_INSTANCE_SUPPORT = FALSE
+
 [BuildOptions]
   *_*_*_CC_FLAGS                 = -D DISABLE_NEW_DEPRECATED_INTERFACES
 !if $(USE_CBMEM_FOR_CONSOLE) == FALSE
@@ -615,6 +617,10 @@
   #
   MdeModulePkg/Core/Dxe/DxeMain.inf {
     <LibraryClasses>
+      !if $(DEBUG_REAL_INSTANCE_SUPPORT) == TRUE
+        
DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
+        
SerialPortLib|UefiPayloadPkg/Library/BaseSerialPortLibHob/BaseSerialPortLibHob.inf
+      !endif
       
NULL|MdeModulePkg/Library/LzmaCustomDecompressLib/LzmaCustomDecompressLib.inf
   }
 
--
2.30.2.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#93002): https://edk2.groups.io/g/devel/message/93002
Mute This Topic: https://groups.io/mt/93343477/21656
Group Owner: devel+ow...@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to