commit d5f645c6cde230004ee5af6c62d451d1329928e9
author Kyösti Mälkki <kyosti.malkki@gmail.com> Sat Sep 28 00:20:27 2019 +0300
committer Patrick Georgi <pgeorgi@google.com> Wed Oct 02 11:21:10 2019 +0000
tree eb87509c96e5ee1fa26e87594c75a399d92402e2
parent d3d38c95b7c23c5bd455d35e1b5bef0bce7b2cc5

soc/intel: Replace config_of_path() with config_of_soc()

The previously provided device path made no difference, all
integrated PCI devices point back to the same chip_info
structure.

Change reduces the exposure of various SA_DEVFN_xx and
PCH_DEVFN_xx from (ugly) soc/pci_devs.h.

Change-Id: Ibf13645fdd3ef7fd3d5c8217bb24d7ede045c790
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: https://review.coreboot.org/c/coreboot/+/35656
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
Reviewed-by: Aaron Durbin <adurbin@chromium.org>

src/mainboard/google/fizz/mainboard.c

diff --git a/src/mainboard/google/fizz/mainboard.c b/src/mainboard/google/fizz/mainboard.c
index 6b3423b..89e6927 100644
--- a/src/mainboard/google/fizz/mainboard.c
+++ b/src/mainboard/google/fizz/mainboard.c
@@ -221,7 +221,7 @@
 
 static void mainboard_enable(struct device *dev)
 {
-	config_t *conf = config_of_path(SA_DEVFN_ROOT);
+	config_t *conf = config_of_soc();
 
 	mainboard_set_power_limits(conf);
 

src/mainboard/google/hatch/variants/akemi/variant.c

diff --git a/src/mainboard/google/hatch/variants/akemi/variant.c b/src/mainboard/google/hatch/variants/akemi/variant.c
index 0717e81..c648a52 100644
--- a/src/mainboard/google/hatch/variants/akemi/variant.c
+++ b/src/mainboard/google/hatch/variants/akemi/variant.c
@@ -23,7 +23,7 @@
 	uint32_t sku_id;
 	struct device *emmc_host;
 	struct device *ssd_host;
-	config_t *cfg = config_of_path(SA_DEVFN_ROOT);
+	config_t *cfg = config_of_soc();
 	emmc_host = pcidev_path_on_root(PCH_DEVFN_EMMC);
 	ssd_host = pcidev_path_on_root(PCH_DEVFN_SATA);
 

src/mainboard/google/hatch/variants/kindred/variant.c

diff --git a/src/mainboard/google/hatch/variants/kindred/variant.c b/src/mainboard/google/hatch/variants/kindred/variant.c
index 6cd017b..1e1d083 100644
--- a/src/mainboard/google/hatch/variants/kindred/variant.c
+++ b/src/mainboard/google/hatch/variants/kindred/variant.c
@@ -23,7 +23,7 @@
 	uint32_t sku_id;
 	struct device *emmc_host;
 	struct device *ssd_host;
-	config_t *cfg = config_of_path(SA_DEVFN_ROOT);
+	config_t *cfg = config_of_soc();
 	emmc_host = pcidev_path_on_root(PCH_DEVFN_EMMC);
 	ssd_host = pcidev_path_on_root(PCH_DEVFN_SATA);
 

src/mainboard/google/poppy/variants/atlas/mainboard.c

diff --git a/src/mainboard/google/poppy/variants/atlas/mainboard.c b/src/mainboard/google/poppy/variants/atlas/mainboard.c
index 9c4b2bc..e1538c6 100644
--- a/src/mainboard/google/poppy/variants/atlas/mainboard.c
+++ b/src/mainboard/google/poppy/variants/atlas/mainboard.c
@@ -39,7 +39,7 @@
 /* Override dev tree settings per board */
 void variant_devtree_update(void)
 {
-	config_t *cfg = config_of_path(SA_DEVFN_ROOT);
+	config_t *cfg = config_of_soc();
 
 	/* Update PL2 based on CPU */
 	cfg->tdp_pl2_override = get_pl2();

src/mainboard/google/poppy/variants/nami/mainboard.c

diff --git a/src/mainboard/google/poppy/variants/nami/mainboard.c b/src/mainboard/google/poppy/variants/nami/mainboard.c
index e48a952..e3855bd 100644
--- a/src/mainboard/google/poppy/variants/nami/mainboard.c
+++ b/src/mainboard/google/poppy/variants/nami/mainboard.c
@@ -241,7 +241,7 @@
 	uint8_t pl2_id = PL2_ID_DEFAULT;
 	struct device *spi_fpmcu = PCH_DEV_GSPI1;
 
-	config_t *cfg = config_of_path(SA_DEVFN_ROOT);
+	config_t *cfg = config_of_soc();
 
 	switch (sku_id) {
 	case SKU_0_SONA:

src/mainboard/google/poppy/variants/nautilus/mainboard.c

diff --git a/src/mainboard/google/poppy/variants/nautilus/mainboard.c b/src/mainboard/google/poppy/variants/nautilus/mainboard.c
index 9aa6b72..b78ec82 100644
--- a/src/mainboard/google/poppy/variants/nautilus/mainboard.c
+++ b/src/mainboard/google/poppy/variants/nautilus/mainboard.c
@@ -44,7 +44,7 @@
 	uint16_t abase;
 	uint32_t val32;
 
-	config_t *cfg = config_of_path(SA_DEVFN_ROOT);
+	config_t *cfg = config_of_soc();
 
 	switch (sku_id) {
 	case SKU_0_NAUTILUS:

src/mainboard/google/poppy/variants/nocturne/mainboard.c

diff --git a/src/mainboard/google/poppy/variants/nocturne/mainboard.c b/src/mainboard/google/poppy/variants/nocturne/mainboard.c
index 7b6b28b..3743cf7 100644
--- a/src/mainboard/google/poppy/variants/nocturne/mainboard.c
+++ b/src/mainboard/google/poppy/variants/nocturne/mainboard.c
@@ -40,7 +40,7 @@
 /* Override dev tree settings per board */
 void variant_devtree_update(void)
 {
-	config_t *cfg = config_of_path(SA_DEVFN_ROOT);
+	config_t *cfg = config_of_soc();
 
 	/* Update PL2 based on CPU */
 	cfg->tdp_pl2_override = get_pl2();