Please don't submit three patches with identical subject. > Added ieee80211_iface_work_handle_pkt_type function. Moved part of Should also use present tense here - "Add ... function, move [...]". This also applies to the other patches. The if/else if/... -> switch change should also be part of this patch, or separate, but not squashed together with a different unrelated change. Might be worth moving "return 0" into each of the cases and return an error in the fall-through, but it doesn't really matter. 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