Sunny:
  I am OK to merge the reviewed patched first. 

Thanks
Liming
> -----邮件原件-----
> 发件人: Sunny Wang <[email protected]>
> 发送时间: 2021年8月3日 22:11
> 收件人: Grzegorz Bernacki <[email protected]>; [email protected];
> [email protected]
> 抄送: [email protected]; Samer El-Haj-Mahmoud
> <[email protected]>; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; Sami Mujawar
> <[email protected]>; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> Thomas Abraham <[email protected]>; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; [email protected];
> Sunny Wang <[email protected]>
> 主题: RE: [edk2-platforms PATCH v6 0/4] Secure Boot default keys
> 
> Hi Ard and Maintainers,
> 
> For this patchset,
>  1/4 - Intel Platforms: add SecureBootVariableLib class resolution
>  2/4 - ARM Silicon and Platforms: add SecureBootVariableLib class
resolution
>  3/4 - RISC-V Platforms: add SecureBootVariableLib class resolution
>  4/4 - Platform/RaspberryPi: Enable default Secure Boot variables
> initialization
> 
> Only Intel platform patch (1/4) hasn't got all Review-bys. I offline sent
a
> reminder to Intel platform Maintainers. I think they may be busy with
other
> things or need more time to review it.
> 
> Therefore, how about we merge another three patches (2/4, 3/4, and 4/4)
> first?
> 
> Best Regards,
> Sunny Wang
> 
> -----Original Message-----
> From: Grzegorz Bernacki <[email protected]>
> Sent: Wednesday, July 14, 2021 8:31 PM
> To: [email protected]
> Cc: [email protected]; [email protected]; Samer El-Haj-Mahmoud
> <[email protected]>; Sunny Wang <[email protected]>;
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; Sami
> Mujawar <[email protected]>; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> Thomas Abraham <[email protected]>; [email protected];
> [email protected]; [email protected];
> [email protected]; [email protected]; [email protected];
> [email protected]; [email protected]; [email protected]; [email protected];
> Grzegorz Bernacki <[email protected]>
> Subject: [edk2-platforms PATCH v6 0/4] Secure Boot default keys
> 
> This patchset is a consequence of "Secure Boot default keys"
> patchset in edk2. It adds SecureBootVariableLib class resolution
> for each platform which uses SecureBootConfigDxe and also
> enables Secure Boot variables initialization for RPi4.
> Previously these commits were part of edk2 patchset, but since
> number of commits increased in v5 version, it is now separate
> patchset.
> 
> Changes related to both edk2 & edk-platform versions:
> Changes since v1:
> - change names:
>   SecBootVariableLib => SecureBootVariableLib
>   SecBootDefaultKeysDxe => SecureBootDefaultKeysDxe
>   SecEnrollDefaultKeysApp => EnrollFromDefaultKeysApp
> - change name of function CheckSetupMode to GetSetupMode
> - remove ShellPkg dependecy from EnrollFromDefaultKeysApp
> - rebase to master
> 
> Changes since v2:
> - fix coding style for functions headers in SecureBootVariableLib.h
> - add header to SecureBootDefaultKeys.fdf.inc
> - remove empty line spaces in SecureBootDefaultKeysDxe files
> - revert FAIL macro in EnrollFromDefaultKeysApp
> - remove functions duplicates and  add SecureBootVariableLib
>   to platforms which used it
> 
> Changes since v3:
> - move SecureBootDefaultKeys.fdf.inc to ArmPlatformPkg
> - leave duplicate of CreateTimeBasedPayload in PlatformVarCleanupLib
> - fix typo in guid description
> 
> Changes since v4:
> - reorder patches to make it bisectable
> - split commits related to more than one platform
> - move edk2-platform commits to separate patchset
> 
> Changes since v5:
> - split SecureBootVariableLib into SecureBootVariableLib and
>   SecureBootVariableProvisionLib
> 
> Grzegorz Bernacki (4):
>   Intel Platforms: add SecureBootVariableLib class resolution
>   ARM Silicon and Platforms: add SecureBootVariableLib class resolution
>   RISC-V Platforms: add SecureBootVariableLib class resolution
>   Platform/RaspberryPi: Enable default Secure Boot variables
>     initialization
> 
>  Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc
> | 2 ++
>  Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc
> | 2 ++
>  Platform/Intel/QuarkPlatformPkg/Quark.dsc
> | 2 ++
>  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc
> | 2 ++
>  Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc
> | 2 ++
>  Platform/Qemu/SbsaQemu/SbsaQemu.dsc
> | 2 ++
>  Platform/RaspberryPi/RPi3/RPi3.dsc
> | 2 ++
>  Platform/RaspberryPi/RPi4/RPi4.dsc
> | 4 ++++
>  Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc
> | 2 ++
> 
> Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc
> | 2 ++
>  Platform/Socionext/DeveloperBox/DeveloperBox.dsc
> | 5 +++++
>  Platform/RaspberryPi/RPi4/RPi4.fdf
> | 2 ++
>  12 files changed, 29 insertions(+)
> 
> --
> 2.25.1
> 
> IMPORTANT NOTICE: The contents of this email and any attachments are
> confidential and may also be privileged. If you are not the intended
recipient,
> please notify the sender immediately and do not disclose the contents to
any
> other person, use it for any purpose, or store or copy the information in
any
> medium. Thank you.




-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#78623): https://edk2.groups.io/g/devel/message/78623
Mute This Topic: https://groups.io/mt/84654266/21656
Group Owner: [email protected]
Unsubscribe: https://edk2.groups.io/g/devel/unsub [[email protected]]
-=-=-=-=-=-=-=-=-=-=-=-


Reply via email to