Johannes Berg schrieb: > On Mon, 2012-07-09 at 11:37 +0200, Johannes Berg wrote: > >>> (6) restructure tx-path of all effected drivers >>> Restructure tx-path of all effected drivers to respect new TPC support in >>> mac80211. TPC support is added to mac80211 by restructuring of struct >>> ieee80211_tx_info. Therfore the tx-path of all effected drivers is modified >>> to receive struct sta from the stack and respect the new ieee80211_tx_info >>> struct. List of modified driver: >> again, separate these patches. > > Oh and I don't mean split it per driver. Just split up the patch into > the logical API evolutions, like first removing the sta pointer and > introducing the tx_control struct. (Where, btw, you lost an important > comment about it being NULL along the way) Hi Johannes, >From the comments so far, I will split this patch into: -keeping the brcmsmac patch that was send earlier (I thought it is as single patch out of context if not combined with the goal to add tpc) -send one single new patch to remove the sta pointer from ieee_tx_info including all driver changes to respect this. (as each patch should not break a kernel build, I guess all of this must come in one patch) -after that introduce the TPC stuff in a separate patch series (where I address the comments that where already given) Greetings Thomas -- 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