On Saturday 06 May 2006 20:24, David Woodhouse wrote: > On Fri, 2006-05-05 at 17:38 +0100, David Woodhouse wrote: > > I still need this hack to work around the fact that softmac doesn't > > attempt to associate when we bring the device up... > > It'd be quite good to get this fixed in 2.6.17 too. Otherwise, the > device doesn't manage to associate if you use the fairly common sequence > of iwconfig then dhclient. > > It's a bit of an evil hack and it should really be fixed in softmac -- > but it's only moving an _existing_ hack from one place in the driver to > another. > > Signed-off-by: David Woodhouse <[EMAIL PROTECTED]>
Signed-off-by: Michael Buesch <[EMAIL PROTECTED]> John, please try to push this before 2.6.17. Thanks. > --- linux-2.6.16.ppc/drivers/net/wireless/bcm43xx/bcm43xx_main.c.orig > 2006-05-05 17:14:26.000000000 +0100 > +++ linux-2.6.16.ppc/drivers/net/wireless/bcm43xx/bcm43xx_main.c > 2006-05-05 17:15:19.000000000 +0100 > @@ -3263,6 +3263,9 @@ static int bcm43xx_init_board(struct bcm > bcm43xx_sysfs_register(bcm); > //FIXME: check for bcm43xx_sysfs_register failure. This function is a > bit messy regarding unwinding, though... > > + /*FIXME: This should be handled by softmac instead. */ > + schedule_work(&bcm->softmac->associnfo.work); > + > assert(err == 0); > out: > return err; > @@ -3937,9 +3940,6 @@ static int bcm43xx_resume(struct pci_dev > > netif_device_attach(net_dev); > > - /*FIXME: This should be handled by softmac instead. */ > - schedule_work(&bcm->softmac->associnfo.work); > - > dprintk(KERN_INFO PFX "Device resumed.\n"); > > return 0; > -- 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