aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBen Gardner <gardner.ben@gmail.com>2007-05-11 01:58:58 -0400
committerGreg Kroah-Hartman <gregkh@suse.de>2007-05-31 19:56:37 -0400
commit3c92c57af9a24a08b8d2f76650b1209239914fcd (patch)
tree203717bccf072e32dc62db493b0fee6aa4201a44
parente3008dedff4bdc96a5f67224cd3d8d12237082a0 (diff)
PCI: Fix pci_find_present
pci_find_present() is only matching the last item in the list of ids. The break after the match is found only escapes the for loop, not the while loop, so found gets reset to NULL on the next pass. Signed-off-by: Ben Gardner <gardner.ben@gmail.com> Cc: Alan Cox <alan@lxorguk.ukuu.org.uk> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r--drivers/pci/search.c3
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/pci/search.c b/drivers/pci/search.c
index b137a27472c7..c13232435dc0 100644
--- a/drivers/pci/search.c
+++ b/drivers/pci/search.c
@@ -403,10 +403,11 @@ const struct pci_device_id *pci_find_present(const struct pci_device_id *ids)
403 while (ids->vendor || ids->subvendor || ids->class_mask) { 403 while (ids->vendor || ids->subvendor || ids->class_mask) {
404 list_for_each_entry(dev, &pci_devices, global_list) { 404 list_for_each_entry(dev, &pci_devices, global_list) {
405 if ((found = pci_match_one_device(ids, dev)) != NULL) 405 if ((found = pci_match_one_device(ids, dev)) != NULL)
406 break; 406 goto exit;
407 } 407 }
408 ids++; 408 ids++;
409 } 409 }
410exit:
410 up_read(&pci_bus_sem); 411 up_read(&pci_bus_sem);
411 return found; 412 return found;
412} 413}