Re: [edk2-devel] [PATCH] ArmPkg: Enable boot discovery policy for ARM package.

2021-08-11 Thread Marcin Wojtas
Hi Ard, śr., 11 sie 2021 o 13:57 Ard Biesheuvel napisał(a): > > On Wed, 11 Aug 2021 at 13:57, Ard Biesheuvel wrote: > > > > On Tue, 10 Aug 2021 at 17:23, Sunny Wang wrote: > > > > > > Hi Ard, > > > > > > Yeah, this is a good point. Greg and I offline discussed this as well. > > > If we don't mi

Re: [edk2-devel] [PATCH] ArmPkg: Enable boot discovery policy for ARM package.

2021-08-11 Thread Ard Biesheuvel
On Wed, 11 Aug 2021 at 13:57, Ard Biesheuvel wrote: > > On Tue, 10 Aug 2021 at 17:23, Sunny Wang wrote: > > > > Hi Ard, > > > > Yeah, this is a good point. Greg and I offline discussed this as well. > > If we don't miss anything, only applying this patch without platform > > changes should be fi

Re: [edk2-devel] [PATCH] ArmPkg: Enable boot discovery policy for ARM package.

2021-08-11 Thread Ard Biesheuvel
On Tue, 10 Aug 2021 at 17:23, Sunny Wang wrote: > > Hi Ard, > > Yeah, this is a good point. Greg and I offline discussed this as well. > If we don't miss anything, only applying this patch without platform changes > should be fine. There should be no behavior change. I added some details > below

Re: [edk2-devel] [PATCH] ArmPkg: Enable boot discovery policy for ARM package.

2021-08-10 Thread Sunny Wang
Hi Ard, Yeah, this is a good point. Greg and I offline discussed this as well. If we don't miss anything, only applying this patch without platform changes should be fine. There should be no behavior change. I added some details below for your reference. - if the platform doesn’t add PcdBootDi

Re: [edk2-devel] [PATCH] ArmPkg: Enable boot discovery policy for ARM package.

2021-08-06 Thread Ard Biesheuvel
On Fri, 6 Aug 2021 at 10:30, Grzegorz Bernacki wrote: > > This commit adds code which check BootDiscoveryPolicy variable and > calls Boot Policy Manager Protocol to connect device specified by > the variable. To enable that mechanism for platform > EfiMdeModulePkgTokenSpaceGuid.PcdBootDiscoveryPol

Re: [edk2-devel] [PATCH] ArmPkg: Enable boot discovery policy for ARM package.

2021-08-06 Thread Sunny Wang
Looks good. Thanks, Greg. Reviewed-by: Sunny Wang For others' reference, this patch is similar to the one below for Rasberry Pi that just got merged. - https://edk2.groups.io/g/devel/message/78514 After merging this change, we should be able to easily enable "boot discovery policy" on other A

[edk2-devel] [PATCH] ArmPkg: Enable boot discovery policy for ARM package.

2021-08-06 Thread Grzegorz Bernacki
This commit adds code which check BootDiscoveryPolicy variable and calls Boot Policy Manager Protocol to connect device specified by the variable. To enable that mechanism for platform EfiMdeModulePkgTokenSpaceGuid.PcdBootDiscoveryPolicy PCD must be added to DSC file and BootDiscoveryPolicyUiLib sh