Skip to contentSkip to conversation

list_for_each: arch-alpha-kernel-pci.c

PATCH In review [email protected] 2 participants 2 messages Sep 1, 2004 – Sep 9, 2004 Alpha port

lore
Share & tools
lore
#0 lore raw

PATCH 1/12 list_for_each: arch-alpha-kernel-pci.c

Hi.

Change for loops with list_for_each().

---

 linux-2.6.9-rc1-bk7-max/arch/alpha/kernel/pci.c |   16 +++++-----------
 1 files changed, 5 insertions(+), 11 deletions(-)
arch/alpha/kernel/pci.c renamed from arch/alpha/kernel/pci.c~list-for-each-arch_alpha_kernel_pci +5 -11
diff -puN arch/alpha/kernel/pci.c~list-for-each-arch_alpha_kernel_pci arch/alpha/kernel/pci.c
@@ -280,7 +280,6 @@ pcibios_fixup_bus(struct pci_bus *bus)
 	/* Propagate hose info into the subordinate devices.  */
 
 	struct pci_controller *hose = bus->sysdata;
-	struct list_head *ln;
 	struct pci_dev *dev = bus->self;
 
 	if (!dev) {
@@ -304,9 +303,7 @@ pcibios_fixup_bus(struct pci_bus *bus)
  		pcibios_fixup_device_resources(dev, bus);
 	} 
 
-	for (ln = bus->devices.next; ln != &bus->devices; ln = ln->next) {
-		struct pci_dev *dev = pci_dev_b(ln);
-
+	list_for_each_entry(dev, &bus->devices, bus_list) {
 		pdev_save_srm_config(dev);
 		if ((dev->class >> 8) != PCI_CLASS_BRIDGE_PCI)
 			pcibios_fixup_device_resources(dev, bus);
@@ -403,11 +400,10 @@ pcibios_set_master(struct pci_dev *dev)
 static void __init
 pcibios_claim_one_bus(struct pci_bus *b)
 {
-	struct list_head *ld;
+	struct pci_dev *dev;
 	struct pci_bus *child_bus;
 
-	for (ld = b->devices.next; ld != &b->devices; ld = ld->next) {
-		struct pci_dev *dev = pci_dev_b(ld);
+	list_for_each_entry(dev, &b->devices, bus_list) {
 		int i;
 
 		for (i = 0; i < PCI_NUM_RESOURCES; i++) {
@@ -426,12 +422,10 @@ pcibios_claim_one_bus(struct pci_bus *b)
 static void __init
 pcibios_claim_console_setup(void)
 {
-	struct list_head *lb;
+	struct pci_bus *b;
 
-	for(lb = pci_root_buses.next; lb != &pci_root_buses; lb = lb->next) {
-		struct pci_bus *b = pci_bus_b(lb);
+	list_for_each_entry(b, &pci_root_buses, node)
 		pcibios_claim_one_bus(b);
-	}
 }
 
 void __init

_

#1 lore raw
Quoted 5 lines · On Wed, Sep 01, 2004 at 10:15:06PM +0200, [email protected] wrote:

Hi.

Change for loops with list_for_each().

Thanks, I've applied all of these (with the exception of the one that Dave Miller already did.)

greg k-h