public inbox for devel@edk2.groups.io
 help / color / mirror / Atom feed
* [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion
@ 2018-05-31  9:01 Ard Biesheuvel
  2018-05-31  9:12 ` Zeng, Star
  0 siblings, 1 reply; 7+ messages in thread
From: Ard Biesheuvel @ 2018-05-31  9:01 UTC (permalink / raw)
  To: edk2-devel; +Cc: leif.lindholm, star.zeng, Ard Biesheuvel, Jiewen Yao

Add a PCD that holds the lowest supported version number, so that we
can allow the platform to configure this at build time.

Cc: Jiewen Yao <jiewen.yao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
As discussed a while ago:
https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html

 SignedCapsulePkg/SignedCapsulePkg.dec | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/SignedCapsulePkg/SignedCapsulePkg.dec b/SignedCapsulePkg/SignedCapsulePkg.dec
index b27e87409b19..f06e07656de9 100644
--- a/SignedCapsulePkg/SignedCapsulePkg.dec
+++ b/SignedCapsulePkg/SignedCapsulePkg.dec
@@ -52,6 +52,10 @@ [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
   # @Prompt GUID of the FFS which contains the Pkcs7TestPublicKeyFile.
   gEfiSignedCapsulePkgTokenSpaceGuid.PcdEdkiiPkcs7TestPublicKeyFileGuid|{0xba, 0xf5, 0x93, 0xf0, 0x37, 0x6f, 0x16, 0x48, 0x9e, 0x52, 0x91, 0xbe, 0xa0, 0xf7, 0xe0, 0xb8}|VOID*|0xA0010002
 
+  ## This is the lowest supported version number that can be upgraded to, as exposed via the System Firmware descriptor.
+  # @Prompt Lowest support version number that can be upgraded to via capsule update
+  gEfiSignedCapsulePkgTokenSpaceGuid.PcdLowestSupportedFirmwareVersion|0x1|UINT32|0xA0010003
+
 [PcdsDynamicEx]
   ## This dynamic PCD holds the EDKII system firmware image descriptor.
   #  This information can be used for version check in EDKII system FMP capsule.
-- 
2.17.0



^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion
  2018-05-31  9:01 [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion Ard Biesheuvel
@ 2018-05-31  9:12 ` Zeng, Star
  2018-06-01 10:05   ` Ard Biesheuvel
  0 siblings, 1 reply; 7+ messages in thread
From: Zeng, Star @ 2018-05-31  9:12 UTC (permalink / raw)
  To: Ard Biesheuvel, edk2-devel@lists.01.org
  Cc: Yao, Jiewen, leif.lindholm@linaro.org, Zeng, Star

It could be reused by platform SystemFirmwareDescriptor, for example.
Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf
QuarkPlatformPkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf

So, I agree this patch, Reviewed-by: Star Zeng <star.zeng@intel.com>.

Please also get Jiewen's RB for the patch.


Thanks,
Star
-----Original Message-----
From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Ard Biesheuvel
Sent: Thursday, May 31, 2018 5:02 PM
To: edk2-devel@lists.01.org
Cc: Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star <star.zeng@intel.com>; leif.lindholm@linaro.org; Ard Biesheuvel <ard.biesheuvel@linaro.org>
Subject: [edk2] [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion

Add a PCD that holds the lowest supported version number, so that we can allow the platform to configure this at build time.

Cc: Jiewen Yao <jiewen.yao@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
---
As discussed a while ago:
https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html

 SignedCapsulePkg/SignedCapsulePkg.dec | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/SignedCapsulePkg/SignedCapsulePkg.dec b/SignedCapsulePkg/SignedCapsulePkg.dec
index b27e87409b19..f06e07656de9 100644
--- a/SignedCapsulePkg/SignedCapsulePkg.dec
+++ b/SignedCapsulePkg/SignedCapsulePkg.dec
@@ -52,6 +52,10 @@ [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
   # @Prompt GUID of the FFS which contains the Pkcs7TestPublicKeyFile.
   gEfiSignedCapsulePkgTokenSpaceGuid.PcdEdkiiPkcs7TestPublicKeyFileGuid|{0xba, 0xf5, 0x93, 0xf0, 0x37, 0x6f, 0x16, 0x48, 0x9e, 0x52, 0x91, 0xbe, 0xa0, 0xf7, 0xe0, 0xb8}|VOID*|0xA0010002
 
+  ## This is the lowest supported version number that can be upgraded to, as exposed via the System Firmware descriptor.
+  # @Prompt Lowest support version number that can be upgraded to via 
+ capsule update
+  
+ gEfiSignedCapsulePkgTokenSpaceGuid.PcdLowestSupportedFirmwareVersion|0
+ x1|UINT32|0xA0010003
+
 [PcdsDynamicEx]
   ## This dynamic PCD holds the EDKII system firmware image descriptor.
   #  This information can be used for version check in EDKII system FMP capsule.
--
2.17.0

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion
  2018-05-31  9:12 ` Zeng, Star
@ 2018-06-01 10:05   ` Ard Biesheuvel
  2018-06-01 15:09     ` Yao, Jiewen
  0 siblings, 1 reply; 7+ messages in thread
From: Ard Biesheuvel @ 2018-06-01 10:05 UTC (permalink / raw)
  To: Zeng, Star, Yao, Jiewen; +Cc: edk2-devel@lists.01.org, leif.lindholm@linaro.org

On 31 May 2018 at 11:12, Zeng, Star <star.zeng@intel.com> wrote:
> It could be reused by platform SystemFirmwareDescriptor, for example.
> Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf
> QuarkPlatformPkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf
>
> So, I agree this patch, Reviewed-by: Star Zeng <star.zeng@intel.com>.
>
> Please also get Jiewen's RB for the patch.
>

@Jiewen, any comments?

Thanks,
Ard.


> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Ard Biesheuvel
> Sent: Thursday, May 31, 2018 5:02 PM
> To: edk2-devel@lists.01.org
> Cc: Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star <star.zeng@intel.com>; leif.lindholm@linaro.org; Ard Biesheuvel <ard.biesheuvel@linaro.org>
> Subject: [edk2] [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion
>
> Add a PCD that holds the lowest supported version number, so that we can allow the platform to configure this at build time.
>
> Cc: Jiewen Yao <jiewen.yao@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> ---
> As discussed a while ago:
> https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html
>
>  SignedCapsulePkg/SignedCapsulePkg.dec | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/SignedCapsulePkg/SignedCapsulePkg.dec b/SignedCapsulePkg/SignedCapsulePkg.dec
> index b27e87409b19..f06e07656de9 100644
> --- a/SignedCapsulePkg/SignedCapsulePkg.dec
> +++ b/SignedCapsulePkg/SignedCapsulePkg.dec
> @@ -52,6 +52,10 @@ [PcdsFixedAtBuild, PcdsPatchableInModule, PcdsDynamic, PcdsDynamicEx]
>    # @Prompt GUID of the FFS which contains the Pkcs7TestPublicKeyFile.
>    gEfiSignedCapsulePkgTokenSpaceGuid.PcdEdkiiPkcs7TestPublicKeyFileGuid|{0xba, 0xf5, 0x93, 0xf0, 0x37, 0x6f, 0x16, 0x48, 0x9e, 0x52, 0x91, 0xbe, 0xa0, 0xf7, 0xe0, 0xb8}|VOID*|0xA0010002
>
> +  ## This is the lowest supported version number that can be upgraded to, as exposed via the System Firmware descriptor.
> +  # @Prompt Lowest support version number that can be upgraded to via
> + capsule update
> +
> + gEfiSignedCapsulePkgTokenSpaceGuid.PcdLowestSupportedFirmwareVersion|0
> + x1|UINT32|0xA0010003
> +
>  [PcdsDynamicEx]
>    ## This dynamic PCD holds the EDKII system firmware image descriptor.
>    #  This information can be used for version check in EDKII system FMP capsule.
> --
> 2.17.0
>
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.01.org
> https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion
  2018-06-01 10:05   ` Ard Biesheuvel
@ 2018-06-01 15:09     ` Yao, Jiewen
  2018-06-01 15:10       ` Ard Biesheuvel
  0 siblings, 1 reply; 7+ messages in thread
From: Yao, Jiewen @ 2018-06-01 15:09 UTC (permalink / raw)
  To: Ard Biesheuvel, Zeng, Star
  Cc: edk2-devel@lists.01.org, leif.lindholm@linaro.org, Yao, Jiewen

I am about to agree.

Would you please share a real example on how this PCD is used in some open source platforms?

Thank you
Yao Jiewen


> -----Original Message-----
> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> Sent: Friday, June 1, 2018 3:06 AM
> To: Zeng, Star <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
> Cc: edk2-devel@lists.01.org; leif.lindholm@linaro.org
> Subject: Re: [edk2] [PATCH] SignedCapsulePkg: add
> PcdLowestSupportedFirmwareVersion
> 
> On 31 May 2018 at 11:12, Zeng, Star <star.zeng@intel.com> wrote:
> > It could be reused by platform SystemFirmwareDescriptor, for example.
> >
> Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwa
> reDescriptor.inf
> >
> QuarkPlatformPkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwa
> reDescriptor.inf
> >
> > So, I agree this patch, Reviewed-by: Star Zeng <star.zeng@intel.com>.
> >
> > Please also get Jiewen's RB for the patch.
> >
> 
> @Jiewen, any comments?
> 
> Thanks,
> Ard.
> 
> 
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Ard
> Biesheuvel
> > Sent: Thursday, May 31, 2018 5:02 PM
> > To: edk2-devel@lists.01.org
> > Cc: Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star <star.zeng@intel.com>;
> leif.lindholm@linaro.org; Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > Subject: [edk2] [PATCH] SignedCapsulePkg: add
> PcdLowestSupportedFirmwareVersion
> >
> > Add a PCD that holds the lowest supported version number, so that we can
> allow the platform to configure this at build time.
> >
> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> > Contributed-under: TianoCore Contribution Agreement 1.1
> > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> > ---
> > As discussed a while ago:
> > https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html
> >
> >  SignedCapsulePkg/SignedCapsulePkg.dec | 4 ++++
> >  1 file changed, 4 insertions(+)
> >
> > diff --git a/SignedCapsulePkg/SignedCapsulePkg.dec
> b/SignedCapsulePkg/SignedCapsulePkg.dec
> > index b27e87409b19..f06e07656de9 100644
> > --- a/SignedCapsulePkg/SignedCapsulePkg.dec
> > +++ b/SignedCapsulePkg/SignedCapsulePkg.dec
> > @@ -52,6 +52,10 @@ [PcdsFixedAtBuild, PcdsPatchableInModule,
> PcdsDynamic, PcdsDynamicEx]
> >    # @Prompt GUID of the FFS which contains the Pkcs7TestPublicKeyFile.
> >
> gEfiSignedCapsulePkgTokenSpaceGuid.PcdEdkiiPkcs7TestPublicKeyFileGuid|{0xb
> a, 0xf5, 0x93, 0xf0, 0x37, 0x6f, 0x16, 0x48, 0x9e, 0x52, 0x91, 0xbe, 0xa0, 0xf7,
> 0xe0, 0xb8}|VOID*|0xA0010002
> >
> > +  ## This is the lowest supported version number that can be upgraded to, as
> exposed via the System Firmware descriptor.
> > +  # @Prompt Lowest support version number that can be upgraded to via
> > + capsule update
> > +
> > +
> gEfiSignedCapsulePkgTokenSpaceGuid.PcdLowestSupportedFirmwareVersion|0
> > + x1|UINT32|0xA0010003
> > +
> >  [PcdsDynamicEx]
> >    ## This dynamic PCD holds the EDKII system firmware image descriptor.
> >    #  This information can be used for version check in EDKII system FMP
> capsule.
> > --
> > 2.17.0
> >
> > _______________________________________________
> > edk2-devel mailing list
> > edk2-devel@lists.01.org
> > https://lists.01.org/mailman/listinfo/edk2-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion
  2018-06-01 15:09     ` Yao, Jiewen
@ 2018-06-01 15:10       ` Ard Biesheuvel
  2018-06-01 15:13         ` Yao, Jiewen
  0 siblings, 1 reply; 7+ messages in thread
From: Ard Biesheuvel @ 2018-06-01 15:10 UTC (permalink / raw)
  To: Yao, Jiewen; +Cc: Zeng, Star, edk2-devel@lists.01.org, leif.lindholm@linaro.org

On 1 June 2018 at 17:09, Yao, Jiewen <jiewen.yao@intel.com> wrote:
> I am about to agree.
>
> Would you please share a real example on how this PCD is used in some open source platforms?
>

Please refer to this patch
https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html


Thanks,
Ard.

>> -----Original Message-----
>> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
>> Sent: Friday, June 1, 2018 3:06 AM
>> To: Zeng, Star <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
>> Cc: edk2-devel@lists.01.org; leif.lindholm@linaro.org
>> Subject: Re: [edk2] [PATCH] SignedCapsulePkg: add
>> PcdLowestSupportedFirmwareVersion
>>
>> On 31 May 2018 at 11:12, Zeng, Star <star.zeng@intel.com> wrote:
>> > It could be reused by platform SystemFirmwareDescriptor, for example.
>> >
>> Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwa
>> reDescriptor.inf
>> >
>> QuarkPlatformPkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwa
>> reDescriptor.inf
>> >
>> > So, I agree this patch, Reviewed-by: Star Zeng <star.zeng@intel.com>.
>> >
>> > Please also get Jiewen's RB for the patch.
>> >
>>
>> @Jiewen, any comments?
>>
>> Thanks,
>> Ard.
>>
>>
>> > -----Original Message-----
>> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of Ard
>> Biesheuvel
>> > Sent: Thursday, May 31, 2018 5:02 PM
>> > To: edk2-devel@lists.01.org
>> > Cc: Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star <star.zeng@intel.com>;
>> leif.lindholm@linaro.org; Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> > Subject: [edk2] [PATCH] SignedCapsulePkg: add
>> PcdLowestSupportedFirmwareVersion
>> >
>> > Add a PCD that holds the lowest supported version number, so that we can
>> allow the platform to configure this at build time.
>> >
>> > Cc: Jiewen Yao <jiewen.yao@intel.com>
>> > Contributed-under: TianoCore Contribution Agreement 1.1
>> > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> > ---
>> > As discussed a while ago:
>> > https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html
>> >
>> >  SignedCapsulePkg/SignedCapsulePkg.dec | 4 ++++
>> >  1 file changed, 4 insertions(+)
>> >
>> > diff --git a/SignedCapsulePkg/SignedCapsulePkg.dec
>> b/SignedCapsulePkg/SignedCapsulePkg.dec
>> > index b27e87409b19..f06e07656de9 100644
>> > --- a/SignedCapsulePkg/SignedCapsulePkg.dec
>> > +++ b/SignedCapsulePkg/SignedCapsulePkg.dec
>> > @@ -52,6 +52,10 @@ [PcdsFixedAtBuild, PcdsPatchableInModule,
>> PcdsDynamic, PcdsDynamicEx]
>> >    # @Prompt GUID of the FFS which contains the Pkcs7TestPublicKeyFile.
>> >
>> gEfiSignedCapsulePkgTokenSpaceGuid.PcdEdkiiPkcs7TestPublicKeyFileGuid|{0xb
>> a, 0xf5, 0x93, 0xf0, 0x37, 0x6f, 0x16, 0x48, 0x9e, 0x52, 0x91, 0xbe, 0xa0, 0xf7,
>> 0xe0, 0xb8}|VOID*|0xA0010002
>> >
>> > +  ## This is the lowest supported version number that can be upgraded to, as
>> exposed via the System Firmware descriptor.
>> > +  # @Prompt Lowest support version number that can be upgraded to via
>> > + capsule update
>> > +
>> > +
>> gEfiSignedCapsulePkgTokenSpaceGuid.PcdLowestSupportedFirmwareVersion|0
>> > + x1|UINT32|0xA0010003
>> > +
>> >  [PcdsDynamicEx]
>> >    ## This dynamic PCD holds the EDKII system firmware image descriptor.
>> >    #  This information can be used for version check in EDKII system FMP
>> capsule.
>> > --
>> > 2.17.0
>> >
>> > _______________________________________________
>> > edk2-devel mailing list
>> > edk2-devel@lists.01.org
>> > https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion
  2018-06-01 15:10       ` Ard Biesheuvel
@ 2018-06-01 15:13         ` Yao, Jiewen
  2018-06-01 15:17           ` Ard Biesheuvel
  0 siblings, 1 reply; 7+ messages in thread
From: Yao, Jiewen @ 2018-06-01 15:13 UTC (permalink / raw)
  To: Ard Biesheuvel
  Cc: Zeng, Star, edk2-devel@lists.01.org, leif.lindholm@linaro.org

Got it. Thanks.

Reviewed-by: Jiewen.yao@intel.com


> -----Original Message-----
> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> Sent: Friday, June 1, 2018 8:11 AM
> To: Yao, Jiewen <jiewen.yao@intel.com>
> Cc: Zeng, Star <star.zeng@intel.com>; edk2-devel@lists.01.org;
> leif.lindholm@linaro.org
> Subject: Re: [edk2] [PATCH] SignedCapsulePkg: add
> PcdLowestSupportedFirmwareVersion
> 
> On 1 June 2018 at 17:09, Yao, Jiewen <jiewen.yao@intel.com> wrote:
> > I am about to agree.
> >
> > Would you please share a real example on how this PCD is used in some open
> source platforms?
> >
> 
> Please refer to this patch
> https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html
> 
> 
> Thanks,
> Ard.
> 
> >> -----Original Message-----
> >> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
> >> Sent: Friday, June 1, 2018 3:06 AM
> >> To: Zeng, Star <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
> >> Cc: edk2-devel@lists.01.org; leif.lindholm@linaro.org
> >> Subject: Re: [edk2] [PATCH] SignedCapsulePkg: add
> >> PcdLowestSupportedFirmwareVersion
> >>
> >> On 31 May 2018 at 11:12, Zeng, Star <star.zeng@intel.com> wrote:
> >> > It could be reused by platform SystemFirmwareDescriptor, for example.
> >> >
> >>
> Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwa
> >> reDescriptor.inf
> >> >
> >>
> QuarkPlatformPkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwa
> >> reDescriptor.inf
> >> >
> >> > So, I agree this patch, Reviewed-by: Star Zeng <star.zeng@intel.com>.
> >> >
> >> > Please also get Jiewen's RB for the patch.
> >> >
> >>
> >> @Jiewen, any comments?
> >>
> >> Thanks,
> >> Ard.
> >>
> >>
> >> > -----Original Message-----
> >> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Ard
> >> Biesheuvel
> >> > Sent: Thursday, May 31, 2018 5:02 PM
> >> > To: edk2-devel@lists.01.org
> >> > Cc: Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star
> <star.zeng@intel.com>;
> >> leif.lindholm@linaro.org; Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >> > Subject: [edk2] [PATCH] SignedCapsulePkg: add
> >> PcdLowestSupportedFirmwareVersion
> >> >
> >> > Add a PCD that holds the lowest supported version number, so that we can
> >> allow the platform to configure this at build time.
> >> >
> >> > Cc: Jiewen Yao <jiewen.yao@intel.com>
> >> > Contributed-under: TianoCore Contribution Agreement 1.1
> >> > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
> >> > ---
> >> > As discussed a while ago:
> >> > https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html
> >> >
> >> >  SignedCapsulePkg/SignedCapsulePkg.dec | 4 ++++
> >> >  1 file changed, 4 insertions(+)
> >> >
> >> > diff --git a/SignedCapsulePkg/SignedCapsulePkg.dec
> >> b/SignedCapsulePkg/SignedCapsulePkg.dec
> >> > index b27e87409b19..f06e07656de9 100644
> >> > --- a/SignedCapsulePkg/SignedCapsulePkg.dec
> >> > +++ b/SignedCapsulePkg/SignedCapsulePkg.dec
> >> > @@ -52,6 +52,10 @@ [PcdsFixedAtBuild, PcdsPatchableInModule,
> >> PcdsDynamic, PcdsDynamicEx]
> >> >    # @Prompt GUID of the FFS which contains the Pkcs7TestPublicKeyFile.
> >> >
> >>
> gEfiSignedCapsulePkgTokenSpaceGuid.PcdEdkiiPkcs7TestPublicKeyFileGuid|{0xb
> >> a, 0xf5, 0x93, 0xf0, 0x37, 0x6f, 0x16, 0x48, 0x9e, 0x52, 0x91, 0xbe, 0xa0,
> 0xf7,
> >> 0xe0, 0xb8}|VOID*|0xA0010002
> >> >
> >> > +  ## This is the lowest supported version number that can be upgraded to,
> as
> >> exposed via the System Firmware descriptor.
> >> > +  # @Prompt Lowest support version number that can be upgraded to via
> >> > + capsule update
> >> > +
> >> > +
> >>
> gEfiSignedCapsulePkgTokenSpaceGuid.PcdLowestSupportedFirmwareVersion|0
> >> > + x1|UINT32|0xA0010003
> >> > +
> >> >  [PcdsDynamicEx]
> >> >    ## This dynamic PCD holds the EDKII system firmware image descriptor.
> >> >    #  This information can be used for version check in EDKII system FMP
> >> capsule.
> >> > --
> >> > 2.17.0
> >> >
> >> > _______________________________________________
> >> > edk2-devel mailing list
> >> > edk2-devel@lists.01.org
> >> > https://lists.01.org/mailman/listinfo/edk2-devel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion
  2018-06-01 15:13         ` Yao, Jiewen
@ 2018-06-01 15:17           ` Ard Biesheuvel
  0 siblings, 0 replies; 7+ messages in thread
From: Ard Biesheuvel @ 2018-06-01 15:17 UTC (permalink / raw)
  To: Yao, Jiewen; +Cc: Zeng, Star, edk2-devel@lists.01.org, leif.lindholm@linaro.org

On 1 June 2018 at 17:13, Yao, Jiewen <jiewen.yao@intel.com> wrote:
> Got it. Thanks.
>
> Reviewed-by: Jiewen.yao@intel.com
>

Thanks all

Pushed as c4061d18ef53

>
>> -----Original Message-----
>> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
>> Sent: Friday, June 1, 2018 8:11 AM
>> To: Yao, Jiewen <jiewen.yao@intel.com>
>> Cc: Zeng, Star <star.zeng@intel.com>; edk2-devel@lists.01.org;
>> leif.lindholm@linaro.org
>> Subject: Re: [edk2] [PATCH] SignedCapsulePkg: add
>> PcdLowestSupportedFirmwareVersion
>>
>> On 1 June 2018 at 17:09, Yao, Jiewen <jiewen.yao@intel.com> wrote:
>> > I am about to agree.
>> >
>> > Would you please share a real example on how this PCD is used in some open
>> source platforms?
>> >
>>
>> Please refer to this patch
>> https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html
>>
>>
>> Thanks,
>> Ard.
>>
>> >> -----Original Message-----
>> >> From: Ard Biesheuvel [mailto:ard.biesheuvel@linaro.org]
>> >> Sent: Friday, June 1, 2018 3:06 AM
>> >> To: Zeng, Star <star.zeng@intel.com>; Yao, Jiewen <jiewen.yao@intel.com>
>> >> Cc: edk2-devel@lists.01.org; leif.lindholm@linaro.org
>> >> Subject: Re: [edk2] [PATCH] SignedCapsulePkg: add
>> >> PcdLowestSupportedFirmwareVersion
>> >>
>> >> On 31 May 2018 at 11:12, Zeng, Star <star.zeng@intel.com> wrote:
>> >> > It could be reused by platform SystemFirmwareDescriptor, for example.
>> >> >
>> >>
>> Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwa
>> >> reDescriptor.inf
>> >> >
>> >>
>> QuarkPlatformPkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwa
>> >> reDescriptor.inf
>> >> >
>> >> > So, I agree this patch, Reviewed-by: Star Zeng <star.zeng@intel.com>.
>> >> >
>> >> > Please also get Jiewen's RB for the patch.
>> >> >
>> >>
>> >> @Jiewen, any comments?
>> >>
>> >> Thanks,
>> >> Ard.
>> >>
>> >>
>> >> > -----Original Message-----
>> >> > From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
>> Ard
>> >> Biesheuvel
>> >> > Sent: Thursday, May 31, 2018 5:02 PM
>> >> > To: edk2-devel@lists.01.org
>> >> > Cc: Yao, Jiewen <jiewen.yao@intel.com>; Zeng, Star
>> <star.zeng@intel.com>;
>> >> leif.lindholm@linaro.org; Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> >> > Subject: [edk2] [PATCH] SignedCapsulePkg: add
>> >> PcdLowestSupportedFirmwareVersion
>> >> >
>> >> > Add a PCD that holds the lowest supported version number, so that we can
>> >> allow the platform to configure this at build time.
>> >> >
>> >> > Cc: Jiewen Yao <jiewen.yao@intel.com>
>> >> > Contributed-under: TianoCore Contribution Agreement 1.1
>> >> > Signed-off-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
>> >> > ---
>> >> > As discussed a while ago:
>> >> > https://lists.01.org/pipermail/edk2-devel/2018-March/022914.html
>> >> >
>> >> >  SignedCapsulePkg/SignedCapsulePkg.dec | 4 ++++
>> >> >  1 file changed, 4 insertions(+)
>> >> >
>> >> > diff --git a/SignedCapsulePkg/SignedCapsulePkg.dec
>> >> b/SignedCapsulePkg/SignedCapsulePkg.dec
>> >> > index b27e87409b19..f06e07656de9 100644
>> >> > --- a/SignedCapsulePkg/SignedCapsulePkg.dec
>> >> > +++ b/SignedCapsulePkg/SignedCapsulePkg.dec
>> >> > @@ -52,6 +52,10 @@ [PcdsFixedAtBuild, PcdsPatchableInModule,
>> >> PcdsDynamic, PcdsDynamicEx]
>> >> >    # @Prompt GUID of the FFS which contains the Pkcs7TestPublicKeyFile.
>> >> >
>> >>
>> gEfiSignedCapsulePkgTokenSpaceGuid.PcdEdkiiPkcs7TestPublicKeyFileGuid|{0xb
>> >> a, 0xf5, 0x93, 0xf0, 0x37, 0x6f, 0x16, 0x48, 0x9e, 0x52, 0x91, 0xbe, 0xa0,
>> 0xf7,
>> >> 0xe0, 0xb8}|VOID*|0xA0010002
>> >> >
>> >> > +  ## This is the lowest supported version number that can be upgraded to,
>> as
>> >> exposed via the System Firmware descriptor.
>> >> > +  # @Prompt Lowest support version number that can be upgraded to via
>> >> > + capsule update
>> >> > +
>> >> > +
>> >>
>> gEfiSignedCapsulePkgTokenSpaceGuid.PcdLowestSupportedFirmwareVersion|0
>> >> > + x1|UINT32|0xA0010003
>> >> > +
>> >> >  [PcdsDynamicEx]
>> >> >    ## This dynamic PCD holds the EDKII system firmware image descriptor.
>> >> >    #  This information can be used for version check in EDKII system FMP
>> >> capsule.
>> >> > --
>> >> > 2.17.0
>> >> >
>> >> > _______________________________________________
>> >> > edk2-devel mailing list
>> >> > edk2-devel@lists.01.org
>> >> > https://lists.01.org/mailman/listinfo/edk2-devel


^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2018-06-01 15:17 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-05-31  9:01 [PATCH] SignedCapsulePkg: add PcdLowestSupportedFirmwareVersion Ard Biesheuvel
2018-05-31  9:12 ` Zeng, Star
2018-06-01 10:05   ` Ard Biesheuvel
2018-06-01 15:09     ` Yao, Jiewen
2018-06-01 15:10       ` Ard Biesheuvel
2018-06-01 15:13         ` Yao, Jiewen
2018-06-01 15:17           ` Ard Biesheuvel

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox