Reviewed-by: jiewen....@intel.com > -----Original Message----- > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > Sent: Tuesday, March 5, 2019 5:33 AM > To: edk2-devel@lists.01.org > Cc: Ard Biesheuvel <ard.biesheu...@linaro.org>; Achin Gupta > <achin.gu...@arm.com>; Supreeth Venkatesh > <supreeth.venkat...@arm.com>; Yao, Jiewen <jiewen....@intel.com>; > Leif Lindholm <leif.lindh...@linaro.org>; Jagadeesh Ujja > <jagadeesh.u...@arm.com> > Subject: [PATCH 05/10] StandaloneMmPkg/StandaloneMmCoreEntryPoint: > drop explicit SerialPortLib call > > Sending DEBUG output to the serial port should only be done via > DebugLib calls, which is in charge of initializing the serial > port when appropriate. So drop the explicit SerialPortInitialize () > invocation, and rely on normal constructor ordering to get the > serial port into the appropriate state at the right time. > > Contributed-under: TianoCore Contribution Agreement 1.1 > Signed-off-by: Ard Biesheuvel <ard.biesheu...@linaro.org> > --- > > StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Standal > oneMmCoreEntryPoint.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git > a/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Stand > aloneMmCoreEntryPoint.c > b/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Stand > aloneMmCoreEntryPoint.c > index 5cca532456fd..c8e11a253d24 100644 > --- > a/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Stand > aloneMmCoreEntryPoint.c > +++ > b/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/AArch64/Stand > aloneMmCoreEntryPoint.c > @@ -232,9 +232,6 @@ _ModuleEntryPoint ( > VOID *TeData; > UINTN TeDataSize; > > - Status = SerialPortInitialize (); > - ASSERT_EFI_ERROR (Status); > - > // Get Secure Partition Manager Version Information > Status = GetSpmVersion (); > if (EFI_ERROR (Status)) { > -- > 2.20.1
_______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
