On Fri, 18 Jan 2019 at 16:27, Yao, Jiewen <jiewen....@intel.com> wrote: > > Usually, we do not encourage to use SerialPortLib directly in a hardware > independent environment. > I do not suggest we bring an architecture dependency on the existence of > SerialPort in a common code. > > However, if ARCH64 has some specific code that must use SerialPortLib > explicitly, I am OK. > Can we move SerialPortLib under [LibraryClasses.AARCH64] ? >
I am happy to remove the SerialPortInitialize call, and instead rely on DebugLib to pull in SerialPortLIb if it wants to (and rely on the constructor to be invoked implicitly) > > > > > -----Original Message----- > > From: Ard Biesheuvel [mailto:ard.biesheu...@linaro.org] > > Sent: Wednesday, January 16, 2019 12:23 PM > > To: edk2-devel@lists.01.org > > Cc: Ard Biesheuvel <ard.biesheu...@linaro.org>; Achin Gupta > > <achin.gu...@arm.com>; Yao, Jiewen <jiewen....@intel.com>; Supreeth > > Venkatesh <supreeth.venkat...@arm.com>; Leif Lindholm > > <leif.lindh...@linaro.org>; Jagadeesh Ujja <jagadeesh.u...@arm.com>; > > Thomas Panakamattam Abraham <thomas.abra...@arm.com>; Sami > > Mujawar <sami.muja...@arm.com> > > Subject: [PATCH v2 05/11] StandaloneMmPkg/StandaloneMmCoreEntryPoint: > > add missing SerialPortLib ref > > > > StandaloneMmCoreEntryPoint calls SerialPortInitialize() explicitly, > > so add SerialPortLib to its list of LibraryClasses. > > > > 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/StandaloneMmCo > > reEntryPoint.inf | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git > > a/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/StandaloneMm > > CoreEntryPoint.inf > > b/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/StandaloneMm > > CoreEntryPoint.inf > > index 3222cd359f3e..769eaeeefbea 100644 > > --- > > a/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/StandaloneMm > > CoreEntryPoint.inf > > +++ > > b/StandaloneMmPkg/Library/StandaloneMmCoreEntryPoint/StandaloneMm > > CoreEntryPoint.inf > > @@ -43,6 +43,7 @@ [Packages.AARCH64] > > [LibraryClasses] > > BaseLib > > DebugLib > > + SerialPortLib > > > > [LibraryClasses.AARCH64] > > StandaloneMmMmuLib > > -- > > 2.17.1 > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel