ACK: [PATCH] acpi: correct ACPI BGRT table type

[ivanhu]

On 2015年06月23日 17:32, Colin King wrote:
> 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(-)Acked-by: Ivan Hu<ivan.hu at canonical.com>
>
>
>
>
> 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 */
Acked-by: Ivan Hu<ivan.hu at canonical.com>