Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-27 Thread Thomas Huehn
Hi Johannes, Johannes Berg schrieb: > I did it myself now, all applied. > Awesome ! I am going to prepare the TPC patches now. Greetings Thomas ___ ath5k-devel mailing list ath5k-devel@lists.ath5k.org https://lists.ath5k.org/mailman/listinfo/ath5k

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-27 Thread Johannes Berg
On Fri, 2012-07-27 at 11:35 +0200, Thomas Huehn wrote: > Hi Johannes, > > Johannes Berg schrieb: > > > > When you do, please remove the last two hunks from your patch that are > > spurious indentation changes in tx.c > > > When Ariks two TI driver patches make it in your tree I will clean up >

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-27 Thread Thomas Huehn
Hi Johannes, Johannes Berg schrieb: > When you do, please remove the last two hunks from your patch that are > spurious indentation changes in tx.c When Ariks two TI driver patches make it in your tree I will clean up this two hunks... no prob. Greetings Thomas ___

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-27 Thread Johannes Berg
On Thu, 2012-07-26 at 22:02 +0200, Thomas Huehn wrote: > > Better to just wait some time. I wrote up something to make it much > > easier for you. Waiting for some internal review before sending it up. > > > > Thx for helping out here, I ws allready checking into what kind of > struct I can put

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-26 Thread Thomas Huehn
Hi Arik, Arik Nemtsov schrieb: > Better to just wait some time. I wrote up something to make it much > easier for you. Waiting for some internal review before sending it up. > Thx for helping out here, I ws allready checking into what kind of struct I can put the sta pointer. But if you have a

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-26 Thread Arik Nemtsov
On Thu, Jul 26, 2012 at 8:04 PM, Arik Nemtsov wrote: > On Thu, Jul 26, 2012 at 7:57 PM, Arik Nemtsov wrote: >> On Thu, Jul 26, 2012 at 7:31 PM, Thomas Huehn >> wrote: >>> Hi Johannes, >>> >>> Johannes Berg schrieb: >>> > > /home/johannes/sys/wireless/drivers/net/wireless/ti/wlcore/tx.c:

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-26 Thread Arik Nemtsov
On Thu, Jul 26, 2012 at 7:31 PM, Thomas Huehn wrote: > Hi Johannes, > > Johannes Berg schrieb: > >>> >>> /home/johannes/sys/wireless/drivers/net/wireless/ti/wlcore/tx.c: In >>> function ‘wl1271_skb_queue_head’: >>> /home/johannes/sys/wireless/drivers/net/wireless/ti/wlcore/tx.c:622:48: >>> warning

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-26 Thread Arik Nemtsov
On Thu, Jul 26, 2012 at 7:57 PM, Arik Nemtsov wrote: > On Thu, Jul 26, 2012 at 7:31 PM, Thomas Huehn > wrote: >> Hi Johannes, >> >> Johannes Berg schrieb: >> /home/johannes/sys/wireless/drivers/net/wireless/ti/wlcore/tx.c: In function ‘wl1271_skb_queue_head’: /home/johannes/sy

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-26 Thread Thomas Huehn
Hi Johannes, Johannes Berg schrieb: >> >> /home/johannes/sys/wireless/drivers/net/wireless/ti/wlcore/tx.c: In >> function ‘wl1271_skb_queue_head’: >> /home/johannes/sys/wireless/drivers/net/wireless/ti/wlcore/tx.c:622:48: >> warning: ‘hlid’ may be used uninitialized in this function >> [-Wuniniti

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-26 Thread Johannes Berg
On Thu, 2012-07-26 at 18:17 +0200, Johannes Berg wrote: > On Thu, 2012-07-26 at 18:09 +0200, Thomas Huehn wrote: > > The pointer control.sta is removed from ieee80211_tx_info to free up > > sufficient memory in SKB_CB on the tx-path to enable new annotations > > per data packet e.g.support of upcom

Re: [ath5k-devel] [PATCH v6] mac80211: Remove control.sta from struct ieee80211_tx_info and restructure tx-path

2012-07-26 Thread Johannes Berg
On Thu, 2012-07-26 at 18:09 +0200, Thomas Huehn wrote: > The pointer control.sta is removed from ieee80211_tx_info to free up > sufficient memory in SKB_CB on the tx-path to enable new annotations > per data packet e.g.support of upcoming Transmit Power Control (TPC). > Now the control.sta pointer