Em Ter 08 Dez 2009, às 18:16:59, Larry Finger escreveu: > On 12/08/2009 09:41 AM, Herton Ronaldo Krzesinski wrote: > > That's why in my last patch I added code to register the radio led, we > > have to go that way to fix properly the issue (also this way we turn the > > led on/off only when mac80211 wants it, which is the right way), so we > > can avoid having to place code in start/stop or creating suspend/resume > > hooks only because the issue. > > I do not understand the reason for adding another LED device, nor why > you think it helps. When I added the LED code, I wanted to get the LED > to indicate that there was I/O activity, which is why I chose the TX > and RX LEDs. Choosing "on" as the default state when there was no > activity and blinking it "off" with I/O seems to give exactly the > right behavior. Unfortunately, the asynchronous behavior of the USB > operations makes it very tricky to get the shutdown right. It's because without registering a radio led the current behaviour is wrong for some cases, the led stays on when it shouldn't. For example, if you bring the interface down (ifconfig <if> down) the led stays on, when it shouldn't. The led should only be active while interface is active, between start and stop (or in other words, the radio is on). After mac80211 stops the device, any i/o (led activity) shouldn't happen. > > Larry -- []'s Herton -- 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
