Re: [edk2-devel] [edk2-staging][PATCH V1 0/5] Resolve dependency from MmUnblockMemoryLib

["Thomas Abraham" <thomas.abraham@arm.com>]

Hi Kun,

> -----Original Message-----
> From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Thomas
> Abraham via groups.io
> Sent: Thursday, March 11, 2021 11:39 AM
> To: devel@edk2.groups.io; kuqin12@gmail.com
> Cc: Chasel Chiu <chasel.chiu@intel.com>; Nate DeSimone
> <nathaniel.l.desimone@intel.com>; Liming Gao <gaoliming@byosoft.com.cn>;
> Eric Dong <eric.dong@intel.com>; Michael D Kinney
> <michael.d.kinney@intel.com>; Kelly Steele <kelly.steele@intel.com>; Zailiang
> Sun <zailiang.sun@intel.com>; Yi Qian <yi.qian@intel.com>; Ard Biesheuvel
> <ardb+tianocore@kernel.org>; Leif Lindholm <leif@nuviainc.com>; nd
> <nd@arm.com>
> Subject: Re: [edk2-devel] [edk2-staging][PATCH V1 0/5] Resolve dependency
> from MmUnblockMemoryLib
> 
> On 3/11/21 1:37 AM, Kun Qin via groups.io wrote:
> > The merging of commit range c5740f360636..59a3ccb09e7a on edk2 master
> has
> > separated Tcg2Smm driver into 2 functional componnents, Tcg2Acpi and Tcg2
> > MM driver. They also introduced dependency of MmUnblockMemoryLib on
> > VariableSmmRuntimeDxe as well as the new Tcg2Acpi driver.
> >
> > This patch series is sent to resolve the dependency mentioned above.
> >
> > Patch v1 branch: https://github.com/kuqin12/edk2-
> > platforms/tree/unblock_dependency_v1
> >
> > Cc: Chasel Chiu <chasel.chiu@intel.com>
> > Cc: Nate DeSimone <nathaniel.l.desimone@intel.com>
> > Cc: Liming Gao <gaoliming@byosoft.com.cn>
> > Cc: Eric Dong <eric.dong@intel.com>
> > Cc: Michael D Kinney <michael.d.kinney@intel.com>
> > Cc: Kelly Steele <kelly.steele@intel.com>
> > Cc: Zailiang Sun <zailiang.sun@intel.com>
> > Cc: Yi Qian <yi.qian@intel.com>
> > Cc: Ard Biesheuvel <ardb+tianocore@kernel.org>
> > Cc: Leif Lindholm <leif@nuviainc.com>
> >
> > Kun Qin (5):
> >   MinPlatformPkg: CoreCommonLib: Added new library for
> >     VariableSmmRuntimeDxe
> >   MinPlatformPkg: Core Include Files: Added Tcg2Acpi driver after
> >     separation
> >   QuarkPlatformPkg: Quark DSC File: Added new library for
> >     VariableSmmRuntimeDxe
> >   Vlv2TbltDevicePkg: PlatformPkg DSC: Added library for
> >     VariableSmmRuntimeDxe
> >   Socionext: DeveloperBox DSC File: Added library for
> >     VariableSmmRuntimeDxe
> >
> >  Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc           | 1 +
> >  Platform/Intel/MinPlatformPkg/Include/Dsc/CoreDxeInclude.dsc          | 1 +
> >  Platform/Intel/MinPlatformPkg/Include/Fdf/CoreSecurityLateInclude.fdf | 3
> ++-
> >  Platform/Intel/QuarkPlatformPkg/Quark.dsc                             | 1 +
> >  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc                  | 1 +
> >  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc                   | 1 +
> >  Platform/Socionext/DeveloperBox/DeveloperBox.dsc                      | 2 ++
> >  7 files changed, 9 insertions(+), 1 deletion(-)
> 
> Can this series be extended to resolve the dependency in Platform/Arm/SgiPkg
> as well? The change being applied in this patch series was tested with SgiPkg it
> does resolve the dependency.

This patch series need not be extended for Platform/Arm/SgiPkg. Sorry for the noise.

Thanks,
Thomas.