From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 Updates PCDs to use the IntelSiliconPkg PCD tokenspace now that the PCDs are declared

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 Updates PCDs to use the IntelSiliconPkg PCD tokenspace now that the PCDs are declared

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 Adds a Traditional MM and Standalone MM SPI FVB Service driver to IntelSiliconPkg.

From: Michael Kubacki <michael.kubacki@...> REF:https://bugzilla.tianocore.org/show_bug.cgi?id=3D3307 SpiFlashCommonLib is duplicated in multiple places across the MinPlatform design in

Hi Omkar, Please find my response inline marked [SAMI]. Regards, Sami Mujawar On 10/07/2021 05:18 PM, Omkar Anand Kulkarni wrote: [SAMI] Should this patch

Hi Omkar, Please find my response below marked [SAMI] Regards, Sami Mujawar On 10/07/2021 05:18 PM, Omkar Anand Kulkarni wrote: [SAMI] Not sure if a

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 -

Thanks Leif, I remove the description from the package name. Abner

... Thanks Pete. I did not want to assume that stakeholders like yourself are ok with such changes if they did not get involved in the discussion. I will merge these now.

+Heinrich

I would think just sending the code contribution patch is sufficient.

Hi Ard, I thought the R-b from Sunny was enough. For what is worth, I briefly tested these changes for v4. Thus: Reviewed-by: Pete Batard <pete@...> Tested-by: Pete Batard <pete@...>

<Samer.El-Haj-Mahmoud@...> wrote: I only see acks from Sunny though. This is why I asked Pete and Andrei to chime in as well.

Ard, Now that the EDK2 changes are merged (aaecef38b9440a65809cbdaf9d97029f4eeb), I think these RPi specific changes are ready to be merged as well.

My interpretation of the normal use of this field is that we can add a comment if the entry applies only to a subset of a package. This adds an entry for a whole new package, and as such should

Reviewed-by: Leif Lindholm <leif@...> Apologies for tardiness.

Please see my comments below inline. Thanks, Chasel

wt., 3 sie 2021 o 08:53 Ard Biesheuvel <ardb@...> napisał(a): A great one though. For OS's there is indeed only bus0 exposed, but I plan to make it tunable, so that to use entire range (e.g.

v8 merged as #1850 Thanks all

Pushed as d84c0545f4b4..73ccc21d918c Thanks,