On Wednesday 23 August 2006 00:07, Larry Finger wrote: > John, > > Please apply this to wireless-2.6. > > Michael - bcm43xx-d80211 probably needs this as well. > > Larry > > --------------------------------------------------- > > When bcm43xx-softmac is given an ifdown/ifup sequence, the value for > bcm->mac_suspended ends up > wrong, which leads to a large number of assert(bcm->mac_suspended>=0) > messages. This one-line patch > fixes this problem.
I think the following is the correct fix for the issue. It is already in the d80211 branch. (Seems like it got lost somehow). Can you test this? Signed-off-by: Michael Buesch <[EMAIL PROTECTED]> Index: wireless-2.6/drivers/net/wireless/bcm43xx/bcm43xx_main.c =================================================================== --- wireless-2.6.orig/drivers/net/wireless/bcm43xx/bcm43xx_main.c 2006-08-23 10:00:27.000000000 +0200 +++ wireless-2.6/drivers/net/wireless/bcm43xx/bcm43xx_main.c 2006-08-23 10:01:45.000000000 +0200 @@ -3349,6 +3349,8 @@ memset(bcm->dma_reason, 0, sizeof(bcm->dma_reason)); bcm->irq_savedstate = BCM43xx_IRQ_INITIAL; + bcm->mac_suspended = 1; + /* Noise calculation context */ memset(&bcm->noisecalc, 0, sizeof(bcm->noisecalc)); -- Greetings Michael. - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html