Re: [edk2-devel] [edk2-platforms Patch 2/5] Vlv2TbltDevicePkg: Convert use of FCE tool to Structured PCD

["Michael D Kinney" <michael.d.kinney@intel.com>]

Liming,

That wiki page does not explain that values are
inherited from STANDARD settings.  It only shows 
an example use of multiple default stores.

Mike

> -----Original Message-----
> From: Gao, Liming
> Sent: Thursday, July 18, 2019 11:41 PM
> To: devel@edk2.groups.io; Kinney, Michael D
> <michael.d.kinney@intel.com>
> Cc: Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi
> <yi.qian@intel.com>; Gary Lin <glin@suse.com>
> Subject: RE: [edk2-devel] [edk2-platforms Patch 2/5]
> Vlv2TbltDevicePkg: Convert use of FCE tool to Structured
> PCD
> 
> Mike:
>   Yes. Wiki
> https://github.com/lgao4/edk2/wiki/StructurePcd-Enable-
> Steps shows the usage of the different default store.
> 
> >-----Original Message-----
> >From: devel@edk2.groups.io
> [mailto:devel@edk2.groups.io] On Behalf Of
> >Michael D Kinney
> >Sent: Friday, July 19, 2019 2:05 PM
> >To: Gao, Liming <liming.gao@intel.com>;
> devel@edk2.groups.io; Kinney,
> >Michael D <michael.d.kinney@intel.com>
> >Cc: Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi
> ><yi.qian@intel.com>; Gary Lin <glin@suse.com>
> >Subject: Re: [edk2-devel] [edk2-platforms Patch 2/5]
> Vlv2TbltDevicePkg:
> >Convert use of FCE tool to Structured PCD
> >
> >Liming,
> >
> >Thanks for the feedback.  I have made both changes in
> the V2 version of
> >the series.
> >
> >We should show some examples of inheriting defaults
> from STANDARD
> >defaults.
> >
> >Mike
> >
> >> -----Original Message-----
> >> From: Gao, Liming
> >> Sent: Thursday, July 18, 2019 8:41 PM
> >> To: devel@edk2.groups.io; Kinney, Michael D
> >> <michael.d.kinney@intel.com>
> >> Cc: Sun, Zailiang <zailiang.sun@intel.com>; Qian, Yi
> >> <yi.qian@intel.com>; Gary Lin <glin@suse.com>
> >> Subject: RE: [edk2-devel] [edk2-platforms Patch 2/5]
> >> Vlv2TbltDevicePkg: Convert use of FCE tool to
> Structured PCD
> >>
> >> Mike:
> >>   I have two comments here.
> >> 1. VarStore is updated to EfiVarStore. So,
> HiiConfigAccess
> >> ExtractConfig() and RouteConfig() can be updated to
> directly return
> >> UNSUPPORTED, they are not used any more.
> >> 2. DSC doesn't need to include
> PlatformSetupDefaults.dsc twice. By
> >> default, MANUFACTURING setting will be inherited from
> STANDARD. Only
> >> different default setting is required to be
> specified. For your case,
> >> MANUFACTURING default is same to STANDARD. So, you
> only need to
> >> include the default setting for STANDARD only.
> >>
> >> [PcdsDynamicExHii.common.DEFAULT.STANDARD]
> >>   !include
> Vlv2TbltDevicePkg/PlatformSetupDefaults.dsc
> >>
> >> [PcdsDynamicExHii.common.DEFAULT.MANUFACTURING]  #
> can be removed.
> >>   !include
> Vlv2TbltDevicePkg/PlatformSetupDefaults.dsc
> >> # can be removed.
> >>
> >> Thanks
> >> Liming