summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorzbao <fishbaozi@gmail.com>2012-05-29 14:59:38 +0800
committerMarc Jones <marcj303@gmail.com>2012-05-30 23:01:23 +0200
commitee8a9f6c5548729cd1e464ccbecfca7a08fcc9a2 (patch)
treeeb5924b213b5d524597c18b7d3e426e38364acf4
parent2dbfcb750fca720fcd415f8cd032eb1cf888ce46 (diff)
Initializer of a static member in union.
It is just me or does anybody have the same build error without this patch? ------ src/arch/x86/boot/acpigen.c: In function 'acpigen_write_empty_PTC': src/arch/x86/boot/acpigen.c:347:3: error: unknown field 'resv' specified in initializer src/arch/x86/boot/acpigen.c:347:3: warning: missing braces around initializer src/arch/x86/boot/acpigen.c:347:3:warning: (near initialization for 'addr.<anonymous>') ------- Anyway, I believe at least this will cause warnings. "resv" is a member of a union, not of acpi_addr_t. So it should be wrapped by a brace in the initializer. Change-Id: I72624386816c987d5bb2d3a3a64c7c58eb9af389 Signed-off-by: Zheng Bao <zheng.bao@amd.com> Signed-off-by: zbao <fishbaozi@gmail.com> Reviewed-on: http://review.coreboot.org/1056 Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org> Reviewed-by: Marc Jones <marcj303@gmail.com>
-rw-r--r--src/arch/x86/boot/acpigen.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/arch/x86/boot/acpigen.c b/src/arch/x86/boot/acpigen.c
index e7a297d357..71aa10cd17 100644
--- a/src/arch/x86/boot/acpigen.c
+++ b/src/arch/x86/boot/acpigen.c
@@ -344,7 +344,9 @@ int acpigen_write_empty_PTC(void)
.space_id = ACPI_ADDRESS_SPACE_FIXED,
.bit_width = 0,
.bit_offset = 0,
- .resv = 0,
+ {
+ .resv = 0
+ },
.addrl = 0,
.addrh = 0,
};