On 04/07/16 13:37, Laszlo Ersek wrote: > On 04/07/16 12:19, Star Zeng wrote: >> Currently, the LockBox protocol is installed in entrypoint of >> OVMF AcpiS3SaveDxe. >> >> We can let the first driver run with LockBoxDxeLib linked to have its >> library constructor to install LockBox protocol on the ImageHandle. >> As other drivers may have gEfiLockBoxProtocolGuid dependency, >> the first driver should run before them. >> >> The later patches to retire AcpiS3SaveDxe for OVMF depends on this patch. >> >> Cc: Laszlo Ersek <ler...@redhat.com> >> Cc: Jordan Justen <jordan.l.jus...@intel.com> >> Cc: Jiewen Yao <jiewen....@intel.com> >> Contributed-under: TianoCore Contribution Agreement 1.0 >> Signed-off-by: Star Zeng <star.z...@intel.com> >> Tested-by: Laszlo Ersek <ler...@redhat.com> >> --- >> OvmfPkg/AcpiS3SaveDxe/AcpiS3Save.c | 13 +----------- >> OvmfPkg/AcpiS3SaveDxe/AcpiS3SaveDxe.inf | 6 +----- >> OvmfPkg/Library/LockBoxLib/LockBoxDxe.c | 31 >> ++++++++++++++++++++++++++-- >> OvmfPkg/Library/LockBoxLib/LockBoxDxeLib.inf | 6 +++++- >> 4 files changed, 36 insertions(+), 20 deletions(-) > > Reviewed-by: Laszlo Ersek <ler...@redhat.com>
I just realized that the series is now fully reviewed; I pushed it for you: 31583f9a8b3e..8ccd1d5b5c8c. Thanks Laszlo _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel