NetBSD-Bugs archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

kern/46310: Please support cardbus bwi(4)



>Number:         46310
>Category:       kern
>Synopsis:       Please support cardbus bwi(4)
>Confidential:   no
>Severity:       serious
>Priority:       medium
>Responsible:    kern-bug-people
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sun Apr 08 20:35:01 +0000 2012
>Originator:     Ryo ONODERA
>Release:        NetBSD 6.99.4
>Organization:
        
>Environment:
        
        
System: NetBSD hydrogen.elements.tetera.org 6.99.4 NetBSD 6.99.4 (LEAFGIRLB) 
#5: Mon Apr 9 00:52:28 JST 2012 
root%hydrogen.elements.tetera.org@localhost:/usr/obj/sys/arch/i386/compile/LEAFGIRLB
 i386
Architecture: i386
Machine: i386
>Description:
NetBSD lacks cardbus version of bwi(4) support.
This is from OpenBSD.

This works at least with Buffalo/Melco WLI-CB-G54 cardbus card
on NetBSD/i386 6.99.3 and 6.99.4.
I have only i386 cardbus machine, but this will work with NetBSD/amd64.

Also src/share/man/man4/bwi.4 should be modified.
It has "driver does not yet support cardbus devices."

        
>How-To-Repeat:
        
>Fix:
        
diff -urN sys/dev/cardbus/files.cardbus sys/dev/cardbus.ryoon/files.cardbus
--- sys/dev/cardbus/files.cardbus       2010-03-27 12:04:51.000000000 +0900
+++ sys/dev/cardbus.ryoon/files.cardbus 2012-04-09 04:57:30.000000000 +0900
@@ -59,6 +59,12 @@
 file   dev/cardbus/if_tlp_cardbus.c    tlp_cardbus
 
 #
+# Broadcom BCM43xx wireless
+#
+attach bwi at cardbus with bwi_cardbus
+file   dev/cardbus/if_bwi_cardbus.c    bwi_cardbus
+
+#
 # OHCI USB controller
 #
 attach ohci at cardbus with ohci_cardbus
diff -urN sys/dev/cardbus/if_bwi_cardbus.c 
sys/dev/cardbus.ryoon/if_bwi_cardbus.c
--- sys/dev/cardbus/if_bwi_cardbus.c    1970-01-01 09:00:00.000000000 +0900
+++ sys/dev/cardbus.ryoon/if_bwi_cardbus.c      2012-04-09 04:53:29.000000000 
+0900
@@ -0,0 +1,287 @@
+/*     $NetBSD$ */
+/*     $OpenBSD: if_bwi_cardbus.c,v 1.13 2010/08/06 05:26:24 mglocker Exp $ */
+
+/*
+ * Copyright (c) 2007 Marcus Glocker <mglocker%openbsd.org@localhost>
+ * Copyright (c) 2006 Claudio Jeker <claudio%openbsd.org@localhost>
+ *
+ * Permission to use, copy, modify, and distribute this software for any
+ * purpose with or without fee is hereby granted, provided that the above
+ * copyright notice and this permission notice appear in all copies.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
+ * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
+ * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
+ * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
+ * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
+ * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
+ * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
+ */
+
+/*
+ * Cardbus front-end for the Broadcom AirForce
+ */
+
+#include <sys/cdefs.h>
+__KERNEL_RCSID(0,"$NetBSD$");
+
+#include "opt_inet.h"
+
+#include <sys/param.h>
+#include <sys/callout.h>
+#include <sys/device.h>
+#include <sys/kernel.h>
+#include <sys/malloc.h>
+#include <sys/mbuf.h>
+#include <sys/socket.h>
+#include <sys/sockio.h>
+#include <sys/systm.h>
+#include <sys/errno.h>
+
+#include <machine/endian.h>
+
+#include <net/if.h>
+#include <net/if_dl.h>
+#include <net/if_ether.h>
+#include <net/if_media.h>
+
+#include <net80211/ieee80211_var.h>
+#include <net80211/ieee80211_amrr.h>
+#include <net80211/ieee80211_radiotap.h>
+
+#include <dev/pci/pcireg.h>
+#include <dev/pci/pcivar.h>
+
+#include <dev/cardbus/cardbusvar.h>
+
+#include <dev/ic/bwireg.h>
+#include <dev/ic/bwivar.h>
+#include <dev/pci/pcidevs.h>
+
+struct bwi_cardbus_softc {
+       struct bwi_softc         csc_bwi;
+
+       /* cardbus specific goo */
+       cardbus_devfunc_t        csc_ct;
+       pcitag_t                 csc_tag;
+       void                    *csc_ih;
+
+       bus_size_t               csc_mapsize;
+       pcireg_t                 csc_bar_val;
+       int                      csc_intrline;
+       pci_chipset_tag_t        csc_pc;
+};
+
+int            bwi_cardbus_match(device_t, cfdata_t, void*);
+void           bwi_cardbus_attach(device_t, device_t, void *);
+int            bwi_cardbus_detach(device_t, int);
+void           bwi_cardbus_setup(struct bwi_cardbus_softc *);
+int            bwi_cardbus_enable(struct bwi_softc *);
+void           bwi_cardbus_disable(struct bwi_softc *);
+void           bwi_cardbus_conf_write(void *, uint32_t, uint32_t);
+uint32_t       bwi_cardbus_conf_read(void *, uint32_t);
+
+CFATTACH_DECL_NEW(bwi_cardbus, sizeof (struct bwi_cardbus_softc),
+    bwi_cardbus_match, bwi_cardbus_attach, bwi_cardbus_detach, NULL);
+
+int
+bwi_cardbus_match(device_t parent, cfdata_t match, void *aux)
+{
+       struct cardbus_attach_args *ca = aux;
+
+       if (PCI_VENDOR(ca->ca_id) == PCI_VENDOR_BROADCOM) {
+               switch (PCI_PRODUCT(ca->ca_id)) {
+               case PCI_PRODUCT_BROADCOM_BCM4303:
+               case PCI_PRODUCT_BROADCOM_BCM4306:
+               case PCI_PRODUCT_BROADCOM_BCM4306_2:
+               case PCI_PRODUCT_BROADCOM_BCM4307:
+               case PCI_PRODUCT_BROADCOM_BCM4309:
+               case PCI_PRODUCT_BROADCOM_BCM4311:
+               case PCI_PRODUCT_BROADCOM_BCM4312:
+               case PCI_PRODUCT_BROADCOM_BCM4318:
+               case PCI_PRODUCT_BROADCOM_BCM4319:
+               case PCI_PRODUCT_BROADCOM_BCM4322:
+               case PCI_PRODUCT_BROADCOM_BCM43XG:
+               case PCI_PRODUCT_BROADCOM_BCM4328:
+                       return (1);
+               default:
+                       return (0);
+               }
+       }
+
+       return (0);
+}
+
+void
+bwi_cardbus_attach(device_t parent, device_t self, void *aux)
+{
+       struct bwi_cardbus_softc *csc = device_private(self);
+       struct cardbus_attach_args *ca = aux;
+       struct bwi_softc *sc = &csc->csc_bwi;
+       cardbus_devfunc_t ct = ca->ca_ct;
+       pcireg_t reg;
+       bus_addr_t base;
+       int error;
+
+       sc->sc_dev = self;
+       sc->sc_dmat = ca->ca_dmat;
+       csc->csc_ct = ct;
+       csc->csc_tag = ca->ca_tag;
+
+       /* power management hooks */
+       sc->sc_enable = bwi_cardbus_enable;
+       sc->sc_disable = bwi_cardbus_disable;
+       //sc->sc_power = bwi_cardbus_power;
+
+       /* map control/status registers */
+       error = Cardbus_mapreg_map(ct, BWI_PCIR_BAR,
+           PCI_MAPREG_TYPE_MEM, 0, &sc->sc_mem_bt,
+           &sc->sc_mem_bh, &base, &csc->csc_mapsize);
+       if (error != 0) {
+               printf(": can't map mem space\n");
+               return;
+       }
+       csc->csc_bar_val = base | PCI_MAPREG_TYPE_MEM;
+
+       /* set up the PCI configuration registers */
+       bwi_cardbus_setup(csc);
+
+       printf(": irq %d", csc->csc_intrline);
+
+       /* we need to access Cardbus config space from the driver */
+       sc->sc_conf_read = bwi_cardbus_conf_read;
+       sc->sc_conf_write = bwi_cardbus_conf_write;
+
+       reg = (sc->sc_conf_read)(sc, PCI_SUBSYS_ID_REG);
+
+       sc->sc_pci_revid = PCI_REVISION(ca->ca_class);
+       sc->sc_pci_did = PCI_PRODUCT(ca->ca_id);
+       sc->sc_pci_subvid = PCI_VENDOR(reg);
+       sc->sc_pci_subdid = PCI_PRODUCT(reg);
+
+       error = bwi_attach(sc);
+       if (error != 0)
+               bwi_cardbus_detach(sc->sc_dev, 0);
+
+       Cardbus_function_disable(ct);
+}
+
+int
+bwi_cardbus_detach(device_t self, int flags)
+{
+       struct bwi_cardbus_softc *csc = device_private(self);
+       struct bwi_softc *sc = &csc->csc_bwi;
+       cardbus_devfunc_t ct = csc->csc_ct;
+       cardbus_chipset_tag_t cc = ct->ct_cc;
+       cardbus_function_tag_t cf = ct->ct_cf;
+
+#ifdef DIAGNOSTIC
+       if (ct == NULL)
+               panic("%s: data structure lacks", device_xname(self));
+
+#endif
+
+       bwi_detach(sc);
+
+       /* unhook the interrupt handler */
+       if (csc->csc_ih != NULL) {
+               cardbus_intr_disestablish(cc, cf, csc->csc_ih);
+               csc->csc_ih = NULL;
+       }
+
+       /* release bus space and close window */
+       Cardbus_mapreg_unmap(ct, BWI_PCIR_BAR, sc->sc_mem_bt,
+           sc->sc_mem_bh, csc->csc_mapsize);
+
+       return (0);
+}
+
+void
+bwi_cardbus_setup(struct bwi_cardbus_softc *csc)
+{
+       cardbus_devfunc_t ct = csc->csc_ct;
+       cardbus_chipset_tag_t cc = ct->ct_cc;
+       pci_chipset_tag_t pc = csc->csc_pc;
+       cardbus_function_tag_t cf = ct->ct_cf;
+       pcireg_t reg;
+
+       /* program the BAR */
+       cardbus_conf_write(cc, cf, csc->csc_tag, BWI_PCIR_BAR,
+           csc->csc_bar_val);
+
+       /* make sure the right access type is on the cardbus bridge */
+       (*cf->cardbus_ctrl)(cc, CARDBUS_MEM_ENABLE);
+       (*cf->cardbus_ctrl)(cc, CARDBUS_BM_ENABLE);
+
+       /* enable the appropriate bits in the PCI CSR */
+       reg = pci_conf_read(pc, csc->csc_tag,
+           PCI_COMMAND_STATUS_REG);
+       reg |= PCI_COMMAND_MASTER_ENABLE | PCI_COMMAND_MEM_ENABLE;
+       Cardbus_conf_write(ct, csc->csc_tag, PCI_COMMAND_STATUS_REG,
+           reg);
+}
+
+int
+bwi_cardbus_enable(struct bwi_softc *sc)
+{
+       struct bwi_cardbus_softc *csc = (struct bwi_cardbus_softc *)sc;
+       cardbus_devfunc_t ct = csc->csc_ct;
+       cardbus_chipset_tag_t cc = ct->ct_cc;
+       cardbus_function_tag_t cf = ct->ct_cf;
+
+       /* power on the socket */
+       Cardbus_function_enable(ct);
+
+       /* setup the PCI configuration registers */
+       bwi_cardbus_setup(csc);
+
+       /* map and establish the interrupt handler */
+       csc->csc_ih = cardbus_intr_establish(cc, cf, IPL_NET,
+           bwi_intr, sc);
+       if (csc->csc_ih == NULL) {
+               aprint_error_dev(sc->sc_dev,
+                   "unable to establish interrupt\n");
+               Cardbus_function_disable(ct);
+               return false;
+       }
+
+       return (0);
+}
+
+void
+bwi_cardbus_disable(struct bwi_softc *sc)
+{
+       struct bwi_cardbus_softc *csc = (struct bwi_cardbus_softc *)sc;
+       cardbus_devfunc_t ct = csc->csc_ct;
+       cardbus_chipset_tag_t cc = ct->ct_cc;
+       cardbus_function_tag_t cf = ct->ct_cf;
+
+       /* unhook the interrupt handler */
+       cardbus_intr_disestablish(cc, cf, csc->csc_ih);
+       csc->csc_ih = NULL;
+
+       /* power down the socket */
+       Cardbus_function_disable(ct);
+}
+
+void
+bwi_cardbus_conf_write(void *self, uint32_t reg, uint32_t val)
+{
+       struct bwi_cardbus_softc *csc = (struct bwi_cardbus_softc *)self;
+       cardbus_devfunc_t ct = csc->csc_ct;
+       cardbus_chipset_tag_t cc = ct->ct_cc;
+       cardbus_function_tag_t cf = ct->ct_cf;
+
+       cardbus_conf_write(cc, cf, csc->csc_tag, reg, val);
+}
+
+uint32_t
+bwi_cardbus_conf_read(void *self, uint32_t reg)
+{
+       struct bwi_cardbus_softc *csc = (struct bwi_cardbus_softc *)self;
+       cardbus_devfunc_t ct = csc->csc_ct;
+       cardbus_chipset_tag_t cc = ct->ct_cc;
+       cardbus_function_tag_t cf = ct->ct_cf;
+
+       return (cardbus_conf_read(cc, cf, csc->csc_tag, reg));
+}
Index: sys/arch/i386/conf/GENERIC
===================================================================
RCS file: /cvsroot/src/sys/arch/i386/conf/GENERIC,v
retrieving revision 1.1069
diff -u -r1.1069 GENERIC
--- sys/arch/i386/conf/GENERIC  7 Apr 2012 05:40:41 -0000       1.1069
+++ sys/arch/i386/conf/GENERIC  8 Apr 2012 19:57:45 -0000
@@ -1061,6 +1061,7 @@
 # CardBus network cards
 ath*   at cardbus? function ?  # Atheros 5210/5211/5212 802.11
 atw*   at cardbus? function ?  # ADMtek ADM8211 (802.11)
+bwi*   at cardbus? function ?  # Broadcom BCM43xx wireless
 ex*    at cardbus? function ?  # 3Com 3c575TX
 fxp*   at cardbus? function ?  # Intel i8255x
 ral*   at cardbus? function ?  # Ralink Technology RT25x0 802.11a/b/g


>Unformatted:
        
        


Home | Main Index | Thread Index | Old Index