* Re: [edk2-devel] [PATCH] IntelSiliconPkg: Define PCDs for microcode location in flash
2021-03-26 6:49 ` [edk2-devel] [PATCH] IntelSiliconPkg: Define PCDs for microcode location in flash Ni, Ray
@ 2021-03-26 7:39 ` Ni, Ray
0 siblings, 0 replies; 2+ messages in thread
From: Ni, Ray @ 2021-03-26 7:39 UTC (permalink / raw)
To: devel@edk2.groups.io; +Cc: Chaganty, Rangasai V
Please ignore this patch mail.
I sent another series:
cover: https://edk2.groups.io/g/devel/message/73346
1/2: https://edk2.groups.io/g/devel/message/73347
2/3: https://edk2.groups.io/g/devel/message/73348
3/3: https://edk2.groups.io/g/devel/message/73349
> -----Original Message-----
> From: Ni, Ray
> Sent: Friday, March 26, 2021 2:49 PM
> To: devel@edk2.groups.io; Ni, Ray <ray.ni@intel.com>
> Cc: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>
> Subject: RE: [edk2-devel] [PATCH] IntelSiliconPkg: Define PCDs for microcode location in flash
>
> Please ignore the "Change-Id" in the commit message.
> I will remove that when checking in.
>
> > -----Original Message-----
> > From: devel@edk2.groups.io <devel@edk2.groups.io> On Behalf Of Ni, Ray
> > Sent: Friday, March 26, 2021 2:48 PM
> > To: devel@edk2.groups.io
> > Cc: Chaganty, Rangasai V <rangasai.v.chaganty@intel.com>
> > Subject: [edk2-devel] [PATCH] IntelSiliconPkg: Define PCDs for microcode location in flash
> >
> > 3 PCDs are defined assuming that microcode is put in the end of a FV:
> > PcdFlashFvMicrocodeBase: The base address of the microcode FV
> > PcdFlashFvMicrocodeSize: The size of the microcode FV
> > PcdFlashMicrocodeOffset: The microcode offset relative to the FV
> > base address
> >
> > The 3 new PCDs are to replace the following existing PCDs in
> > MinPlatformPkg and IntelFsp2WrapperPkg.
> >
> > The ones in MinPlatformPkg will be removed in a separate patch.
> > The ones in IntelFsp2WrapperPkg will be re-positioned as only using
> > inside IntelFsp2WrapperPkg\Library\SecFspWrapperPlatformSecLibSample.
> >
> > Defining PCDs in IntelSiliconPkg is because certain close-source
> > silicon code depends on these PCDs but the silicon code cannot depend
> > on MinPlatformPkg.
> >
> > MinPlatformPkg contains the below 3 PCDs:
> > PcdFlashFvMicrocodeBase: Same meaning as
> > IntelSiliconPkg.PcdFlashFvMicrocodeBase
> > PcdFlashFvMicrocodeSize: Same meaning as
> > IntelSiliconPkg.PcdFlashFvMicrocodeSize
> > PcdFlashFvMicrocodeOffset: The FV offset relative to the firmware
> > base address.
> > Note: MinPlatformPkg doesn't contain the PCD that tells the microcode
> > offset relative to the FV base address.
> >
> > IntelFsp2WrapperPkg contains the below 3 PCDs:
> > PcdCpuMicrocodePatchAddress: Same meaning as
> > IntelSiliconPkg.PcdFlashFvMicrocodeBase
> > PcdCpuMicrocodePatchRegionSize: Same meaning as
> > IntelSiliconPkg.PcdFlashFvMicrocodeSize
> > PcdFlashMicrocodeOffset: Same meaning as
> > IntelSiliconPkg.PcdFlashMicrocodeOffset
> >
> > Change-Id: I708abfa4a9309cbaedd5c4e4f483ec9f0d5e7625
> > Reviewed-by: Ray Ni <ray.ni@intel.com>
> > Cc: Rangasai V Chaganty <rangasai.v.chaganty@intel.com>
> > ---
> > Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec | 9 ++++++++-
> > 1 file changed, 8 insertions(+), 1 deletion(-)
> >
> > diff --git a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
> > index 4a2cbca5..6611c3af 100644
> > --- a/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
> > +++ b/Silicon/Intel/IntelSiliconPkg/IntelSiliconPkg.dec
> > @@ -3,7 +3,7 @@
> > #
> >
> > # This package provides common open source Intel silicon modules.
> >
> > #
> >
> > -# Copyright (c) 2016 - 2020, Intel Corporation. All rights reserved.<BR>
> >
> > +# Copyright (c) 2016 - 2021, Intel Corporation. All rights reserved.<BR>
> >
> > # SPDX-License-Identifier: BSD-2-Clause-Patent
> >
> > #
> >
> > ##
> >
> > @@ -90,6 +90,13 @@
> > # @Prompt Error code for VTd error.
> >
> > gIntelSiliconPkgTokenSpaceGuid.PcdErrorCodeVTdError|0x02008000|UINT32|0x00000005
> >
> >
> >
> > + gIntelSiliconPkgTokenSpaceGuid.PcdFlashFvMicrocodeBase|0|UINT32|0x00000007
> >
> > + gIntelSiliconPkgTokenSpaceGuid.PcdFlashFvMicrocodeSize|0|UINT32|0x00000008
> >
> > +
> >
> > + ## This is the microcode offset relative to the FV base address.
> >
> > + # Microcode address equals to PcdFlashFvMicrocodeBase + PcdFlashMicrocodeOffset.
> >
> > + gIntelSiliconPkgTokenSpaceGuid.PcdFlashMicrocodeOffset|0|UINT32|0x0000000A
> >
> > +
> >
> > [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
> >
> > ## This is the GUID of the FFS which contains the Graphics Video BIOS Table (VBT)
> >
> > # The VBT content is stored as a RAW section which is consumed by GOP PEI/UEFI driver.
> >
> > --
> > 2.27.0.windows.1
> >
> >
> >
> > -=-=-=-=-=-=
> > Groups.io Links: You receive all messages sent to this group.
> > View/Reply Online (#73313): https://edk2.groups.io/g/devel/message/73313
> > Mute This Topic: https://groups.io/mt/81622347/1712937
> > Group Owner: devel+owner@edk2.groups.io
> > Unsubscribe: https://edk2.groups.io/g/devel/unsub [ray.ni@intel.com]
> > -=-=-=-=-=-=
> >
^ permalink raw reply [flat|nested] 2+ messages in thread