From 42b37f537ff59eb28b85f99cc592d14724788575 Mon Sep 17 00:00:00 2001 From: Stefan Reinauer Date: Tue, 27 Jun 2017 00:27:32 +0200 Subject: clang: Allow initializer overrides In the code we do the following in a number of places to pre-initialize an array with a certain value before overwriting some of the array with other values: u8 mainboard_picr_data[FCH_INT_TABLE_SIZE] = { [0 ... FCH_INT_TABLE_SIZE-1] = 0x1F, } clang does not like that behavior unless we specify the option -Wno-initializer-overrides. Remove the check for gcc in those places, too, because 1) it would silently change array contents between compilers 2) the check isn't sufficient to determine compilation on clang vs gcc Signed-off-by: Stefan Reinauer Change-Id: I93cc121b6fec099fcdbd5fd1114c2ff7cbc291dc Reviewed-on: https://review.coreboot.org/20384 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Patrick Georgi --- src/mainboard/amd/db-ft3b-lc/mainboard.c | 4 ---- 1 file changed, 4 deletions(-) (limited to 'src/mainboard/amd') diff --git a/src/mainboard/amd/db-ft3b-lc/mainboard.c b/src/mainboard/amd/db-ft3b-lc/mainboard.c index 419a57d8fa..a99d15ce8c 100644 --- a/src/mainboard/amd/db-ft3b-lc/mainboard.c +++ b/src/mainboard/amd/db-ft3b-lc/mainboard.c @@ -43,9 +43,7 @@ * MP Tables. TODO: Make ACPI use these values too. */ static const u8 mainboard_picr_data[FCH_INT_TABLE_SIZE] = { -#if defined(__GNUC__) [0 ... FCH_INT_TABLE_SIZE-1] = 0x1F, -#endif /* INTA# - INTH# */ [0x00] = 0x0A,0x0B,0x0A,0x0B,0x0A,0x0B,0x0A,0x0B, /* Misc-nil,0,1,2, INT from Serial irq */ @@ -61,9 +59,7 @@ static const u8 mainboard_picr_data[FCH_INT_TABLE_SIZE] = { }; static const u8 mainboard_intr_data[FCH_INT_TABLE_SIZE] = { -#if defined(__GNUC__) [0 ... FCH_INT_TABLE_SIZE-1] = 0x1F, -#endif /* INTA# - INTH# */ [0x00] = 0x10,0x11,0x12,0x13,0x14,0x15,0x16,0x17, /* Misc-nil,0,1,2, INT from Serial irq */ -- cgit v1.2.3