[coreboot-gerrit] Change in coreboot[master]: nb/intel/x4x: Factor out hiding PCI devs in pure fn

[Arthur Heymans (Code Review)]

Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/37291 )

Change subject: nb/intel/x4x: Factor out hiding PCI devs in pure fn
......................................................................

nb/intel/x4x: Factor out hiding PCI devs in pure fn

This increases readability.

Also change the update expression. '--variable' does not make much
sense there.

Change-Id: I64db2460115f5fb35ca197b83440f8ee47470761
Signed-off-by: Arthur Heymans 
---
M src/northbridge/intel/x4x/northbridge.c
1 file changed, 21 insertions(+), 20 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/91/37291/1

diff --git a/src/northbridge/intel/x4x/northbridge.c b/src/northbridge/intel/x4x/northbridge.c
index 39f24d3..efdaeef 100644
--- a/src/northbridge/intel/x4x/northbridge.c
+++ b/src/northbridge/intel/x4x/northbridge.c
@@ -206,44 +206,45 @@
    	dev->ops = &cpu_bus_ops;
 }

+static void hide_pci_fn(const int dev_bit_base, const struct device *dev)
+{
+	if (!dev || dev->enabled)
+		return;
+	const unsigned int fn = PCI_FUNC(dev->path.pci.devfn);
+	const struct device *const d0f0 = pcidev_on_root(0, 0);
+	pci_update_config32(d0f0, D0F0_DEVEN, ~(1 << (dev_bit_base + fn)), 0);
+}
+
+static void hide_pci_dev(const int dev, int functions, const int dev_bit_base)
+{
+	for (; functions >= 0; functions--)
+		hide_pci_fn(dev_bit_base, pcidev_on_root(dev, functions));
+}
+
 static void x4x_init(void *const chip_info)
 {
-	int dev, fn, bit_base;
-
+	int dev;
    struct device *const d0f0 = pcidev_on_root(0x0, 0);

/* Hide internal functions based on devicetree info. */
-	for (dev = 6; dev > 0; --dev) {
+	for (dev = 6; dev > 0; dev--) {
    	switch (dev) {
    	case 6: /* PEG1: only on P45 */
-			fn = 0;
-			bit_base = 13;
+			hide_pci_dev(dev, 0, 13);
    		break;
    	case 3: /* ME */
-			fn = 3;
-			bit_base = 6;
+			hide_pci_dev(dev, 3, 6);
    		break;
    	case 2: /* IGD */
-			fn = 1;
-			bit_base = 3;
+			hide_pci_dev(dev, 1, 3);
    		break;
    	case 1: /* PEG0 */
-			fn = 0;
-			bit_base = 1;
+			hide_pci_dev(dev, 0, 1);
    		break;
    	case 4: /* Nothing to do */
    	case 5:
    		continue;
    	}
-		for (; fn >= 0; --fn) {
-			const struct device *const d =
-				pcidev_on_root(dev, fn);
-			if (!d || d->enabled)
-				continue;
-			const u32 deven = pci_read_config32(d0f0, D0F0_DEVEN);
-			pci_write_config32(d0f0, D0F0_DEVEN,
-					   deven & ~(1 << (bit_base + fn)));
-		}
    }

const u32 deven = pci_read_config32(d0f0, D0F0_DEVEN);