On Fri, 2019-01-04 at 12:03 +0100, Ard Biesheuvel wrote: > ASSERT_EFI_ERROR (x) is a shorthand for ASSERT(!EFI_ERROR(x)), and so > it should only be used with EFI_STATUS type expressions. > > So drop two instances that operate on other types, since neither > looks > particularly useful. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> Reviewed-by: Supreeth Venkatesh <supreeth.venkat...@arm.com>
> --- > StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Standalon > eMmCoreEntryPoint.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git > a/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Standalo > neMmCoreEntryPoint.c > b/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Standalo > neMmCoreEntryPoint.c > index 05ed6c8dd0b5..5cca532456fd 100644 > --- > a/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Standalo > neMmCoreEntryPoint.c > +++ > b/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Standalo > neMmCoreEntryPoint.c > @@ -295,7 +295,6 @@ _ModuleEntryPoint ( > // > ProcessModuleEntryPointList (HobStart); > > - ASSERT_EFI_ERROR (CpuDriverEntryPoint); > DEBUG ((DEBUG_INFO, "Shared Cpu Driver EP 0x%lx\n", (UINT64) > CpuDriverEntryPoint)); > > finish: > @@ -303,5 +302,4 @@ finish: > InitMmFoundationSvcArgs.Arg0 = > ARM_SVC_ID_SP_EVENT_COMPLETE_AARCH64; > InitMmFoundationSvcArgs.Arg1 = Status; > DelegatedEventLoop (&InitMmFoundationSvcArgs); > - ASSERT_EFI_ERROR (0); > } _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel