Acked-by: Daniel Halperin <dhalperi@xxxxxxxxxxxxxxxxx> Tested-by: Daniel Halperin <dhalperi@xxxxxxxxxxxxxxxxx> (I suppose those are redundant?) Dan On Fri, Feb 25, 2011 at 7:49 AM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > > On Fri, 2011-02-25 at 07:31 -0800, Guy, Wey-Yi wrote: > > On Fri, 2011-02-25 at 07:39 -0800, Johannes Berg wrote: > > > On Fri, 2011-02-25 at 07:36 -0800, wwguy wrote: > > > > > > > > @@ -503,7 +488,46 @@ static void iwlagn_check_needed_chains(s > > > > > * will soon tell us about that. > > > > > */ > > > > > need_multiple = false; > > > > > + rcu_read_unlock(); > > > > > + break; > > > > > > > > why unlock and break here instead stay do it later like before? > > > > > > Only because the indentation there was getting so deep it was annoying > > > to fit into 80 cols :-) > > > > how you want to submit this patch? through iwlagn, or yo uwill send it > > along with mac80211 to John? > > Don't really care. I just sent them both here to keep them together and > for Daniel and Swati ... First though, I'd like them to test it maybe. > > johannes > -- 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