Re: [patch] 'Start All Wireless Devices' bug fix



On Sat, 25 Jun 2005, Robert Love wrote:
> What I _think_ we want (to be fair, I am not 100% sure of the intent of
> this code) is to only down and disable the device if !enabled.  But to
> set ->wireless_enabled and call nm_policy_schedule_device_change_check()
> either way.  Attached patch does this.
> 
> Tested, and it seems to work.
> 
> Makes sense?  If so, may I apply?

Looks good to me.  Go ahead.  I'm fairly sure that was the intent of the code, 
and that stuffing all this inside the if(!enabled) was an oversight on my part.

Dan



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]