Re: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update.
Good question again. That is because the platform PEI phase does not report all FV information to DXE. I did use this way in my early version. I give up later, because I realize if so we need update all platforms to let it report all FV information. It is a burden. Using PCD is a standalone solution and much simpler. Also, if there is some other components need to know the information, they can just use PCD, instead of scanning FV/FD. Thank you Yao Jiewen From: Kinney, Michael D Sent: Thursday, October 27, 2016 9:23 AM To: Yao, Jiewen <jiewen@intel.com>; edk2-devel@lists.01.org; Kinney, Michael D <michael.d.kin...@intel.com> Cc: Tian, Feng <feng.t...@intel.com>; Gao, Liming <liming@intel.com>; Zeng, Star <star.z...@intel.com>; Zhang, Chao B <chao.b.zh...@intel.com>; Wei, David <david....@intel.com> Subject: RE: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update. Jiewen, They why do we need the PCD? We should be able to find this section in current FLASH image and in new FLASH image. Right? Mike From: Yao, Jiewen Sent: Wednesday, October 26, 2016 6:15 PM To: Kinney, Michael D <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>>; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> Cc: Tian, Feng <feng.t...@intel.com<mailto:feng.t...@intel.com>>; Gao, Liming <liming@intel.com<mailto:liming@intel.com>>; Zeng, Star <star.z...@intel.com<mailto:star.z...@intel.com>>; Zhang, Chao B <chao.b.zh...@intel.com<mailto:chao.b.zh...@intel.com>>; Wei, David <david@intel.com<mailto:david@intel.com>> Subject: RE: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update. Mike That is a good question. We create a standalone FFS SECTION to hold the descriptor purposely. The reason is that: the SystemFirmwareUpdate need to know the version of the *new image*. SystemFirmwareUpdate can know the version and lowest supported version of the current image by using PCD. SystemFirmwareUpdate also need to know the version of the new image, so that it can compare them. There is not any inform on the new image, because no code runs for the new image. The only way is to parse the image binary. The SystemFirmwareUpdate will parse the FV and find the FFS Section to get the image descriptor. That is why we put a standalone FFS SECTION there. Thank you Yao Jiewen From: Kinney, Michael D Sent: Thursday, October 27, 2016 8:31 AM To: Yao, Jiewen <jiewen@intel.com<mailto:jiewen@intel.com>>; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>; Kinney, Michael D <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>> Cc: Tian, Feng <feng.t...@intel.com<mailto:feng.t...@intel.com>>; Gao, Liming <liming@intel.com<mailto:liming@intel.com>>; Zeng, Star <star.z...@intel.com<mailto:star.z...@intel.com>>; Zhang, Chao B <chao.b.zh...@intel.com<mailto:chao.b.zh...@intel.com>>; Wei, David <david@intel.com<mailto:david@intel.com>> Subject: RE: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update. Jiewen, Why is .aslc file used in this PEIM. It is not an ACPI table. You are generating a C structure that is used to set the value of a PCD. The C structure could be a global variable in the PEIM that is initialized the same way the .aslc file does or you could initialize the fields in the PEIM entry point. Or some combination of the two. Thanks, Mike > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiewen > Yao > Sent: Saturday, October 22, 2016 7:33 PM > To: edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> > Cc: Tian, Feng <feng.t...@intel.com<mailto:feng.t...@intel.com>>; Gao, Liming > <liming@intel.com<mailto:liming@intel.com>>; Zeng, Star > <star.z...@intel.com<mailto:star.z...@intel.com>>; Kinney, Michael D > <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>>; Zhang, Chao B > <chao.b.zh...@intel.com<mailto:chao.b.zh...@intel.com>>; Wei, David > <david@intel.com<mailto:david@intel.com>> > Subject: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: > Add > Descriptor for capsule update. > > Add SystemFirmwareDescriptor for capsule update. > The PEIM extracts SystemFirmwareDescriptor info from FFS and reports it via > PCD. > > Cc: David Wei <david@intel.com<mailto:david@intel.com>> > Cc: Feng Tian <feng.t...@intel.com<mailto:feng.t...@intel.com>> > Cc: Star Zeng <star.z...@intel.com<mailto:star.
Re: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update.
Jiewen, They why do we need the PCD? We should be able to find this section in current FLASH image and in new FLASH image. Right? Mike From: Yao, Jiewen Sent: Wednesday, October 26, 2016 6:15 PM To: Kinney, Michael D <michael.d.kin...@intel.com>; edk2-devel@lists.01.org Cc: Tian, Feng <feng.t...@intel.com>; Gao, Liming <liming@intel.com>; Zeng, Star <star.z...@intel.com>; Zhang, Chao B <chao.b.zh...@intel.com>; Wei, David <david....@intel.com> Subject: RE: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update. Mike That is a good question. We create a standalone FFS SECTION to hold the descriptor purposely. The reason is that: the SystemFirmwareUpdate need to know the version of the *new image*. SystemFirmwareUpdate can know the version and lowest supported version of the current image by using PCD. SystemFirmwareUpdate also need to know the version of the new image, so that it can compare them. There is not any inform on the new image, because no code runs for the new image. The only way is to parse the image binary. The SystemFirmwareUpdate will parse the FV and find the FFS Section to get the image descriptor. That is why we put a standalone FFS SECTION there. Thank you Yao Jiewen From: Kinney, Michael D Sent: Thursday, October 27, 2016 8:31 AM To: Yao, Jiewen <jiewen@intel.com<mailto:jiewen@intel.com>>; edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org>; Kinney, Michael D <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>> Cc: Tian, Feng <feng.t...@intel.com<mailto:feng.t...@intel.com>>; Gao, Liming <liming@intel.com<mailto:liming@intel.com>>; Zeng, Star <star.z...@intel.com<mailto:star.z...@intel.com>>; Zhang, Chao B <chao.b.zh...@intel.com<mailto:chao.b.zh...@intel.com>>; Wei, David <david@intel.com<mailto:david@intel.com>> Subject: RE: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update. Jiewen, Why is .aslc file used in this PEIM. It is not an ACPI table. You are generating a C structure that is used to set the value of a PCD. The C structure could be a global variable in the PEIM that is initialized the same way the .aslc file does or you could initialize the fields in the PEIM entry point. Or some combination of the two. Thanks, Mike > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiewen > Yao > Sent: Saturday, October 22, 2016 7:33 PM > To: edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> > Cc: Tian, Feng <feng.t...@intel.com<mailto:feng.t...@intel.com>>; Gao, Liming > <liming@intel.com<mailto:liming@intel.com>>; Zeng, Star > <star.z...@intel.com<mailto:star.z...@intel.com>>; Kinney, Michael D > <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>>; Zhang, Chao B > <chao.b.zh...@intel.com<mailto:chao.b.zh...@intel.com>>; Wei, David > <david@intel.com<mailto:david@intel.com>> > Subject: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: > Add > Descriptor for capsule update. > > Add SystemFirmwareDescriptor for capsule update. > The PEIM extracts SystemFirmwareDescriptor info from FFS and reports it via > PCD. > > Cc: David Wei <david@intel.com<mailto:david@intel.com>> > Cc: Feng Tian <feng.t...@intel.com<mailto:feng.t...@intel.com>> > Cc: Star Zeng <star.z...@intel.com<mailto:star.z...@intel.com>> > Cc: Michael D Kinney > <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>> > Cc: Liming Gao <liming@intel.com<mailto:liming@intel.com>> > Cc: Chao Zhang <chao.b.zh...@intel.com<mailto:chao.b.zh...@intel.com>> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jiewen Yao <jiewen@intel.com<mailto:jiewen@intel.com>> > Reviewed-by: David Wei <david@intel.com<mailto:david@intel.com>> > --- > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.asl > c | 89 > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf > | 46 ++ > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptorPei. > c | 66 +++ > 3 files changed, 201 insertions(+) > > diff --git > a/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.a > slc > b/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.a > slc > new file mode 100644 > index
Re: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update.
Mike That is a good question. We create a standalone FFS SECTION to hold the descriptor purposely. The reason is that: the SystemFirmwareUpdate need to know the version of the *new image*. SystemFirmwareUpdate can know the version and lowest supported version of the current image by using PCD. SystemFirmwareUpdate also need to know the version of the new image, so that it can compare them. There is not any inform on the new image, because no code runs for the new image. The only way is to parse the image binary. The SystemFirmwareUpdate will parse the FV and find the FFS Section to get the image descriptor. That is why we put a standalone FFS SECTION there. Thank you Yao Jiewen From: Kinney, Michael D Sent: Thursday, October 27, 2016 8:31 AM To: Yao, Jiewen <jiewen@intel.com>; edk2-devel@lists.01.org; Kinney, Michael D <michael.d.kin...@intel.com> Cc: Tian, Feng <feng.t...@intel.com>; Gao, Liming <liming@intel.com>; Zeng, Star <star.z...@intel.com>; Zhang, Chao B <chao.b.zh...@intel.com>; Wei, David <david....@intel.com> Subject: RE: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update. Jiewen, Why is .aslc file used in this PEIM. It is not an ACPI table. You are generating a C structure that is used to set the value of a PCD. The C structure could be a global variable in the PEIM that is initialized the same way the .aslc file does or you could initialize the fields in the PEIM entry point. Or some combination of the two. Thanks, Mike > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiewen > Yao > Sent: Saturday, October 22, 2016 7:33 PM > To: edk2-devel@lists.01.org<mailto:edk2-devel@lists.01.org> > Cc: Tian, Feng <feng.t...@intel.com<mailto:feng.t...@intel.com>>; Gao, Liming > <liming@intel.com<mailto:liming@intel.com>>; Zeng, Star > <star.z...@intel.com<mailto:star.z...@intel.com>>; Kinney, Michael D > <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>>; Zhang, Chao B > <chao.b.zh...@intel.com<mailto:chao.b.zh...@intel.com>>; Wei, David > <david....@intel.com<mailto:david@intel.com>> > Subject: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: > Add > Descriptor for capsule update. > > Add SystemFirmwareDescriptor for capsule update. > The PEIM extracts SystemFirmwareDescriptor info from FFS and reports it via > PCD. > > Cc: David Wei <david@intel.com<mailto:david@intel.com>> > Cc: Feng Tian <feng.t...@intel.com<mailto:feng.t...@intel.com>> > Cc: Star Zeng <star.z...@intel.com<mailto:star.z...@intel.com>> > Cc: Michael D Kinney > <michael.d.kin...@intel.com<mailto:michael.d.kin...@intel.com>> > Cc: Liming Gao <liming@intel.com<mailto:liming@intel.com>> > Cc: Chao Zhang <chao.b.zh...@intel.com<mailto:chao.b.zh...@intel.com>> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jiewen Yao <jiewen@intel.com<mailto:jiewen@intel.com>> > Reviewed-by: David Wei <david@intel.com<mailto:david@intel.com>> > --- > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.asl > c | 89 > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf > | 46 ++ > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptorPei. > c | 66 +++ > 3 files changed, 201 insertions(+) > > diff --git > a/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.a > slc > b/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.a > slc > new file mode 100644 > index 000..4c9b63e > --- /dev/null > +++ > b/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.a > slc > @@ -0,0 +1,89 @@ > +/** @file > + System Firmware descriptor. > + > + Copyright (c) 2016, Intel Corporation. All rights reserved. > + This program and the accompanying materials > + are licensed and made available under the terms and conditions of the BSD > License > + which accompanies this distribution. The full text of the license may be > found at > + http://opensource.org/licenses/bsd-license.php > + > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR > IMPLIED. > + > +**/ > + > +#include > +#include > +#include > + > +#define
Re: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update.
Jiewen, Why is .aslc file used in this PEIM. It is not an ACPI table. You are generating a C structure that is used to set the value of a PCD. The C structure could be a global variable in the PEIM that is initialized the same way the .aslc file does or you could initialize the fields in the PEIM entry point. Or some combination of the two. Thanks, Mike > -Original Message- > From: edk2-devel [mailto:edk2-devel-boun...@lists.01.org] On Behalf Of Jiewen > Yao > Sent: Saturday, October 22, 2016 7:33 PM > To: edk2-devel@lists.01.org > Cc: Tian, Feng <feng.t...@intel.com>; Gao, Liming <liming@intel.com>; > Zeng, Star > <star.z...@intel.com>; Kinney, Michael D <michael.d.kin...@intel.com>; Zhang, > Chao B > <chao.b.zh...@intel.com>; Wei, David <david@intel.com> > Subject: [edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: > Add > Descriptor for capsule update. > > Add SystemFirmwareDescriptor for capsule update. > The PEIM extracts SystemFirmwareDescriptor info from FFS and reports it via > PCD. > > Cc: David Wei <david@intel.com> > Cc: Feng Tian <feng.t...@intel.com> > Cc: Star Zeng <star.z...@intel.com> > Cc: Michael D Kinney <michael.d.kin...@intel.com> > Cc: Liming Gao <liming@intel.com> > Cc: Chao Zhang <chao.b.zh...@intel.com> > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Jiewen Yao <jiewen@intel.com> > Reviewed-by: David Wei <david@intel.com> > --- > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.asl > c | 89 > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf > | 46 ++ > > Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptorPei. > c | 66 +++ > 3 files changed, 201 insertions(+) > > diff --git > a/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.a > slc > b/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.a > slc > new file mode 100644 > index 000..4c9b63e > --- /dev/null > +++ > b/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.a > slc > @@ -0,0 +1,89 @@ > +/** @file > + System Firmware descriptor. > + > + Copyright (c) 2016, Intel Corporation. All rights reserved. > + This program and the accompanying materials > + are licensed and made available under the terms and conditions of the BSD > License > + which accompanies this distribution. The full text of the license may be > found at > + http://opensource.org/licenses/bsd-license.php > + > + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, > + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR > IMPLIED. > + > +**/ > + > +#include > +#include > +#include > + > +#define PACKAGE_VERSION 0x > +#define PACKAGE_VERSION_STRING L"Unknown" > + > +#define CURRENT_FIRMWARE_VERSION0x0002 > +#define CURRENT_FIRMWARE_VERSION_STRING L"0x0002" > +#define LOWEST_SUPPORTED_FIRMWARE_VERSION 0x0001 > + > +#define IMAGE_IDSIGNATURE_64('V', 'L', 'V', '2', > '_', '_', > 'F', 'd') > +#define IMAGE_ID_STRING L"Vlv2Fd" > + > +// PcdSystemFmpCapsuleImageTypeIdGuid > +#define IMAGE_TYPE_ID_GUID { 0x4096267b, 0xda0a, 0x42eb, { > 0xb5, > 0xeb, 0xfe, 0xf3, 0x1d, 0x20, 0x7c, 0xb4 } } > + > +typedef struct { > + EDKII_SYSTEM_FIRMWARE_IMAGE_DESCRIPTOR Descriptor; > + // real string data > + CHAR16 ImageIdNameStr[16]; > + CHAR16 VersionNameStr[16]; > + CHAR16 PackageVersionNameStr[16]; > +} IMAGE_DESCRIPTOR; > + > +IMAGE_DESCRIPTOR mImageDescriptor = > +{ > + { > +EDKII_SYSTEM_FIRMWARE_IMAGE_DESCRIPTOR_SIGNATURE, > +sizeof(EDKII_SYSTEM_FIRMWARE_IMAGE_DESCRIPTOR), > +sizeof(IMAGE_DESCRIPTOR), > +PACKAGE_VERSION, // PackageVersion > +OFFSET_OF (IMAGE_DESCRIPTOR, PackageVersionNameStr), // > PackageVersionName > +1, // ImageIndex; > +{0x0}, // Reserved > +IMAGE_TYPE_ID_GUID,// ImageTypeId; > +IMAGE_ID, // ImageId; &g
[edk2] [PATCH V4 03/10] Vlv2TbltDevicePkg/SystemFirmwareDescriptor: Add Descriptor for capsule update.
Add SystemFirmwareDescriptor for capsule update. The PEIM extracts SystemFirmwareDescriptor info from FFS and reports it via PCD. Cc: David WeiCc: Feng Tian Cc: Star Zeng Cc: Michael D Kinney Cc: Liming Gao Cc: Chao Zhang Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Jiewen Yao Reviewed-by: David Wei --- Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.aslc | 89 Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.inf | 46 ++ Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptorPei.c | 66 +++ 3 files changed, 201 insertions(+) diff --git a/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.aslc b/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.aslc new file mode 100644 index 000..4c9b63e --- /dev/null +++ b/Vlv2TbltDevicePkg/Feature/Capsule/SystemFirmwareDescriptor/SystemFirmwareDescriptor.aslc @@ -0,0 +1,89 @@ +/** @file + System Firmware descriptor. + + Copyright (c) 2016, Intel Corporation. All rights reserved. + This program and the accompanying materials + are licensed and made available under the terms and conditions of the BSD License + which accompanies this distribution. The full text of the license may be found at + http://opensource.org/licenses/bsd-license.php + + THE PROGRAM IS DISTRIBUTED UNDER THE BSD LICENSE ON AN "AS IS" BASIS, + WITHOUT WARRANTIES OR REPRESENTATIONS OF ANY KIND, EITHER EXPRESS OR IMPLIED. + +**/ + +#include +#include +#include + +#define PACKAGE_VERSION 0x +#define PACKAGE_VERSION_STRING L"Unknown" + +#define CURRENT_FIRMWARE_VERSION0x0002 +#define CURRENT_FIRMWARE_VERSION_STRING L"0x0002" +#define LOWEST_SUPPORTED_FIRMWARE_VERSION 0x0001 + +#define IMAGE_IDSIGNATURE_64('V', 'L', 'V', '2', '_', '_', 'F', 'd') +#define IMAGE_ID_STRING L"Vlv2Fd" + +// PcdSystemFmpCapsuleImageTypeIdGuid +#define IMAGE_TYPE_ID_GUID { 0x4096267b, 0xda0a, 0x42eb, { 0xb5, 0xeb, 0xfe, 0xf3, 0x1d, 0x20, 0x7c, 0xb4 } } + +typedef struct { + EDKII_SYSTEM_FIRMWARE_IMAGE_DESCRIPTOR Descriptor; + // real string data + CHAR16 ImageIdNameStr[16]; + CHAR16 VersionNameStr[16]; + CHAR16 PackageVersionNameStr[16]; +} IMAGE_DESCRIPTOR; + +IMAGE_DESCRIPTOR mImageDescriptor = +{ + { +EDKII_SYSTEM_FIRMWARE_IMAGE_DESCRIPTOR_SIGNATURE, +sizeof(EDKII_SYSTEM_FIRMWARE_IMAGE_DESCRIPTOR), +sizeof(IMAGE_DESCRIPTOR), +PACKAGE_VERSION, // PackageVersion +OFFSET_OF (IMAGE_DESCRIPTOR, PackageVersionNameStr), // PackageVersionName +1, // ImageIndex; +{0x0}, // Reserved +IMAGE_TYPE_ID_GUID,// ImageTypeId; +IMAGE_ID, // ImageId; +OFFSET_OF (IMAGE_DESCRIPTOR, ImageIdNameStr), // ImageIdName; +CURRENT_FIRMWARE_VERSION, // Version; +OFFSET_OF (IMAGE_DESCRIPTOR, VersionNameStr), // VersionName; +{0x0}, // Reserved2 +FixedPcdGet32(PcdFlashAreaSize), // Size; +IMAGE_ATTRIBUTE_IMAGE_UPDATABLE | + IMAGE_ATTRIBUTE_RESET_REQUIRED | + IMAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED | + IMAGE_ATTRIBUTE_IN_USE, // AttributesSupported; +IMAGE_ATTRIBUTE_IMAGE_UPDATABLE | + IMAGE_ATTRIBUTE_RESET_REQUIRED | + IMAGE_ATTRIBUTE_AUTHENTICATION_REQUIRED | + IMAGE_ATTRIBUTE_IN_USE, // AttributesSetting; +0x0, // Compatibilities; +LOWEST_SUPPORTED_FIRMWARE_VERSION, // LowestSupportedImageVersion; +0x,// LastAttemptVersion; +0, // LastAttemptStatus; +{0x0}, // Reserved3 +0, // HardwareInstance; + }, + // real string data + {IMAGE_ID_STRING}, + {CURRENT_FIRMWARE_VERSION_STRING}, + {PACKAGE_VERSION_STRING}, +}; + + +VOID* +ReferenceAcpiTable ( + VOID + ) +{ + // + // Reference the table being generated to prevent the optimizer from + // removing the data structure from the executable +