DynamicTablesPkg: Fix ACPI table rev field width

The VS2017 compiler reports 'warning C4244: '=': conversion from
'const UINT32' to 'UINT8', possible loss of data' when the ACPI
table revision field is being updated.

The width of the revision field in the EFI_ACPI_DESCRIPTION_HEADER
struct is 8-bit wide. Therefore, to fix the above warning make the
ACPI Table revision field usage 8-bit wide across Dynamic Tables
Framework.

Signed-off-by: Sami Mujawar <sami.mujawar@arm.com>
Reviewed-by: Alexei Fedorov <Alexei.Fedorov@arm.com>
This commit is contained in:
Sami Mujawar 2019-07-09 15:11:49 +01:00 committed by mergify[bot]
parent aa9aff2d4e
commit 90e52483bf
2 changed files with 3 additions and 3 deletions

View File

@ -287,10 +287,10 @@ typedef struct AcpiTableGenerator {
UINT32 AcpiTableSignature;
/// The ACPI table revision.
UINT32 AcpiTableRevision;
UINT8 AcpiTableRevision;
/// The minimum supported ACPI table revision.
UINT32 MinAcpiTableRevision;
UINT8 MinAcpiTableRevision;
/// The ACPI table creator ID.
UINT32 CreatorId;

View File

@ -79,7 +79,7 @@ typedef struct CmAStdObjAcpiTableInfo {
UINT32 AcpiTableSignature;
/// The ACPI table revision
UINT32 AcpiTableRevision;
UINT8 AcpiTableRevision;
/// The ACPI Table Generator ID
ACPI_TABLE_GENERATOR_ID TableGeneratorId;