commit 4af4e7f06eddad71f86eda3e401967e79d3a9ddb
author Kyösti Mälkki <kyosti.malkki@gmail.com> Sun Jul 14 05:50:20 2019 +0300
committer Kyösti Mälkki <kyosti.malkki@gmail.com> Thu Jul 18 15:24:07 2019 +0000
tree 568f515b03d889441084381559068b9d0b709550
parent 28dc7dce83131cdd54cad5b338af2f4b89d8969b

soc/intel: Fix invalid use of 'static'

Just keep the variables on the stack.

Change-Id: I36b29d8fb7dac159b29609033cba450bea9adf77
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/34326
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Furquan Shaikh <furquan@google.com>

src/soc/intel/apollolake/chip.c

diff --git a/src/soc/intel/apollolake/chip.c b/src/soc/intel/apollolake/chip.c
index 3c4bf96..361e6a4 100644
--- a/src/soc/intel/apollolake/chip.c
+++ b/src/soc/intel/apollolake/chip.c
@@ -294,7 +294,7 @@
 /* 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 @@
 /* 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 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. */