EDAC, pci_sysfs: Use for_each_pci_dev to simplify the code
authorWei Yongjun <yongjun_wei@trendmicro.com.cn>
Tue, 4 Dec 2012 05:01:42 +0000 (00:01 -0500)
committerBorislav Petkov <bp@alien8.de>
Tue, 4 Dec 2012 07:27:39 +0000 (08:27 +0100)
Use for_each_pci_dev to simplify the code.

Signed-off-by: Wei Yongjun <yongjun_wei@trendmicro.com.cn>
[Boris: cleanup comments and drop loop brackets]
Signed-off-by: Borislav Petkov <bp@alien8.de>
drivers/edac/edac_pci_sysfs.c

index e164c555a337fb3818817dbef13f1673532ac260..dc6e905ee1a5402126496bd06838302a1b26165f 100644 (file)
@@ -645,20 +645,16 @@ typedef void (*pci_parity_check_fn_t) (struct pci_dev *dev);
 
 /*
  * pci_dev parity list iterator
- *     Scan the PCI device list for one pass, looking for SERRORs
- *     Master Parity ERRORS or Parity ERRORs on primary or secondary devices
+ *
+ *     Scan the PCI device list looking for SERRORs, Master Parity ERRORS or
+ *     Parity ERRORs on primary or secondary devices.
  */
 static inline void edac_pci_dev_parity_iterator(pci_parity_check_fn_t fn)
 {
        struct pci_dev *dev = NULL;
 
-       /* request for kernel access to the next PCI device, if any,
-        * and while we are looking at it have its reference count
-        * bumped until we are done with it
-        */
-       while ((dev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, dev)) != NULL) {
+       for_each_pci_dev(dev)
                fn(dev);
-       }
 }
 
 /*