On 4 June 2015 at 16:34, Yao, Jiewen <jiewen....@intel.com> wrote: > Hi > > Here is patch to support UEFI2.5, chapter 4, section 4.6 Properties Table > feature. > > > > MdePkg – add Properties table definition. > > MdeModulePkg – add properties table support in DXE core. > > MdeModulePkg – add PropertiesTableAttributesDxe driver to set > ACPINvs/Reserved memory type to be XP. > > > > Signed-off-by: Yao, Jiewen jiewen....@intel.com > > Reviewed-by: Zeng, Star star.z...@intel.com > > Reviewed-by: Gao, Liming liming....@intel.com >
This patch breaks the build for GCC on 64-bit ARM 13:03:43 /home/buildslave/workspace/leg-virt-tianocore-edk2-upstream/edk2/MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c: In function ‘InsertImageRecord’: 13:03:43 /home/buildslave/workspace/leg-virt-tianocore-edk2-upstream/edk2/MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c:1113:10: error: variable ‘Magic’ set but not used [-Werror=unused-but-set-variable] 13:03:43 UINT16 Magic; 13:03:43 ^ 13:03:43 /home/buildslave/workspace/leg-virt-tianocore-edk2-upstream/edk2/MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c:1110:10: error: variable ‘ImageType’ set but not used [-Werror=unused-but-set-variable] 13:03:43 UINT16 ImageType; 13:03:43 ^ and honestly, i can't be bothered to look at the code myself, considering the way it was dumped into the mailing list and the repository. Please get this fixed Regards, Ard. > > ------------------------------------------------------------------------------ > > _______________________________________________ > edk2-devel mailing list > edk2-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/edk2-devel > ------------------------------------------------------------------------------ _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel