Re: bcm43xx regression in 2.6.24 (with patch)

Alexey Zaytsev
2008-02-26T21:39:14Z

On Tue, Feb 26, 2008 at 11:20 PM, John W. Linville
<linville@tuxdriver.com> wrote:
> On Sat, Feb 23, 2008 at 11:20:58PM +0100, Michael Buesch wrote:
> > On Saturday 23 February 2008 22:32:46 Alexey Zaytsev wrote:
>
>
> > > And you knew that the new driver did no work with the bcm4311
> > > chips, which is the sad thing.
> >
> > That is not true. It doesn't work with exactly _one_ revision
> > of the bcm4311 card. And that is already fixed in 2.6.25.
> > I'd like to have that in .24-stable, too, but I guess it's too big.
> > It changes some parts of the DMA engine code.
>
> Which patch is it? Is it this one? Or is there another one that touches 4311?
>
> commit 013978b688d2a27af3ab55ca739e8c8ac7254870
> Author: Larry Finger <Larry.Finger@lwfinger.net>
> Date: Mon Nov 26 10:29:47 2007 -0600
>
> b43: Changes to enable BCM4311 rev 02 with wireless core revision 13
>
> The BCM94311MCG rev 02 chip has an 802.11 core with revision 13 and
> has not been supported until now. The changes include the following:
>
> <snip>
>
> If so, perhaps we should just consider applying this to -stable?
>

I see a few problems here:

This commit seems to depend on some commits prior to it, as it touches
drivers/net/wireless/b43/wa.c which is absent in 2.6.24.3. When I get home,
I'll test if it still works if you ignore wa.c

The patch may be too complex for -stable. I think there was a hard line
count limit, like no more than 100 lines. I'm not sure if this still stands.

This does not fix the bcm43xx driver.

> I really do wish everyone would tone down their rhetoric on this
> issue...
>
> John
> --
> John W. Linville
> linville@tuxdriver.com
>