[PATCH] acpi: correct ACPI BGRT table type

Colin King colin.king at canonical.com
Tue Jun 23 09:32:48 UTC 2015


From: Colin Ian King <colin.king at canonical.com>

Fix two mistakes in BGRT, missing header and spelling mistake in the
image_offset_y field.  Update acpidump to cater for the mistake.

Signed-off-by: Colin Ian King <colin.king at canonical.com>
---
 src/acpi/acpidump/acpidump.c | 2 +-
 src/lib/include/fwts_acpi.h  | 3 ++-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/acpi/acpidump/acpidump.c b/src/acpi/acpidump/acpidump.c
index dce8ba0..c439b80 100644
--- a/src/acpi/acpidump/acpidump.c
+++ b/src/acpi/acpidump/acpidump.c
@@ -1579,7 +1579,7 @@ static void acpidump_bgrt(fwts_framework *fw, const fwts_acpi_table_info *table)
 		FIELD_UINT("Image Type", 	fwts_acpi_table_bgrt, image_type),
 		FIELD_UINT("Image Address", 	fwts_acpi_table_bgrt, image_addr),
 		FIELD_UINT("Image Offset X", 	fwts_acpi_table_bgrt, image_offset_x),
-		FIELD_UINT("Image Offset Y", 	fwts_acpi_table_bgrt, image_offset_t),
+		FIELD_UINT("Image Offset Y", 	fwts_acpi_table_bgrt, image_offset_y),
 		FIELD_END
 	};
 
diff --git a/src/lib/include/fwts_acpi.h b/src/lib/include/fwts_acpi.h
index bc2e0ec..63138b9 100644
--- a/src/lib/include/fwts_acpi.h
+++ b/src/lib/include/fwts_acpi.h
@@ -678,12 +678,13 @@ typedef struct {
 
 /* 5.2.22 Boot Graphics Resource Table (BGRT) ACPI 5.0 Spec */
 typedef struct {
+	fwts_acpi_table_header header;
 	uint16_t	version;
 	uint8_t		status;
 	uint8_t		image_type;
 	uint64_t	image_addr;
 	uint32_t	image_offset_x;
-	uint32_t	image_offset_t;
+	uint32_t	image_offset_y;
 } __attribute__ ((packed)) fwts_acpi_table_bgrt;
 
 /* 5.2.23 Firmware Performance Data Table (FPDT) ACPI 5.0 spec */
-- 
2.1.4




More information about the fwts-devel mailing list