| Message ID | 1368550322-1045-14-git-send-email-jiang.liu@huawei.com (mailing list archive) |
|---|---|
| State | New, archived |
| Delegated to: | Bjorn Helgaas |
| Headers | show |
On 05/15/2013 12:51 AM, Jiang Liu wrote: > Enhance EDAC drviers to use hotplug-safe iterators to walk PCI buses. > > Signed-off-by: Jiang Liu <jiang.liu@huawei.com> > Cc: Mauro Carvalho Chehab <mchehab@redhat.com> > Cc: Doug Thompson <dougthompson@xmission.com> > Cc: linux-edac@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > drivers/edac/i7core_edac.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/edac/i7core_edac.c b/drivers/edac/i7core_edac.c > index 0ec3e95..39d5823 100644 > --- a/drivers/edac/i7core_edac.c > +++ b/drivers/edac/i7core_edac.c > @@ -1296,7 +1296,7 @@ static unsigned i7core_pci_lastbus(void) > int last_bus = 0, bus; > struct pci_bus *b = NULL; > > - while ((b = pci_find_next_bus(b)) != NULL) { > + for_each_pci_root_bus(b) missing the "{": + for_each_pci_root_bus(b) { > bus = b->number; > edac_dbg(0, "Found bus %d\n", bus); > if (bus > last_bus) -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed 22 May 2013 12:12:19 PM CST, Gu Zheng wrote: > On 05/15/2013 12:51 AM, Jiang Liu wrote: > >> Enhance EDAC drviers to use hotplug-safe iterators to walk PCI buses. >> >> Signed-off-by: Jiang Liu <jiang.liu@huawei.com> >> Cc: Mauro Carvalho Chehab <mchehab@redhat.com> >> Cc: Doug Thompson <dougthompson@xmission.com> >> Cc: linux-edac@vger.kernel.org >> Cc: linux-kernel@vger.kernel.org >> --- >> drivers/edac/i7core_edac.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/edac/i7core_edac.c b/drivers/edac/i7core_edac.c >> index 0ec3e95..39d5823 100644 >> --- a/drivers/edac/i7core_edac.c >> +++ b/drivers/edac/i7core_edac.c >> @@ -1296,7 +1296,7 @@ static unsigned i7core_pci_lastbus(void) >> int last_bus = 0, bus; >> struct pci_bus *b = NULL; >> >> - while ((b = pci_find_next_bus(b)) != NULL) { >> + for_each_pci_root_bus(b) > > missing the "{": > + for_each_pci_root_bus(b) { Hi Zheng, Thanks for review! Will fix it in next version. Regards! Gerry > >> bus = b->number; >> edac_dbg(0, "Found bus %d\n", bus); >> if (bus > last_bus) > > -- To unsubscribe from this list: send the line "unsubscribe linux-pci" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/edac/i7core_edac.c b/drivers/edac/i7core_edac.c index 0ec3e95..39d5823 100644 --- a/drivers/edac/i7core_edac.c +++ b/drivers/edac/i7core_edac.c @@ -1296,7 +1296,7 @@ static unsigned i7core_pci_lastbus(void) int last_bus = 0, bus; struct pci_bus *b = NULL; - while ((b = pci_find_next_bus(b)) != NULL) { + for_each_pci_root_bus(b) bus = b->number; edac_dbg(0, "Found bus %d\n", bus); if (bus > last_bus)
Enhance EDAC drviers to use hotplug-safe iterators to walk PCI buses. Signed-off-by: Jiang Liu <jiang.liu@huawei.com> Cc: Mauro Carvalho Chehab <mchehab@redhat.com> Cc: Doug Thompson <dougthompson@xmission.com> Cc: linux-edac@vger.kernel.org Cc: linux-kernel@vger.kernel.org --- drivers/edac/i7core_edac.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)