From a73a40561c8c3ac92a515ca547eeb8f0cdd96d5c Mon Sep 17 00:00:00 2001 From: Pyun YongHyeon Date: Mon, 11 Aug 2008 01:45:05 +0000 Subject: [PATCH] Partial back out r180952. pci_get_vendor() and pci_get_device() don't do configuration space accessses so cahcing them makes no sense. Pointed out by: jhb, imp, des --- sys/dev/bfe/if_bfe.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/sys/dev/bfe/if_bfe.c b/sys/dev/bfe/if_bfe.c index 716d9f84583..93409929003 100644 --- a/sys/dev/bfe/if_bfe.c +++ b/sys/dev/bfe/if_bfe.c @@ -171,14 +171,12 @@ static int bfe_probe(device_t dev) { struct bfe_type *t; - uint16_t vendor, devid; t = bfe_devs; - vendor = pci_get_vendor(dev); - devid = pci_get_device(dev); while (t->bfe_name != NULL) { - if (vendor == t->bfe_vid && devid == t->bfe_did) { + if (pci_get_vendor(dev) == t->bfe_vid && + pci_get_device(dev) == t->bfe_did) { device_set_desc_copy(dev, t->bfe_name); return (BUS_PROBE_DEFAULT); }