Re: Call For Review: Sonics Silicon Backplane(SSB) driver

On Mon, Nov 30, 2009 at 06:22:42AM +0100, Gonzalo Nemmi wrote:
> On Mon, Nov 30, 2009 at 4:41 AM, Weongyo Jeong <weongyo.jeong_at_gmail.com> wrote:
> > Hello,
> >
> > I've written a Sonics Silicon Backplane driver which is widely used for
> > Broadcom Wireless driver that bwi(4) can use this driver though it has
> > its own routines in it.
> >
> > I don't have any knowledge to write a bus driver like this so I'm not
> > sure my approach is correct. ?Please review!
> >
> > You can download a driver at:
> >
> > ? ? ? ?http://people.freebsd.org/~weongyo/ssb_20091129.tar.gz
> >
> > regards,
> > Weongyo Jeong
> 
> Hi there Weongyo!
> I´d like to know .. will it compile under 8.0-RELEASE or -STABLE?

Yes it should be.

> Should I try your module instead of bwi(4)?

No.  ssb driver is a kind of bus driver that it's not a wireless driver
so can not use.  But I'm working on new driver (aka bwn(4)) for
replacing bwi(4) that it uses v4 firmware (latest firmware) and supports
more H/Ws.  It's almost done that it works to associate with AP and
pings for RX/TX but not public yet.  When it's available let you know
for testing. :-)

> 
> And just in case you find it usefull, in ssb_pci.c, {
> PCI_VENDOR_BROADCOM, 0x4315, "Unknown" }, is actually  a 'Broadcom
> Wireless b/g (BCM4315/BCM22062000)'. I have one .. here you´ll find my
> pciconf -lcvb usinf bwi(4) with parado´s firmware:
> http://pastebin.com/f1f40900c
> 
> #bwi0_at_pci0:12:0:0:       class=0x028000 card=0x000b1028
> chip=0x431514e4 rev=0x01 hdr=0x00
> #    vendor     = 'Broadcom Corporation'
> #    device     = 'Broadcom Wireless b/g (BCM4315/BCM22062000)'
> #    class      = network
> #    cap 01[40] = powerspec 3  supports D0 D1 D2 D3  current D0
> #    cap 09[58] = vendor (length 120)
> #    cap 05[e8] = MSI supports 1 message, 64 bit
> #    cap 10[d0] = PCI-Express 1 endpoint max data 128(128) link x1(x1)
> 
> bwi0: PHY: type 5, rev 1, ver 6
> bwi0: RF: manu 0x17f, type 0x2062, rev 2
> 
> Hope that help :)

Thanks you for informations.

regards,
Weongyo Jeong