Hi All,
This is my first patch(s).
I've decided to refactor ieee80211_iface_work function and break it down
to smaller better defined function.
I think these changes make the code much more readable and do not impose
no overhead.
I've tested these patches with sparse and checkpatch.pl
Function names might not be descriptive enough.
Hope you find this useful.
Alex Briskin (4):
0) [28e464b19aaaba90c8946fb979b58709d55dffcf]
Added new function ieee80211_is_skb_handled_by_pkt_type and moved
some code from ieee80211_iface_work to reduce complexity and
improve readability
1) [486e3d5abb4dc6361cdd923254a2b68d43dcdaba]
Refactored code in ieee80211_is_skb_handled_by_pkt_type.
"if () {} else if ()" replaced by switch case.
2) [9ef2eab8e831420bc6748a4466ffa6b7a99bf447]
Added new function ieee80211_is_handled_by_frame_control and moved
some code from ieee80211_iface_work to it.
3) [1de8cdf9a0c05c6a21d9e43e5b55862f6efcf450]
Added new function ieee80211_handle_by_vif_type with code from
ieee80211_iface_work.
At this point ieee80211_iface_work seems to me much more readable
and better understood.
net/mac80211/iface.c | 264 +++++++++++++++++++++++++++++----------------------
1 file changed, 150 insertions(+), 114 deletions(-)
--
2.5.0
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
