On Wed, Jun 23, 2010 at 11:25:40PM +0530, Pavel Roskin wrote: > On Wed, 2010-06-23 at 12:08 +0530, Vivek Natarajan wrote: > > LED should be ON when the radio is put into FULL SLEEP mode during the idle > > unassociated state. > > Why? Is there any written policy for the LED behavior? I think we want > to conserve power if sleeping, which means turning LEDs off. There is no written policy for this behaviour, may be the comment needs to be changed. It makes sense to turn off the LED to save power but turning off the LED while wireless interface is still up is kind of misleading, also this one we got as a requirement internally and did not want to maintain any private patch. > > Or is it a workaround for some quirk in Atheros hardware? nope. > > In any case, I don't like the new comment. The comment says "Keep the > LED on...", but the code is still turning the LED off. yeah, this comment seems to be implicit. thanks for the review. Vasanth -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
