summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorArthur Heymans <arthur@aheymans.xyz>2022-02-18 13:22:25 +0100
committerFelix Held <felix-coreboot@felixheld.de>2022-02-21 15:28:46 +0000
commit02967e6113b40412ee430c429f4ac5f3c41da204 (patch)
tree2abe7d4d916aea8a23a90bc43a7ae841e9f5bcc5
parentb53a55930e93f87ca4039857dd3a030c7bde9f8a (diff)
downloadcoreboot-02967e6113b40412ee430c429f4ac5f3c41da204.tar.gz
coreboot-02967e6113b40412ee430c429f4ac5f3c41da204.tar.bz2
coreboot-02967e6113b40412ee430c429f4ac5f3c41da204.zip
soc/intel/alderlake: Fix function pointer type
const void is not a proper return type for a function. It's the function pointer themselves that need to be const. This fixes building with clang. Change-Id: I99888ab9d9d80f1d6edb33b9f4a3f556f211a6e2 Signed-off-by: Arthur Heymans <arthur@aheymans.xyz> Reviewed-on: https://review.coreboot.org/c/coreboot/+/62178 Tested-by: build bot (Jenkins) <no-reply@coreboot.org> Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
-rw-r--r--src/soc/intel/alderlake/fsp_params.c2
-rw-r--r--src/soc/intel/alderlake/romstage/fsp_params.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/src/soc/intel/alderlake/fsp_params.c b/src/soc/intel/alderlake/fsp_params.c
index 90e6ab25f5d7..dac36934dfd3 100644
--- a/src/soc/intel/alderlake/fsp_params.c
+++ b/src/soc/intel/alderlake/fsp_params.c
@@ -799,7 +799,7 @@ static void soc_silicon_init_params(FSP_S_CONFIG *s_cfg,
/* Override settings per board if required. */
mainboard_update_soc_chip_config(config);
- const void (*fill_fsps_params[])(FSP_S_CONFIG *s_cfg,
+ void (*const fill_fsps_params[])(FSP_S_CONFIG *s_cfg,
const struct soc_intel_alderlake_config *config) = {
fill_fsps_lpss_params,
fill_fsps_cpu_params,
diff --git a/src/soc/intel/alderlake/romstage/fsp_params.c b/src/soc/intel/alderlake/romstage/fsp_params.c
index b92a5da41737..23e82651e3e3 100644
--- a/src/soc/intel/alderlake/romstage/fsp_params.c
+++ b/src/soc/intel/alderlake/romstage/fsp_params.c
@@ -330,7 +330,7 @@ static void fill_fspm_trace_params(FSP_M_CONFIG *m_cfg,
static void soc_memory_init_params(FSP_M_CONFIG *m_cfg,
const struct soc_intel_alderlake_config *config)
{
- const void (*fill_fspm_params[])(FSP_M_CONFIG *m_cfg,
+ void (*const fill_fspm_params[])(FSP_M_CONFIG *m_cfg,
const struct soc_intel_alderlake_config *config) = {
fill_fspm_igd_params,
fill_fspm_mrc_params,