On Feb 3, 2008 4:37 PM, Johannes Berg <johannes@xxxxxxxxxxxxxxxx> wrote: > > > I'm finding it a bit cumbersome,. now when tx_rate is pointer and > > doesn't resides in the ieee80211_tx_control > > I've seen in your patch you've just ripped this code off but our rate > > scale algorithms doesn't work without it actually the code that read > > this value in rate scale algorithm is there but it actually looking > > into empty space... > > > > - tx_status->control.tx_rate = > > - iwl4965_hw_get_rate_n_flags(tx_resp->rate_n_flags); > > > > I'm just asking if you had something in mind or it's just something you forgot. > > TBH, that confused me completely since the rate scale algorithm tells > the hardware what rate to use so why does the hw need to assign a > different rate in the tx status? I see. Rate scale algorithm only tels FW from what rate to start transmission. FW is responsible to do downscaling. Which is quite reasonable as retransmission is real time process. Instead of retransmitting on the same rate FW downscales. T X response provides to the rate scaling algorithm the successful rate. There is a little difference between 4956 and 3945 how the rate scale table is given to FW, but this is off the topic. Thanks Tomas > 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
