REF: https://bugzilla.tianocore.org/show_bug.cgi?id=3224
Fix build errors caused by "-Werror=unused-but-set-variable" with GCC5. These build errors occur only with DEBUG target because RELEASE_GCC5_X64_CC_FLAGS has "-Wno-unused-but-set-variable", but DEBUG_GCC5_X64_CC_FLAGS doesn't. Cc: Sai Chaganty <rangasai.v.chaga...@intel.com> Cc: Nate DeSimone <nathaniel.l.desim...@intel.com> Cc: Heng Luo <heng....@intel.com> Signed-off-by: Takuto Naito <nait...@gmail.com> --- .../Library/BasePlatformHookLib/BasePlatformHookLib.c | 8 -------- .../Library/BoardInitLib/PeiTigerlakeURvpInitPreMemLib.c | 2 ++ 2 files changed, 2 insertions(+), 8 deletions(-) diff --git a/Platform/Intel/TigerlakeOpenBoardPkg/Library/BasePlatformHookLib/BasePlatformHookLib.c b/Platform/Intel/TigerlakeOpenBoardPkg/Library/BasePlatformHookLib/BasePlatformHookLib.c index cc5337698b..0baee8795d 100644 --- a/Platform/Intel/TigerlakeOpenBoardPkg/Library/BasePlatformHookLib/BasePlatformHookLib.c +++ b/Platform/Intel/TigerlakeOpenBoardPkg/Library/BasePlatformHookLib/BasePlatformHookLib.c @@ -182,14 +182,6 @@ PlatformHookSerialPortInitialize ( VOID ) { - UINT16 IndexPort; - UINT16 DataPort; - UINT8 Index; - - IndexPort = 0; - DataPort = 0; - Index = 0; - // // Enable I/O decoding for COM1(3F8h-3FFh), COM2(2F8h-2FFh), I/O port 2Eh/2Fh, 4Eh/4Fh, 60h/64Fh and 62h/66h. // diff --git a/Platform/Intel/TigerlakeOpenBoardPkg/TigerlakeURvp/Library/BoardInitLib/PeiTigerlakeURvpInitPreMemLib.c b/Platform/Intel/TigerlakeOpenBoardPkg/TigerlakeURvp/Library/BoardInitLib/PeiTigerlakeURvpInitPreMemLib.c index 2ad229c1cd..0bba41e6d4 100644 --- a/Platform/Intel/TigerlakeOpenBoardPkg/TigerlakeURvp/Library/BoardInitLib/PeiTigerlakeURvpInitPreMemLib.c +++ b/Platform/Intel/TigerlakeOpenBoardPkg/TigerlakeURvp/Library/BoardInitLib/PeiTigerlakeURvpInitPreMemLib.c @@ -393,6 +393,7 @@ TigerlakeURvpInitPreMem ( /// Performing PlatformInitPreMemCallBack after PeiReadOnlyVariable2 PPI produced /// Status = PeiServicesNotifyPpi (&mPreMemNotifyList); + ASSERT_EFI_ERROR (Status); /// /// After code reorangized, memorycallback will run because the PPI is already @@ -400,6 +401,7 @@ TigerlakeURvpInitPreMem ( /// done before. /// Status = PeiServicesNotifyPpi (&mMemDiscoveredNotifyList); + ASSERT_EFI_ERROR (Status); DEBUG ((DEBUG_INFO, "TigerlakeURvpInitPreMem End!\n")); -- 2.30.1 -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#72132): https://edk2.groups.io/g/devel/message/72132 Mute This Topic: https://groups.io/mt/80867667/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-