Cool, thanks.<div><br></div><div>We'll eventually cut this code out. I don't think anyone shipped an AR7010+AR5418.</div><div><br></div><div><br></div><div><br></div><div><br></div><div><br>Adrian<br><br><div class="gmail_quote">
On 30 March 2013 01:15, Oleksij Rempel <span dir="ltr"><<a href="mailto:linux@rempel-privat.de" target="_blank">linux@rempel-privat.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
reason for this part of code:<br>
"AR5416/AR5418 doesn't work with RTS + aggregates longer than 8KiB,<br>
so the driver has to either limit aggregate sizes to 8KiB or less,<br>
or disable RTS on longer aggregates. This was fixed in subsequent chips."<br>
  Adrian Chadd<br>
<br>
Signed-off-by: Oleksij Rempel <<a href="mailto:linux@rempel-privat.de">linux@rempel-privat.de</a>><br>
---<br>
 target_firmware/wlan/if_owl.c | 8 +++++---<br>
 1 file changed, 5 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/target_firmware/wlan/if_owl.c b/target_firmware/wlan/if_owl.c<br>
index e42e6be..4ab16c7 100755<br>
--- a/target_firmware/wlan/if_owl.c<br>
+++ b/target_firmware/wlan/if_owl.c<br>
@@ -339,7 +339,7 @@ static void ath_buf_set_rate(struct ath_softc_tgt *sc, struct ath_tx_buf *bf)<br>
     HAL_11N_RATE_SERIES series[4];<br>
     a_int32_t i, flags;<br>
     a_uint8_t rix, cix, rtsctsrate;<br>
-    a_uint32_t aggr_limit_with_rts;<br>
+    a_uint32_t aggr_limit_with_rts = 0;<br>
     a_uint32_t ctsduration = 0;<br>
     a_int32_t prot_mode = AH_FALSE;<br>
<br>
@@ -367,8 +367,10 @@ static void ath_buf_set_rate(struct ath_softc_tgt *sc, struct ath_tx_buf *bf)<br>
            }<br>
     }<br>
<br>
-    ath_hal_getrtsaggrlimit(sc->sc_ah, &aggr_limit_with_rts);<br>
-<br>
+    /* TODO: this code is artifact from old chipsets<br>
+     * AR5416/AR5418, for now we will disable it with aggr_limit_with_rts=0<br>
+     * If some body will complain - we will need to take closer look.<br>
+     * If not, remove it. (30.03.2013) */<br>
     if (bf->bf_isaggr && aggr_limit_with_rts &&<br>
        bf->bf_al > aggr_limit_with_rts) {<br>
            flags &= ~(HAL_TXDESC_RTSENA);<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.8.1.2<br>
<br>
</font></span></blockquote></div><br></div>