From 4af4e7f06eddad71f86eda3e401967e79d3a9ddb Mon Sep 17 00:00:00 2001 From: Kyösti Mälkki Date: Sun, 14 Jul 2019 05:50:20 +0300 Subject: soc/intel: Fix invalid use of 'static' MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Just keep the variables on the stack. Change-Id: I36b29d8fb7dac159b29609033cba450bea9adf77 Signed-off-by: Kyösti Mälkki Reviewed-on: https://review.coreboot.org/c/coreboot/+/34326 Tested-by: build bot (Jenkins) Reviewed-by: Furquan Shaikh --- src/mainboard/purism/librem_skl/hda_verb.c | 2 +- src/soc/intel/apollolake/chip.c | 6 +++--- src/soc/intel/skylake/chip_fsp20.c | 2 +- src/soc/intel/skylake/thermal.c | 2 +- 4 files changed, 6 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/mainboard/purism/librem_skl/hda_verb.c b/src/mainboard/purism/librem_skl/hda_verb.c index c0de990d80..206af8db7e 100644 --- a/src/mainboard/purism/librem_skl/hda_verb.c +++ b/src/mainboard/purism/librem_skl/hda_verb.c @@ -40,7 +40,7 @@ static void codecs_init(u8 *base, u32 codec_mask) static void mb_hda_codec_init(void *unused) { - static struct soc_intel_skylake_config *config; + struct soc_intel_skylake_config *config; u8 *base; struct resource *res; u32 codec_mask; diff --git a/src/soc/intel/apollolake/chip.c b/src/soc/intel/apollolake/chip.c index 3c4bf968e9..361e6a417e 100644 --- a/src/soc/intel/apollolake/chip.c +++ b/src/soc/intel/apollolake/chip.c @@ -294,7 +294,7 @@ static void pcie_override_devicetree_after_silicon_init(void) /* Configure package power limits */ static void set_power_limits(void) { - static struct soc_intel_apollolake_config *cfg; + struct soc_intel_apollolake_config *cfg; msr_t rapl_msr_reg, limit; uint32_t power_unit; uint32_t tdp, min_power, max_power; @@ -361,7 +361,7 @@ static void set_power_limits(void) /* Overwrites the SCI IRQ if another IRQ number is given by device tree. */ static void set_sci_irq(void) { - static struct soc_intel_apollolake_config *cfg; + struct soc_intel_apollolake_config *cfg; uint32_t scis; cfg = config_of_path(SA_DEVFN_ROOT); @@ -666,7 +666,7 @@ void __weak mainboard_devtree_update(struct device *dev) void platform_fsp_silicon_init_params_cb(FSPS_UPD *silupd) { FSP_S_CONFIG *silconfig = &silupd->FspsConfig; - static struct soc_intel_apollolake_config *cfg; + struct soc_intel_apollolake_config *cfg; struct device *dev; /* Load VBT before devicetree-specific config. */ diff --git a/src/soc/intel/skylake/chip_fsp20.c b/src/soc/intel/skylake/chip_fsp20.c index 5b61df3c68..064f71e2f2 100644 --- a/src/soc/intel/skylake/chip_fsp20.c +++ b/src/soc/intel/skylake/chip_fsp20.c @@ -232,7 +232,7 @@ void platform_fsp_silicon_init_params_cb(FSPS_UPD *supd) { FSP_S_CONFIG *params = &supd->FspsConfig; FSP_S_TEST_CONFIG *tconfig = &supd->FspsTestConfig; - static struct soc_intel_skylake_config *config; + struct soc_intel_skylake_config *config; struct device *dev; uintptr_t vbt_data = (uintptr_t)vbt_get(); int i; diff --git a/src/soc/intel/skylake/thermal.c b/src/soc/intel/skylake/thermal.c index 97cd1b7779..936543c7bf 100644 --- a/src/soc/intel/skylake/thermal.c +++ b/src/soc/intel/skylake/thermal.c @@ -62,7 +62,7 @@ static void pch_thermal_set_bar(struct device *dev, uintptr_t tempbar) /* PCH Low Temp Threshold (LTT) */ static uint16_t pch_get_ltt_value(struct device *dev) { - static struct soc_intel_skylake_config *config; + struct soc_intel_skylake_config *config; uint16_t ltt_value; uint16_t trip_temp = DEFAULT_TRIP_TEMP; -- cgit v1.2.3