diff options
| author | Wei Yongjun <yongjun_wei@trendmicro.com.cn> | 2012-12-04 00:01:42 -0500 |
|---|---|---|
| committer | Borislav Petkov <bp@alien8.de> | 2012-12-04 08:27:39 +0100 |
| commit | 3bfe5aae8edd8436d26cddfeab783492d8950821 (patch) | |
| tree | 1efcdf3b2194fbaf6af30543e581277af4c94bc3 | |
| parent | 2d56b109e3a50cf316e60f07541bbeb1d8fe251a (diff) | |
EDAC, pci_sysfs: Use for_each_pci_dev to simplify the code
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>
| -rw-r--r-- | drivers/edac/edac_pci_sysfs.c | 12 |
1 files changed, 4 insertions, 8 deletions
diff --git a/drivers/edac/edac_pci_sysfs.c b/drivers/edac/edac_pci_sysfs.c index e164c555a337..dc6e905ee1a5 100644 --- a/drivers/edac/edac_pci_sysfs.c +++ b/drivers/edac/edac_pci_sysfs.c @@ -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); - } } /* |