Re: Associate on 'ifconfig up'

[Michael Buesch <mb@bu3sch.de>]

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 <dwmw2@infradead.org>

Signed-off-by: Michael Buesch <mb@bu3sch.de>

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.