From aa9aff2d4e1aa93ceadd881dc1749f8ae7061b3f Mon Sep 17 00:00:00 2001 From: Sami Mujawar Date: Tue, 9 Jul 2019 15:04:45 +0100 Subject: DynamicTablesPkg: Fix Boot arch flag width The ArmBootArch field of the FADT table is 16-bit wide. The VS2017 compiler reports 'warning C4244: '=': conversion from 'UINT32' to 'UINT16', possible loss of data' when assigning the CM_ARM_BOOT_ARCH_INFO.BootArchFlags value as the width of this field in CM_ARM_BOOT_ARCH_INFO is 32-bit wide. To fix this warning, update the CM_ARM_BOOT_ARCH_INFO struct to make the BootArchFlags field 16-bit wide. This also makes it compatible with the ACPI FADT specification. Signed-off-by: Sami Mujawar Reviewed-by: Alexei Fedorov --- DynamicTablesPkg/Include/ArmNameSpaceObjects.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'DynamicTablesPkg/Include') diff --git a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h index 2874938b66..2f32696031 100644 --- a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h +++ b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h @@ -68,7 +68,7 @@ typedef struct CmArmBootArchInfo { /** This is the ARM_BOOT_ARCH flags field of the FADT Table described in the ACPI Table Specification. */ - UINT32 BootArchFlags; + UINT16 BootArchFlags; } CM_ARM_BOOT_ARCH_INFO; /** A structure that describes the -- cgit v1.2.3