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 <sami.muja...@arm.com> Reviewed-by: Alexei Fedorov <alexei.fedo...@arm.com> --- Notes: v2: Splitting patch series and re-submitting DynamicTablesPkg [SAMI] patches from https://edk2.groups.io/g/devel/message/46261 DynamicTablesPkg/Include/ArmNameSpaceObjects.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/DynamicTablesPkg/Include/ArmNameSpaceObjects.h b/DynamicTablesPkg/Include/ArmNameSpaceObjects.h index 2874938b663c33edf5bdaac9f41495f75abfc945..2f326960313c1d7658029205e5cb09b89b0cdd88 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 -- 'Guid(CE165669-3EF3-493F-B85D-6190EE5B9759)' -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#56568): https://edk2.groups.io/g/devel/message/56568 Mute This Topic: https://groups.io/mt/72631545/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-