[edk2-devel] [edk2-platforms][PATCH v3 0/6] Resolving SecureBootVariableLib dependency

Kun Qin posted 6 patches 1 week, 2 days ago
Failed in applying to current master (apply log)
Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                         | 1 +
Platform/RaspberryPi/RPi3/RPi3.dsc                                   | 1 +
Platform/RaspberryPi/RPi4/RPi4.dsc                                   | 1 +
Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc           | 1 +
Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 1 +
Platform/Socionext/DeveloperBox/DeveloperBox.dsc                     | 1 +
Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc                 | 1 +
Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc            | 1 +
8 files changed, 8 insertions(+)
[edk2-devel] [edk2-platforms][PATCH v3 0/6] Resolving SecureBootVariableLib dependency
Posted by Kun Qin 1 week, 2 days ago
This v2 series is a follow up of previously submitted patches:
https://edk2.groups.io/g/devel/message/91669

The main update in v3 patches are:
- Updated a commit message to include Review-by tag
- Dumped a commit for Qemu platform as it was fixed by others

Patch v3 branch:https://github.com/kuqin12/edk2-platforms/tree/fix_sb_dep_v3

Cc: Ard Biesheuvel<ardb+tianocore@kernel.org>
Cc: Leif Lindholm<quic_llindhol@quicinc.com>
Cc: Graeme Gregory<graeme@nuviainc.com>
Cc: Radoslaw Biernacki<rad@semihalf.com>
Cc: Masami Hiramatsu<masami.hiramatsu@linaro.org>
Cc: Nhi Pham<nhi@os.amperecomputing.com>
Cc: Vu Nguyen<vunguyen@os.amperecomputing.com>
Cc: Thang Nguyen<thang@os.amperecomputing.com>
Cc: Chuong Tran<chuong@os.amperecomputing.com>
Cc: Thomas Abraham<thomas.abraham@arm.com>
Cc: Sami Mujawar<sami.mujawar@arm.com>
Cc: Abner Chang<abner.chang@hpe.com>
Cc: Gilbert Chen<gilbert.chen@hpe.com>
Cc: Daniel Schaefer<daniel.schaefer@hpe.com>
Cc: Jeremy Linton<jeremy.linton@arm.com>
Cc: Peng Xie<xiepeng@phytium.com.cn>
Cc: Ling Jia<jialing@phytium.com.cn>
Cc: Yiqi Shu<shuyiqi@phytium.com.cn>

Kun Qin (6):
  RaspberryPi: Pipeline: Resolving newly introduced dependency
  U5SeriesPkg: Pipeline: Resolving newly introduced dependency
  VExpressPkg: Pipeline: Resolving newly introduced dependency
  Socionext: Pipeline: Resolving newly introduced dependency
  AmpereAltraPkg: Pipeline: Resolving newly introduced dependency
  PhytiumCommonPkg: Pipeline: Resolving newly introduced dependency

 Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                         | 1 +
 Platform/RaspberryPi/RPi3/RPi3.dsc                                   | 1 +
 Platform/RaspberryPi/RPi4/RPi4.dsc                                   | 1 +
 Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc           | 1 +
 Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 1 +
 Platform/Socionext/DeveloperBox/DeveloperBox.dsc                     | 1 +
 Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc                 | 1 +
 Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc            | 1 +
 8 files changed, 8 insertions(+)

-- 
2.37.1.windows.1



-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92063): https://edk2.groups.io/g/devel/message/92063
Mute This Topic: https://groups.io/mt/92782210/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platforms][PATCH v3 0/6] Resolving SecureBootVariableLib dependency
Posted by Ard Biesheuvel 1 week ago
On Wed, 3 Aug 2022 at 00:59, Kun Qin <kuqin12@gmail.com> wrote:
>
> This v2 series is a follow up of previously submitted patches:
> https://edk2.groups.io/g/devel/message/91669
>
> The main update in v3 patches are:
> - Updated a commit message to include Review-by tag
> - Dumped a commit for Qemu platform as it was fixed by others
>
> Patch v3 branch:https://github.com/kuqin12/edk2-platforms/tree/fix_sb_dep_v3

The patches look fine to me, but what does 'Pipeline' mean?

>
> Cc: Ard Biesheuvel<ardb+tianocore@kernel.org>
> Cc: Leif Lindholm<quic_llindhol@quicinc.com>
> Cc: Graeme Gregory<graeme@nuviainc.com>
> Cc: Radoslaw Biernacki<rad@semihalf.com>
> Cc: Masami Hiramatsu<masami.hiramatsu@linaro.org>
> Cc: Nhi Pham<nhi@os.amperecomputing.com>
> Cc: Vu Nguyen<vunguyen@os.amperecomputing.com>
> Cc: Thang Nguyen<thang@os.amperecomputing.com>
> Cc: Chuong Tran<chuong@os.amperecomputing.com>
> Cc: Thomas Abraham<thomas.abraham@arm.com>
> Cc: Sami Mujawar<sami.mujawar@arm.com>
> Cc: Abner Chang<abner.chang@hpe.com>
> Cc: Gilbert Chen<gilbert.chen@hpe.com>
> Cc: Daniel Schaefer<daniel.schaefer@hpe.com>
> Cc: Jeremy Linton<jeremy.linton@arm.com>
> Cc: Peng Xie<xiepeng@phytium.com.cn>
> Cc: Ling Jia<jialing@phytium.com.cn>
> Cc: Yiqi Shu<shuyiqi@phytium.com.cn>
>
> Kun Qin (6):
>   RaspberryPi: Pipeline: Resolving newly introduced dependency
>   U5SeriesPkg: Pipeline: Resolving newly introduced dependency
>   VExpressPkg: Pipeline: Resolving newly introduced dependency
>   Socionext: Pipeline: Resolving newly introduced dependency
>   AmpereAltraPkg: Pipeline: Resolving newly introduced dependency
>   PhytiumCommonPkg: Pipeline: Resolving newly introduced dependency
>
>  Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                         | 1 +
>  Platform/RaspberryPi/RPi3/RPi3.dsc                                   | 1 +
>  Platform/RaspberryPi/RPi4/RPi4.dsc                                   | 1 +
>  Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc           | 1 +
>  Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 1 +
>  Platform/Socionext/DeveloperBox/DeveloperBox.dsc                     | 1 +
>  Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc                 | 1 +
>  Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc            | 1 +
>  8 files changed, 8 insertions(+)
>
> --
> 2.37.1.windows.1
>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92110): https://edk2.groups.io/g/devel/message/92110
Mute This Topic: https://groups.io/mt/92782210/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-
Re: [edk2-devel] [edk2-platforms][PATCH v3 0/6] Resolving SecureBootVariableLib dependency
Posted by Kun Qin 6 days, 23 hours ago
Hi Ard,

The term "Pipeline" is not applicable in the scope of edk2-platforms. 
Thanks for catching it.

A new version was sent for this fix: 
https://edk2.groups.io/g/devel/message/92128. Please let
me know if there is any other feedback.

Regards,
Kun

On 8/4/2022 3:26 AM, Ard Biesheuvel wrote:
> On Wed, 3 Aug 2022 at 00:59, Kun Qin <kuqin12@gmail.com> wrote:
>> This v2 series is a follow up of previously submitted patches:
>> https://edk2.groups.io/g/devel/message/91669
>>
>> The main update in v3 patches are:
>> - Updated a commit message to include Review-by tag
>> - Dumped a commit for Qemu platform as it was fixed by others
>>
>> Patch v3 branch:https://github.com/kuqin12/edk2-platforms/tree/fix_sb_dep_v3
> The patches look fine to me, but what does 'Pipeline' mean?
>
>> Cc: Ard Biesheuvel<ardb+tianocore@kernel.org>
>> Cc: Leif Lindholm<quic_llindhol@quicinc.com>
>> Cc: Graeme Gregory<graeme@nuviainc.com>
>> Cc: Radoslaw Biernacki<rad@semihalf.com>
>> Cc: Masami Hiramatsu<masami.hiramatsu@linaro.org>
>> Cc: Nhi Pham<nhi@os.amperecomputing.com>
>> Cc: Vu Nguyen<vunguyen@os.amperecomputing.com>
>> Cc: Thang Nguyen<thang@os.amperecomputing.com>
>> Cc: Chuong Tran<chuong@os.amperecomputing.com>
>> Cc: Thomas Abraham<thomas.abraham@arm.com>
>> Cc: Sami Mujawar<sami.mujawar@arm.com>
>> Cc: Abner Chang<abner.chang@hpe.com>
>> Cc: Gilbert Chen<gilbert.chen@hpe.com>
>> Cc: Daniel Schaefer<daniel.schaefer@hpe.com>
>> Cc: Jeremy Linton<jeremy.linton@arm.com>
>> Cc: Peng Xie<xiepeng@phytium.com.cn>
>> Cc: Ling Jia<jialing@phytium.com.cn>
>> Cc: Yiqi Shu<shuyiqi@phytium.com.cn>
>>
>> Kun Qin (6):
>>    RaspberryPi: Pipeline: Resolving newly introduced dependency
>>    U5SeriesPkg: Pipeline: Resolving newly introduced dependency
>>    VExpressPkg: Pipeline: Resolving newly introduced dependency
>>    Socionext: Pipeline: Resolving newly introduced dependency
>>    AmpereAltraPkg: Pipeline: Resolving newly introduced dependency
>>    PhytiumCommonPkg: Pipeline: Resolving newly introduced dependency
>>
>>   Platform/ARM/VExpressPkg/ArmVExpress.dsc.inc                         | 1 +
>>   Platform/RaspberryPi/RPi3/RPi3.dsc                                   | 1 +
>>   Platform/RaspberryPi/RPi4/RPi4.dsc                                   | 1 +
>>   Platform/SiFive/U5SeriesPkg/FreedomU500VC707Board/U500.dsc           | 1 +
>>   Platform/SiFive/U5SeriesPkg/FreedomU540HiFiveUnleashedBoard/U540.dsc | 1 +
>>   Platform/Socionext/DeveloperBox/DeveloperBox.dsc                     | 1 +
>>   Silicon/Ampere/AmpereAltraPkg/AmpereAltraPkg.dsc.inc                 | 1 +
>>   Silicon/Phytium/PhytiumCommonPkg/PhytiumCommonPkg.dsc.inc            | 1 +
>>   8 files changed, 8 insertions(+)
>>
>> --
>> 2.37.1.windows.1
>>


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.
View/Reply Online (#92135): https://edk2.groups.io/g/devel/message/92135
Mute This Topic: https://groups.io/mt/92782210/1787277
Group Owner: devel+owner@edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub [importer@patchew.org]
-=-=-=-=-=-=-=-=-=-=-=-