commit 6f37017c57db271beef5dd8f9f9d1229a1f99219
author Kyösti Mälkki <kyosti.malkki@gmail.com> Thu Mar 19 15:26:52 2015 +0200
committer Kyösti Mälkki <kyosti.malkki@gmail.com> Thu Jun 04 11:20:02 2015 +0200
tree e4695a6d74e08de2846916f0421bcdd0f9ac30a5
parent cd37a2ba4159c62dfbff1bb7397e9e2d4da0c694

devicetree: Rename unused parameter max in domain_scan_bus()

For the PCI root node, input parameter max==0 and output value
max is not relevant for operation.

Change-Id: I23adab24aa957c4d51d703098a9a40ed660b4e6c
Signed-off-by: Kyösti Mälkki <kyosti.malkki@gmail.com>
Reviewed-on: http://review.coreboot.org/8855
Tested-by: build bot (Jenkins)
Reviewed-by: Timothy Pearson <tpearson@raptorengineeringinc.com>

src/device/pci_device.c

diff --git a/src/device/pci_device.c b/src/device/pci_device.c
index 4651258..e43d4e5 100644
--- a/src/device/pci_device.c
+++ b/src/device/pci_device.c
@@ -1271,10 +1271,11 @@
  * @param max The highest bus number assigned up to now.
  * @return The maximum bus number found, after scanning all subordinate busses.
  */
-unsigned int pci_domain_scan_bus(device_t dev, unsigned int max)
+unsigned int pci_domain_scan_bus(device_t dev, unsigned int unused)
 {
-	max = pci_scan_bus(dev->link_list, PCI_DEVFN(0, 0), 0xff, max);
-	return max;
+	struct bus *link = dev->link_list;
+	link->subordinate = pci_scan_bus(link, PCI_DEVFN(0, 0), 0xff, link->secondary);
+	return unused;
 }
 
 /**

src/northbridge/amd/agesa/family10/northbridge.c

diff --git a/src/northbridge/amd/agesa/family10/northbridge.c b/src/northbridge/amd/agesa/family10/northbridge.c
index c9e4200..bb2f1d8 100644
--- a/src/northbridge/amd/agesa/family10/northbridge.c
+++ b/src/northbridge/amd/agesa/family10/northbridge.c
@@ -905,7 +905,7 @@
 	}
 }
 
-static u32 amdfam10_domain_scan_bus(device_t dev, u32 max)
+static u32 amdfam10_domain_scan_bus(device_t dev, u32 unused)
 {
 	u32 reg;
 	int i;
@@ -942,7 +942,7 @@
 			pci_write_config32(f0_dev, HT_TRANSACTION_CONTROL, httc);
 		}
 	}
-	return max;
+	return unused;
 }
 
 

src/northbridge/amd/agesa/family15/northbridge.c

diff --git a/src/northbridge/amd/agesa/family15/northbridge.c b/src/northbridge/amd/agesa/family15/northbridge.c
index 5a08f23..093d0c0 100644
--- a/src/northbridge/amd/agesa/family15/northbridge.c
+++ b/src/northbridge/amd/agesa/family15/northbridge.c
@@ -951,11 +951,11 @@
 }
 
 /* all family15's pci devices are under 0x18.0, so we search from dev 0x18 fun 0 */
-static unsigned int f15_pci_domain_scan_bus(device_t dev, unsigned int max)
+static unsigned int f15_pci_domain_scan_bus(device_t dev, unsigned int unused)
 {
 	struct bus *link = dev->link_list;
 	link->subordinate = pci_scan_bus(link, PCI_DEVFN(0x18, 0), 0xff, link->secondary);
-	return link->subordinate;
+	return unused;
 }
 
 static struct device_operations pci_domain_ops = {

src/northbridge/amd/amdk8/northbridge.c

diff --git a/src/northbridge/amd/amdk8/northbridge.c b/src/northbridge/amd/amdk8/northbridge.c
index c0174de..22bc51c 100644
--- a/src/northbridge/amd/amdk8/northbridge.c
+++ b/src/northbridge/amd/amdk8/northbridge.c
@@ -1095,7 +1095,7 @@
 
 }
 
-static u32 amdk8_domain_scan_bus(device_t dev, u32 max)
+static u32 amdk8_domain_scan_bus(device_t dev, u32 unused)
 {
 	u32 reg;
 	int i;
@@ -1131,7 +1131,7 @@
 			pci_write_config32(f0_dev, HT_TRANSACTION_CONTROL, httc);
 		}
 	}
-	return max;
+	return unused;
 }
 
 static struct device_operations pci_domain_ops = {