Oi Tue Jul 2008, Dmitry wrote: > 2008/7/22 Henrique de Moraes Holschuh <hmh@xxxxxxxxxx>: > > On Tue, 22 Jul 2008, Dmitry Baryshkov wrote: > >> Provide default activate function to set the state of the led > >> when the led becomes bound to the trigger > > > > I think I understand what this is for, but I better ask to be sure... > > What does it do, what is the use case? > > Register rfkill-> registers led trigger > Register led with rfkill trigger. > > However as rfkill doesn't provide the led_trigger->activate callback, > the led won't be in the proper state till the next rfkill state change. > With this patch, when led_trigger_set calls led_trigger->activate, > leds is set up with correct brightness. Thought as much. Please add that to the patch description... Acked-by: Henrique de Moraes Holschuh <hmh@xxxxxxxxxx> Now you just need Ivo's ACKs, since mine don't mean much :-) -- "One disk to rule them all, One disk to find them. One disk to bring them all and in the darkness grind them. In the Land of Redmond where the shadows lie." -- The Silicon Valley Tarot Henrique Holschuh -- 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