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

Reply via email to