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 02:27:39 -0500 |
commit | 3bfe5aae8edd8436d26cddfeab783492d8950821 (patch) | |
tree | 1efcdf3b2194fbaf6af30543e581277af4c94bc3 /drivers/edac/edac_pci_sysfs.c | |
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>
Diffstat (limited to 'drivers/edac/edac_pci_sysfs.c')
-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); | |||
645 | 645 | ||
646 | /* | 646 | /* |
647 | * pci_dev parity list iterator | 647 | * pci_dev parity list iterator |
648 | * Scan the PCI device list for one pass, looking for SERRORs | 648 | * |
649 | * Master Parity ERRORS or Parity ERRORs on primary or secondary devices | 649 | * Scan the PCI device list looking for SERRORs, Master Parity ERRORS or |
650 | * Parity ERRORs on primary or secondary devices. | ||
650 | */ | 651 | */ |
651 | static inline void edac_pci_dev_parity_iterator(pci_parity_check_fn_t fn) | 652 | static inline void edac_pci_dev_parity_iterator(pci_parity_check_fn_t fn) |
652 | { | 653 | { |
653 | struct pci_dev *dev = NULL; | 654 | struct pci_dev *dev = NULL; |
654 | 655 | ||
655 | /* request for kernel access to the next PCI device, if any, | 656 | for_each_pci_dev(dev) |
656 | * and while we are looking at it have its reference count | ||
657 | * bumped until we are done with it | ||
658 | */ | ||
659 | while ((dev = pci_get_device(PCI_ANY_ID, PCI_ANY_ID, dev)) != NULL) { | ||
660 | fn(dev); | 657 | fn(dev); |
661 | } | ||
662 | } | 658 | } |
663 | 659 | ||
664 | /* | 660 | /* |