- Remove unnecessary ETHER_* macros
authorSepherosa Ziehau <sephe@dragonflybsd.org>
Sun, 20 Nov 2005 11:59:54 +0000 (11:59 +0000)
committerSepherosa Ziehau <sephe@dragonflybsd.org>
Sun, 20 Nov 2005 11:59:54 +0000 (11:59 +0000)
- Utilize pcidevs.h for PCI device id and vendor id
- Prefer PCIR_BAR() macro to PCIR_MAPS
- Use common device probing style, epic_devtype() is merged
- Nuke epic_softc_t.cardvend, since it is only used in epic_common_attach().
  Move epic_softc_t.cardid to the end of the struct
- Minor style changes

sys/dev/netif/tx/if_tx.c
sys/dev/netif/tx/if_txreg.h
sys/dev/netif/tx/if_txvar.h

index 180ac28..f9c84fc 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/tx/if_tx.c,v 1.61.2.1 2002/10/29 01:43:49 semenu Exp $
- * $DragonFly: src/sys/dev/netif/tx/if_tx.c,v 1.29 2005/11/20 09:46:31 sephe Exp $
+ * $DragonFly: src/sys/dev/netif/tx/if_tx.c,v 1.30 2005/11/20 11:59:54 sephe Exp $
  */
 
 /*
 
 #include <bus/pci/pcireg.h>
 #include <bus/pci/pcivar.h>
+#include <bus/pci/pcidevs.h>
 
-#include "../mii_layer/mii.h"
-#include "../mii_layer/miivar.h"
-#include "../mii_layer/miidevs.h"
-#include "../mii_layer/lxtphyreg.h"
+#include <dev/netif/mii_layer/mii.h>
+#include <dev/netif/mii_layer/miivar.h>
+#include <dev/netif/mii_layer/miidevs.h>
+#include <dev/netif/mii_layer/lxtphyreg.h>
 
 #include "miibus_if.h"
 
-#include "if_txreg.h"
-#include "if_txvar.h"
+#include <dev/netif/tx/if_txreg.h>
+#include <dev/netif/tx/if_txvar.h>
 
 static int epic_ifioctl(struct ifnet *, u_long, caddr_t, struct ucred *);
 static void epic_intr(void *);
@@ -123,7 +124,6 @@ static int epic_probe(device_t);
 static int epic_attach(device_t);
 static void epic_shutdown(device_t);
 static int epic_detach(device_t);
-static struct epic_type *epic_devtype(device_t);
 
 static device_method_t epic_methods[] = {
        /* Device interface */
@@ -155,7 +155,7 @@ DRIVER_MODULE(if_tx, pci, epic_driver, epic_devclass, 0, 0);
 DRIVER_MODULE(miibus, tx, miibus_driver, miibus_devclass, 0, 0);
 
 static struct epic_type epic_devs[] = {
-       { SMC_VENDORID, SMC_DEVICEID_83C170,
+       { PCI_VENDOR_SMC, PCI_PRODUCT_SMC_83C170,
                "SMC EtherPower II 10/100" },
        { 0, 0, NULL }
 };
@@ -164,40 +164,25 @@ static int
 epic_probe(device_t dev)
 {
        struct epic_type *t;
-
-       t = epic_devtype(dev);
-
-       if (t != NULL) {
-               device_set_desc(dev, t->name);
-               return(0);
-       }
-
-       return(ENXIO);
-}
-
-static struct epic_type *
-epic_devtype(device_t dev)
-{
-       struct epic_type *t;
-
-       t = epic_devs;
-
-       while(t->name != NULL) {
-               if ((pci_get_vendor(dev) == t->ven_id) &&
-                   (pci_get_device(dev) == t->dev_id)) {
-                       return(t);
+       uint16_t vid, did;
+
+       vid = pci_get_vendor(dev);
+       did = pci_get_device(dev);
+       for (t = epic_devs; t->name != NULL; ++t) {
+               if (vid == t->ven_id && did == t->dev_id) {
+                       device_set_desc(dev, t->name);
+                       return 0;
                }
-               t++;
        }
-       return (NULL);
+       return ENXIO;
 }
 
 #if defined(EPIC_USEIOSPACE)
 #define        EPIC_RES        SYS_RES_IOPORT
-#define        EPIC_RID        PCIR_BASEIO
+#define EPIC_RID       PCIR_BAR(0)
 #else
 #define        EPIC_RES        SYS_RES_MEMORY
-#define        EPIC_RID        PCIR_BASEMEM
+#define EPIC_RID       PCIR_BAR(1)
 #endif
 
 /*
@@ -451,6 +436,7 @@ epic_ifioctl(struct ifnet *ifp, u_long command, caddr_t data, struct ucred *cr)
 static int
 epic_common_attach(epic_softc_t *sc)
 {
+       uint16_t sub_vid;
        int i;
 
        sc->tx_flist = malloc(sizeof(struct epic_frag_list)*TX_RING_SIZE,
@@ -482,11 +468,11 @@ epic_common_attach(epic_softc_t *sc)
        sc->serinst = -1;
 
        /* Fetch card id */
-       sc->cardvend = pci_get_subvendor(sc->dev);
+       sub_vid = pci_get_subvendor(sc->dev);
        sc->cardid = pci_get_subdevice(sc->dev);
 
-       if (sc->cardvend != SMC_VENDORID)
-               device_printf(sc->dev, "unknown card vendor %04xh\n", sc->cardvend);
+       if (sub_vid != PCI_VENDOR_SMC)
+               device_printf(sc->dev, "unknown card vendor %04xh\n", sub_vid);
 
        return 0;
 }
index 8b8b8bb..106ec87 100644 (file)
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/tx/if_txreg.h,v 1.12.2.1 2002/10/29 01:43:50 semenu Exp $
- * $DragonFly: src/sys/dev/netif/tx/if_txreg.h,v 1.2 2003/06/17 04:28:32 dillon Exp $
+ * $DragonFly: src/sys/dev/netif/tx/if_txreg.h,v 1.3 2005/11/20 11:59:54 sephe Exp $
  */
 
 #define        EPIC_MAX_MTU            1600    /* This is experiment-derived value */
 
-/* PCI aux configuration registers */
-#define        PCIR_BASEIO     (PCIR_MAPS + 0x0)       /* Base IO Address */
-#define        PCIR_BASEMEM    (PCIR_MAPS + 0x4)       /* Base Memory Address */
-
-/* PCI identification */
-#define SMC_VENDORID           0x10B8
-#define SMC_DEVICEID_83C170    0x0005
-
 /* EPIC's registers */
 #define        COMMAND         0x0000
 #define        INTSTAT         0x0004          /* Interrupt status. See below */
index 1e73375..eed964a 100644 (file)
@@ -24,7 +24,7 @@
  * SUCH DAMAGE.
  *
  * $FreeBSD: src/sys/dev/tx/if_txvar.h,v 1.14.2.1 2002/10/29 01:43:50 semenu Exp $
- * $DragonFly: src/sys/dev/netif/tx/if_txvar.h,v 1.5 2005/06/14 12:25:32 joerg Exp $
+ * $DragonFly: src/sys/dev/netif/tx/if_txvar.h,v 1.6 2005/11/20 11:59:54 sephe Exp $
  */
 
 /*
 /*#define      EPIC_USEIOSPACE 1*/
 /*#define      EPIC_EARLY_RX   1*/
 
-#ifndef ETHER_MAX_LEN
-#define ETHER_MAX_LEN          1518
-#endif
-#ifndef ETHER_MIN_LEN
-#define ETHER_MIN_LEN          64
-#endif
-#ifndef ETHER_CRC_LEN
-#define ETHER_CRC_LEN          4
-#endif
 #define TX_RING_SIZE           16              /* Leave this a power of 2 */
 #define RX_RING_SIZE           16              /* And this too, to do not */
                                                /* confuse RX(TX)_RING_MASK */
 
 /* This is driver's structure to define EPIC descriptors */
 struct epic_rx_buffer {
-       struct mbuf *           mbuf;           /* mbuf receiving packet */
+       struct mbuf             *mbuf;          /* mbuf receiving packet */
 };
 
 struct epic_tx_buffer {
-       struct mbuf *           mbuf;           /* mbuf contained packet */
+       struct mbuf             *mbuf;          /* mbuf contained packet */
 };
 
 /* PHY, known by tx driver */
@@ -96,12 +87,11 @@ typedef struct {
        u_int32_t               cur_rx;
        u_int32_t               dirty_tx;
        u_int32_t               pending_txs;
-       u_int16_t               cardvend;
-       u_int16_t               cardid;
        struct mii_softc        *physc;
        u_int32_t               phyid;
        int                     serinst;
        void                    *pool;
+       u_int16_t               cardid;
 } epic_softc_t;
 
 struct epic_type {