Thanks for the explanation.
Reviewed-by: Ray Ni <ray...@intel.com>

> -----Original Message-----
> From: Xie, Yuanhao <yuanhao....@intel.com>
> Sent: Friday, April 1, 2022 5:56 PM
> To: Ni, Ray <ray...@intel.com>; devel@edk2.groups.io
> Cc: Dong, Guo <guo.d...@intel.com>; Ma, Maurice <maurice...@intel.com>; You, 
> Benjamin <benjamin....@intel.com>;
> Rhodes, Sean <sean@starlabs.systems>; Liu, Zhiguang <zhiguang....@intel.com>
> Subject: RE: [PATCH] UefiPayloadPkg: Change some configuration of the payload
> 
> Hi Ray,
> 
> The reason why we use PeiDxeDebugLibReportStatusCode instead of 
> BaseDebugLibSerialPort is that
> Base Debug library is based on Serial Port library, and it can only send 
> debug messages to serial port device, while SerialPort
> supports to print debug msg in other places.
> 
> Thanks
> Yuanhao
> 
> -----Original Message-----
> From: Ni, Ray <ray...@intel.com>
> Sent: Friday, April 1, 2022 10:05 AM
> To: Xie, Yuanhao <yuanhao....@intel.com>; devel@edk2.groups.io
> Cc: Dong, Guo <guo.d...@intel.com>; Ma, Maurice <maurice...@intel.com>; You, 
> Benjamin <benjamin....@intel.com>;
> Rhodes, Sean <sean@starlabs.systems>; Liu, Zhiguang <zhiguang....@intel.com>
> Subject: RE: [PATCH] UefiPayloadPkg: Change some configuration of the payload
> 
> >
> >  [LibraryClasses.common.DXE_CORE]
> > -  DebugLib|MdePkg/Library/BaseDebugLibSerialPort/BaseDebugLibSerialPort.inf
> > +  
> > DebugLib|MdeModulePkg/Library/PeiDxeDebugLibReportStatusCode/PeiDxeDebugLibReportStatusCode.inf
> 
> If DxeCore uses the ReportStatusCode instance, some debug messages may 
> disappear comparing to the case using SerialPort
> instance.
> Because the ReportStatusCode driver runs a bit later. Is it for saving the 
> payload binary size?



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


Reply via email to