soc/intel: Drop `dev` parameter from soc_get_gen_io_dec_range()

This change drops the parameter `struct device *dev` from the function
`soc_get_gen_io_dec_range()`. This function uses the parameter dev to
get a pointer to config structure for extracting the decode ranges
configured by mainboard in device tree. However, there is no separate
chip driver for the LPC device which means that the SoC code can use
`config_of_soc()` to get to SoC chip config instead of using the LPC
device.

This change is being done in preparation to clean up the device
tree/chip config access in early stages that allows for optimizing
the inclusion of device tree elements in the early stages.

Change-Id: I3ea53ddc771f592dd0ea5e5e809be2d2eff7f16d
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/49209
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Angel Pons <th3fanbus@gmail.com>
Reviewed-by: Tim Wawrzynczak <twawrzynczak@chromium.org>
Reviewed-by: Karthik Ramasubramanian <kramasub@google.com>

10 files changed, 18 insertions, 25 deletions

diff --git a/src/soc/intel/alderlake/espi.c b/src/soc/intel/alderlake/espi.c
index c9090301673a..b2377274e539 100644
--- a/src/soc/intel/alderlake/espi.c
+++ b/src/soc/intel/alderlake/espi.c
@@ -35,9 +35,9 @@ const struct lpc_mmio_range *soc_get_fixed_mmio_ranges()
 	return lpc_fixed_mmio_ranges;
 }
 
-void soc_get_gen_io_dec_range(const struct device *dev, uint32_t *gen_io_dec)
+void soc_get_gen_io_dec_range(uint32_t *gen_io_dec)
 {
-	const config_t *config = config_of(dev);
+	const config_t *config = config_of_soc();
 
 	gen_io_dec[0] = config->gen1_dec;
 	gen_io_dec[1] = config->gen2_dec;
diff --git a/src/soc/intel/cannonlake/lpc.c b/src/soc/intel/cannonlake/lpc.c
index b21c117df06a..c8e35c4d5546 100644
--- a/src/soc/intel/cannonlake/lpc.c
+++ b/src/soc/intel/cannonlake/lpc.c
@@ -33,9 +33,9 @@ const struct lpc_mmio_range *soc_get_fixed_mmio_ranges()
 	return cnl_lpc_fixed_mmio_ranges;
 }
 
-void soc_get_gen_io_dec_range(const struct device *dev, uint32_t *gen_io_dec)
+void soc_get_gen_io_dec_range(uint32_t *gen_io_dec)
 {
-	const config_t *config = config_of(dev);
+	const config_t *config = config_of_soc();
 
 	gen_io_dec[0] = config->gen1_dec;
 	gen_io_dec[1] = config->gen2_dec;
diff --git a/src/soc/intel/common/block/include/intelblocks/lpc_lib.h b/src/soc/intel/common/block/include/intelblocks/lpc_lib.h
index 2fdcdef3e6a3..82cba8ec7f5d 100644
--- a/src/soc/intel/common/block/include/intelblocks/lpc_lib.h
+++ b/src/soc/intel/common/block/include/intelblocks/lpc_lib.h
@@ -98,8 +98,7 @@ void lpc_io_setup_comm_a_b(void);
 /* Enable PCH LPC by setting up generic decode range registers. */
 void pch_enable_lpc(void);
 /* Get SoC's generic IO decoder range register settings. */
-void soc_get_gen_io_dec_range(const struct device *dev,
-			uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]);
+void soc_get_gen_io_dec_range(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]);
 /* Mirror generic IO decoder range register settings into DMI PCR. */
 void soc_setup_dmi_pcr_io_dec(uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES]);
 /* Add resource into LPC PCI device space */
diff --git a/src/soc/intel/common/block/lpc/lpc_lib.c b/src/soc/intel/common/block/lpc/lpc_lib.c
index 67dd6614603b..d050ee36e5c5 100644
--- a/src/soc/intel/common/block/lpc/lpc_lib.c
+++ b/src/soc/intel/common/block/lpc/lpc_lib.c
@@ -270,15 +270,9 @@ static void lpc_set_gen_decode_range(
 
 void pch_enable_lpc(void)
 {
-	/* Lookup device tree in romstage */
-	const struct device *dev;
 	uint32_t gen_io_dec[LPC_NUM_GENERIC_IO_RANGES];
 
-	dev = pcidev_on_root(PCH_DEV_SLOT_LPC, 0);
-	if (!dev)
-		return;
-
-	soc_get_gen_io_dec_range(dev, gen_io_dec);
+	soc_get_gen_io_dec_range(gen_io_dec);
 	lpc_set_gen_decode_range(gen_io_dec);
 	soc_setup_dmi_pcr_io_dec(gen_io_dec);
 	if (ENV_PAYLOAD_LOADER)
diff --git a/src/soc/intel/elkhartlake/espi.c b/src/soc/intel/elkhartlake/espi.c
index a531fccbb081..8f82755e551b 100644
--- a/src/soc/intel/elkhartlake/espi.c
+++ b/src/soc/intel/elkhartlake/espi.c
@@ -31,9 +31,9 @@ const struct lpc_mmio_range *soc_get_fixed_mmio_ranges()
 	return ehl_lpc_fixed_mmio_ranges;
 }
 
-void soc_get_gen_io_dec_range(const struct device *dev, uint32_t *gen_io_dec)
+void soc_get_gen_io_dec_range(uint32_t *gen_io_dec)
 {
-	const config_t *config = config_of(dev);
+	const config_t *config = config_of_soc();
 
 	gen_io_dec[0] = config->gen1_dec;
 	gen_io_dec[1] = config->gen2_dec;
diff --git a/src/soc/intel/icelake/espi.c b/src/soc/intel/icelake/espi.c
index 7f9ff2fee2d4..4107d7e27458 100644
--- a/src/soc/intel/icelake/espi.c
+++ b/src/soc/intel/icelake/espi.c
@@ -32,9 +32,9 @@ const struct lpc_mmio_range *soc_get_fixed_mmio_ranges()
 	return icl_lpc_fixed_mmio_ranges;
 }
 
-void soc_get_gen_io_dec_range(const struct device *dev, uint32_t *gen_io_dec)
+void soc_get_gen_io_dec_range(uint32_t *gen_io_dec)
 {
-	const config_t *config = config_of(dev);
+	const config_t *config = config_of_soc();
 
 	gen_io_dec[0] = config->gen1_dec;
 	gen_io_dec[1] = config->gen2_dec;
diff --git a/src/soc/intel/jasperlake/espi.c b/src/soc/intel/jasperlake/espi.c
index 0977857684e8..11299a9552f7 100644
--- a/src/soc/intel/jasperlake/espi.c
+++ b/src/soc/intel/jasperlake/espi.c
@@ -32,9 +32,9 @@ const struct lpc_mmio_range *soc_get_fixed_mmio_ranges()
 	return jsl_lpc_fixed_mmio_ranges;
 }
 
-void soc_get_gen_io_dec_range(const struct device *dev, uint32_t *gen_io_dec)
+void soc_get_gen_io_dec_range(uint32_t *gen_io_dec)
 {
-	const config_t *config = config_of(dev);
+	const config_t *config = config_of_soc();
 
 	gen_io_dec[0] = config->gen1_dec;
 	gen_io_dec[1] = config->gen2_dec;
diff --git a/src/soc/intel/skylake/lpc.c b/src/soc/intel/skylake/lpc.c
index f9c663980e30..b83fadd22313 100644
--- a/src/soc/intel/skylake/lpc.c
+++ b/src/soc/intel/skylake/lpc.c
@@ -29,9 +29,9 @@ const struct lpc_mmio_range *soc_get_fixed_mmio_ranges(void)
 	return skl_lpc_fixed_mmio_ranges;
 }
 
-void soc_get_gen_io_dec_range(const struct device *dev, uint32_t *gen_io_dec)
+void soc_get_gen_io_dec_range(uint32_t *gen_io_dec)
 {
-	const config_t *config = config_of(dev);
+	const config_t *config = config_of_soc();
 
 	gen_io_dec[0] = config->gen1_dec;
 	gen_io_dec[1] = config->gen2_dec;
diff --git a/src/soc/intel/tigerlake/espi.c b/src/soc/intel/tigerlake/espi.c
index acd3b1224409..8602b0dc59f2 100644
--- a/src/soc/intel/tigerlake/espi.c
+++ b/src/soc/intel/tigerlake/espi.c
@@ -38,9 +38,9 @@ const struct lpc_mmio_range *soc_get_fixed_mmio_ranges()
 	return tgl_lpc_fixed_mmio_ranges;
 }
 
-void soc_get_gen_io_dec_range(const struct device *dev, uint32_t *gen_io_dec)
+void soc_get_gen_io_dec_range(uint32_t *gen_io_dec)
 {
-	const config_t *config = config_of(dev);
+	const config_t *config = config_of_soc();
 
 	gen_io_dec[0] = config->gen1_dec;
 	gen_io_dec[1] = config->gen2_dec;
diff --git a/src/soc/intel/xeon_sp/lpc.c b/src/soc/intel/xeon_sp/lpc.c
index 3169545bd4f0..dee99b70ac0e 100644
--- a/src/soc/intel/xeon_sp/lpc.c
+++ b/src/soc/intel/xeon_sp/lpc.c
@@ -18,9 +18,9 @@ const struct lpc_mmio_range *soc_get_fixed_mmio_ranges(void)
 	return xeon_lpc_fixed_mmio_ranges;
 }
 
-void soc_get_gen_io_dec_range(const struct device *dev, uint32_t *gen_io_dec)
+void soc_get_gen_io_dec_range(uint32_t *gen_io_dec)
 {
-	const config_t *config = config_of(dev);
+	const config_t *config = config_of_soc();
 
 	gen_io_dec[0] = config->gen1_dec;
 	gen_io_dec[1] = config->gen2_dec;