Re: bcm43xx regression in 2.6.24 (with patch)

[Michael Buesch]

On Tuesday 26 February 2008 22:38:56 Alexey Zaytsev wrote:
> On Tue, Feb 26, 2008 at 11:20 PM, John W. Linville
> <linville@xxxxxxxxxxxxx> 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?

Yeah, I think that's it.

> >  commit 013978b688d2a27af3ab55ca739e8c8ac7254870
> >  Author: Larry Finger <Larry.Finger@xxxxxxxxxxxx>
> >  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

Yeah, seems so. But if you are willing to test an adjusted version, I can
probably do a backported version of this patch.
This patch is well tested and does the right thing, so I'm willing to sign
off a backported version of this.
First need to find the place in the old phy.c code that matches the new wa.c code.

> 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.

It does not even touch the bcm43xx driver.
Besides that the bcm43xx driver is not broken. That's the whole reason
this damn thread started at all. So it can't be broken.

-- 
Greetings Michael.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/