Patrick Georgi has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/54094 )
Change subject: src: Match array format in function declarations and definitions ......................................................................
src: Match array format in function declarations and definitions
gcc 11.1 complains when we're passing a type* into a function that was declared to get a type[], even if the ABI has identical parameter passing for both.
To prepare for newer compilers, adapt to this added constraint.
Change-Id: I5a1b3824a85a178431177620c4c0d5fddc993b4f Signed-off-by: Patrick Georgi pgeorgi@google.com --- M src/mainboard/lenovo/t400/romstage.c M src/mainboard/lenovo/x200/romstage.c M src/mainboard/roda/rk9/romstage.c M src/soc/intel/alderlake/espi.c M src/soc/intel/cannonlake/lpc.c M src/soc/intel/elkhartlake/espi.c M src/soc/intel/icelake/espi.c M src/soc/intel/jasperlake/espi.c M src/soc/intel/skylake/lpc.c M src/soc/intel/tigerlake/espi.c M src/soc/intel/xeon_sp/lpc.c M src/vendorcode/mediatek/mt8192/dramc/dramc_top.c 12 files changed, 12 insertions(+), 12 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/94/54094/1
diff --git a/src/mainboard/lenovo/t400/romstage.c b/src/mainboard/lenovo/t400/romstage.c index b4766ed..aa3462a 100644 --- a/src/mainboard/lenovo/t400/romstage.c +++ b/src/mainboard/lenovo/t400/romstage.c @@ -15,7 +15,7 @@ sysinfo->enable_peg = peg; }
-void get_mb_spd_addrmap(u8 *spd_addrmap) +void get_mb_spd_addrmap(u8 spd_addrmap[4]) { spd_addrmap[0] = 0x50; spd_addrmap[2] = 0x51; diff --git a/src/mainboard/lenovo/x200/romstage.c b/src/mainboard/lenovo/x200/romstage.c index 46cedfb..6764644 100644 --- a/src/mainboard/lenovo/x200/romstage.c +++ b/src/mainboard/lenovo/x200/romstage.c @@ -3,7 +3,7 @@ #include <southbridge/intel/common/gpio.h> #include <northbridge/intel/gm45/gm45.h>
-void get_mb_spd_addrmap(u8 *spd_addrmap) +void get_mb_spd_addrmap(u8 spd_addrmap[4]) { spd_addrmap[0] = 0x50; spd_addrmap[2] = 0x51; diff --git a/src/mainboard/roda/rk9/romstage.c b/src/mainboard/roda/rk9/romstage.c index be8ba5d..dabef34 100644 --- a/src/mainboard/roda/rk9/romstage.c +++ b/src/mainboard/roda/rk9/romstage.c @@ -2,7 +2,7 @@
#include <northbridge/intel/gm45/gm45.h>
-void get_mb_spd_addrmap(u8 *spd_addrmap) +void get_mb_spd_addrmap(u8 spd_addrmap[4]) { spd_addrmap[0] = 0x50; spd_addrmap[2] = 0x52; diff --git a/src/soc/intel/alderlake/espi.c b/src/soc/intel/alderlake/espi.c index feec465..dd0edcd 100644 --- a/src/soc/intel/alderlake/espi.c +++ b/src/soc/intel/alderlake/espi.c @@ -20,7 +20,7 @@ #include <soc/pcr_ids.h> #include <soc/soc_chip.h>
-void soc_get_gen_io_dec_range(uint32_t *gen_io_dec) +void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]) { const config_t *config = config_of_soc();
diff --git a/src/soc/intel/cannonlake/lpc.c b/src/soc/intel/cannonlake/lpc.c index 20704e5..0e63e0d 100644 --- a/src/soc/intel/cannonlake/lpc.c +++ b/src/soc/intel/cannonlake/lpc.c @@ -17,7 +17,7 @@
#include "chip.h"
-void soc_get_gen_io_dec_range(uint32_t *gen_io_dec) +void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]) { const config_t *config = config_of_soc();
diff --git a/src/soc/intel/elkhartlake/espi.c b/src/soc/intel/elkhartlake/espi.c index 1737a47..46646d8 100644 --- a/src/soc/intel/elkhartlake/espi.c +++ b/src/soc/intel/elkhartlake/espi.c @@ -16,7 +16,7 @@ #include <soc/pcr_ids.h> #include <soc/soc_chip.h>
-void soc_get_gen_io_dec_range(uint32_t *gen_io_dec) +void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]) { const config_t *config = config_of_soc();
diff --git a/src/soc/intel/icelake/espi.c b/src/soc/intel/icelake/espi.c index 489fe34..d634cf8 100644 --- a/src/soc/intel/icelake/espi.c +++ b/src/soc/intel/icelake/espi.c @@ -16,7 +16,7 @@ #include <soc/pcr_ids.h> #include <soc/soc_chip.h>
-void soc_get_gen_io_dec_range(uint32_t *gen_io_dec) +void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]) { const config_t *config = config_of_soc();
diff --git a/src/soc/intel/jasperlake/espi.c b/src/soc/intel/jasperlake/espi.c index c3b50de..1d1f94e 100644 --- a/src/soc/intel/jasperlake/espi.c +++ b/src/soc/intel/jasperlake/espi.c @@ -16,7 +16,7 @@ #include <soc/pcr_ids.h> #include <soc/soc_chip.h>
-void soc_get_gen_io_dec_range(uint32_t *gen_io_dec) +void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]) { const config_t *config = config_of_soc();
diff --git a/src/soc/intel/skylake/lpc.c b/src/soc/intel/skylake/lpc.c index 5abae76..5d38bb8 100644 --- a/src/soc/intel/skylake/lpc.c +++ b/src/soc/intel/skylake/lpc.c @@ -14,7 +14,7 @@
#include "chip.h"
-void soc_get_gen_io_dec_range(uint32_t *gen_io_dec) +void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]) { const config_t *config = config_of_soc();
diff --git a/src/soc/intel/tigerlake/espi.c b/src/soc/intel/tigerlake/espi.c index 8386cd9..4278676 100644 --- a/src/soc/intel/tigerlake/espi.c +++ b/src/soc/intel/tigerlake/espi.c @@ -22,7 +22,7 @@ #include <soc/pcr_ids.h> #include <soc/soc_chip.h>
-void soc_get_gen_io_dec_range(uint32_t *gen_io_dec) +void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]) { const config_t *config = config_of_soc();
diff --git a/src/soc/intel/xeon_sp/lpc.c b/src/soc/intel/xeon_sp/lpc.c index f0cb6db..dad0a49 100644 --- a/src/soc/intel/xeon_sp/lpc.c +++ b/src/soc/intel/xeon_sp/lpc.c @@ -8,7 +8,7 @@
#include <chip.h>
-void soc_get_gen_io_dec_range(uint32_t *gen_io_dec) +void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]) { const config_t *config = config_of_soc();
diff --git a/src/vendorcode/mediatek/mt8192/dramc/dramc_top.c b/src/vendorcode/mediatek/mt8192/dramc/dramc_top.c index 8af6a36..04fd62a 100644 --- a/src/vendorcode/mediatek/mt8192/dramc/dramc_top.c +++ b/src/vendorcode/mediatek/mt8192/dramc/dramc_top.c @@ -475,7 +475,7 @@ #if (FOR_DV_SIMULATION_USED==0) #if !__FLASH_TOOL_DA__ && !__ETT__
-void get_dram_rank_size(u64 dram_rank_size[DRAMC_MAX_RK]) +void get_dram_rank_size(u64 dram_rank_size[]) { #ifdef COMBO_MCP int index, rank_nr, i;