Sorry to forget change the commit message. I originally want to use PeiDxeDebugLibReportStatusCode to replace the UefiDebugLibConOut. There may be some typos in other patches in these series. And there is also a comment conflicted with Doxygen rule . I would check and fix them.
Thanks, Zhichao > -----Original Message----- > From: Wu, Hao A > Sent: Thursday, April 11, 2019 4:45 PM > To: [email protected]; Gao, Zhichao <[email protected]> > Cc: Wang, Jian J <[email protected]>; Ni, Ray <[email protected]>; Zeng, > Star <[email protected]>; Gao, Liming <[email protected]>; Sean > Brogan <[email protected]>; Michael Turner > <[email protected]>; Bret Barkelew > <[email protected]> > Subject: RE: [edk2-devel] [PATCH 1/5] MdeModulePkg: Change the SMM > debug lib instance > > > -----Original Message----- > > From: [email protected] [mailto:[email protected]] On Behalf Of > > Gao, Zhichao > > Sent: Wednesday, April 10, 2019 3:03 PM > > To: [email protected] > > Cc: Wang, Jian J; Ni, Ray; Zeng, Star; Gao, Liming; Sean Brogan; > > Michael Turner; Bret Barkelew > > Subject: [edk2-devel] [PATCH 1/5] MdeModulePkg: Change the SMM > debug > > lib instance > > > > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1416 > > > > The UefiDebugLibConOut will not support DXE_SMM_DRIVER, change > > UefiDebugLibConOut to PeiDxeDebugLibReportStatusCode. > > Is this a typo? > Should be 'BaseDebugLibNull' according to your change. > > With this addressed, > Reviewed-by: Hao Wu <[email protected]> > > > Best Regards, > Hao Wu > > > > > Cc: Jian J Wang <[email protected]> > > Cc: Ray Ni <[email protected]> > > Cc: Star Zeng <[email protected]> > > Cc: Liming Gao <[email protected]> > > Cc: Sean Brogan <[email protected]> > > Cc: Michael Turner <[email protected]> > > Cc: Bret Barkelew <[email protected]> > > Signed-off-by: Zhichao Gao <[email protected]> > > --- > > MdeModulePkg/MdeModulePkg.dsc | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/MdeModulePkg/MdeModulePkg.dsc > > b/MdeModulePkg/MdeModulePkg.dsc index 7ef39af847..0da28e74c2 > 100644 > > --- a/MdeModulePkg/MdeModulePkg.dsc > > +++ b/MdeModulePkg/MdeModulePkg.dsc > > @@ -144,7 +144,7 @@ > > > > [LibraryClasses.common.DXE_SMM_DRIVER] > > HobLib|MdePkg/Library/DxeHobLib/DxeHobLib.inf > > - > DebugLib|MdePkg/Library/UefiDebugLibConOut/UefiDebugLibConOut.inf > > + DebugLib|MdePkg/Library/BaseDebugLibNull/BaseDebugLibNull.inf > > > > > MemoryAllocationLib|MdePkg/Library/SmmMemoryAllocationLib/SmmMe > > moryAllocationLib.inf > > > > > MmServicesTableLib|MdePkg/Library/MmServicesTableLib/MmServicesTabl > > eLib.inf > > > > > SmmServicesTableLib|MdePkg/Library/SmmServicesTableLib/SmmServicesT > > ableLib.inf > > -- > > 2.21.0.windows.1 > > > > > > -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#38864): https://edk2.groups.io/g/devel/message/38864 Mute This Topic: https://groups.io/mt/31016457/21656 Group Owner: [email protected] Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]] -=-=-=-=-=-=-=-=-=-=-=-
