From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4095 Updates several debug macros in KabylakeSiliconPkg to correctly match print specifiers

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4095 Updates several debug macros in CoffeelakeSiliconPkg to correctly match print

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4095 Fixes several debug macros in Silicon/Intel that have a mismatched number of print

Yes. Thanks Michael!

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4094 Updates several debug macros in WhitleyOpenBoardPkg to correctly match print

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4094 Removes an argument passed to a debug macro without a print specifier. The argument

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4094 Updates several debug macros in MinPlatformPkg to correctly match print specifiers to

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D4094 Fixes several debug macros in Platform/Intel that have a mismatched number of print

It looks like you only need my review on patch 2/2 which I provided. I see Nate gave R-b on 1/2 so you should be good to go.

Reviewed-by: Michael Kubacki <michael.kubacki@...>

Thanks for the quick review, especially during your holiday.

Hi Michael Kubacki, Please help to review below patch series which will resolve the duplicate library header in DEC

For the series: Reviewed-by: Hao A Wu <hao.a.wu@...> Best Regards, Hao Wu

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4092 Duplicate library class header issue will be resolved and CI exception for FspWrapperPlatformMultiPhaseLib should be removed too. Cc: Nate

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4092 To comply with coding style rule each library class should have its own header even if it is just a private child library instance consumed

To comply with coding style rule each library class should have its own header even if it is just a private child library instance consumed by parent public library in the same package. Cc: Nate

Hello Sami Thank you so much for your review, I apologize for the late response. My comment in line about the handle manager [GM]. Best Regards Girish [GM] I agree with the idea of having a map of

Reviewed-by: Nate DeSimone <nathaniel.l.desimone@...>

REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4092 To comply with coding style rule each library class should have its own header even if it is just a private child library instance consumed

Patch has been merged: https://github.com/tianocore/edk2-platforms/commit/ae75c51f27e21036b6ee021a2d5b9f365f951413 Thanks, Chasel