netdev
[Top] [All Lists]

Re: [patch 11/26] janitor: net/tg3: pci_find_device to pci_dev_present

To: domen@xxxxxxxxxxxx
Subject: Re: [patch 11/26] janitor: net/tg3: pci_find_device to pci_dev_present
From: Jeff Garzik <jgarzik@xxxxxxxxx>
Date: Wed, 09 Mar 2005 15:25:42 -0500
Cc: netdev@xxxxxxxxxxx, sfeldma@xxxxxxxxx, janitor@xxxxxxxxxxxxxx, "David S. Miller" <davem@xxxxxxxxxxxxx>
In-reply-to: <20050306103312.3155B1E46E@xxxxxxxxxxxxxxxxxxx>
References: <20050306103312.3155B1E46E@xxxxxxxxxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.7.3) Gecko/20040922
domen@xxxxxxxxxxxx wrote:
Replace pci_find_device with pci_dev_present.  Compile tested.

Signed-off-by: Scott Feldman <sfeldma@xxxxxxxxx>
Signed-off-by: Maximilian Attems <janitor@xxxxxxxxxxxxxx>
Signed-off-by: Domen Puncer <domen@xxxxxxxxxxxx>
---


 kj-domen/drivers/net/tg3.c |   24 ++++++++++++++----------
 1 files changed, 14 insertions(+), 10 deletions(-)

diff -puN drivers/net/tg3.c~remove-pci-find-device-drivers_net_tg3 
drivers/net/tg3.c
--- kj/drivers/net/tg3.c~remove-pci-find-device-drivers_net_tg3 2005-03-05 
16:09:45.000000000 +0100
+++ kj-domen/drivers/net/tg3.c  2005-03-05 16:09:45.000000000 +0100
@@ -7829,6 +7829,19 @@ static int __devinit tg3_is_sun_570X(str
static int __devinit tg3_get_invariants(struct tg3 *tp)
 {
+       static struct pci_device_id write_reorder_chipsets[] = {
+               { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
+                            PCI_DEVICE_ID_INTEL_82801AA_8) },
+               { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
+                            PCI_DEVICE_ID_INTEL_82801AB_8) },
+               { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
+                            PCI_DEVICE_ID_INTEL_82801BA_11) },
+               { PCI_DEVICE(PCI_VENDOR_ID_INTEL,
+                            PCI_DEVICE_ID_INTEL_82801BA_6) },
+               { PCI_DEVICE(PCI_VENDOR_ID_AMD,
+                            PCI_DEVICE_ID_AMD_FE_GATE_700C) },
+               { },
+       };
        u32 misc_ctrl_reg;
        u32 cacheline_sz_reg;
        u32 pci_state_reg, grc_misc_cfg;
@@ -7847,16 +7860,7 @@ static int __devinit tg3_get_invariants(
         * every mailbox register write to force the writes to be
         * posted to the chip in order.
         */
-       if (pci_find_device(PCI_VENDOR_ID_INTEL,
-                           PCI_DEVICE_ID_INTEL_82801AA_8, NULL) ||
-           pci_find_device(PCI_VENDOR_ID_INTEL,
-                           PCI_DEVICE_ID_INTEL_82801AB_8, NULL) ||
-           pci_find_device(PCI_VENDOR_ID_INTEL,
-                           PCI_DEVICE_ID_INTEL_82801BA_11, NULL) ||
-           pci_find_device(PCI_VENDOR_ID_INTEL,
-                           PCI_DEVICE_ID_INTEL_82801BA_6, NULL) ||
-           pci_find_device(PCI_VENDOR_ID_AMD,
-                           PCI_DEVICE_ID_AMD_FE_GATE_700C, NULL))
+       if (pci_dev_present(write_reorder_chipsets))
                tp->tg3_flags |= TG3_FLAG_MBOX_WRITE_REORDER;

seems OK to me.  DaveM should be applying this, though.

        Jeff




<Prev in Thread] Current Thread [Next in Thread>