On Wed, Mar 17, 2010 at 11:37:29PM -0700, selvam wrote:
> Hi,
>
> The 4 ms frame length table is updated to support up to MCS 31.
>
> These values are calculated based on the equation 9.1 from the IEEE 802.11n standard.
> These values are rounded off to multiple of 4 bytes. If the final value is greater than 65535 then it is changed to 65532, since the maximum A-MPDU length is 65535 bytes.
Thanks for this patch, but can you elaborate on what your patch does other
than the above. For example, you may want to mention that although the current
ath9k driver does not support up to MCS31 your patch would enable such support
for future chipsets which could be supported. You also forgot to Signed-off-by
it, and provide a clearer commit log entry. Check git log for examples on
wireless-testing, also read:
http://wireless.kernel.org/en/developers/Documentation/SubmittingPatches
http://wireless.kernel.org/en/developers/Documentation
Luis
On Thu, Mar 18, 2010 at 09:30:34PM +0530, Luis R. Rodriguez wrote:
> On Wed, Mar 17, 2010 at 11:37:29PM -0700, selvam wrote:
> > Hi,
> >
> > The 4 ms frame length table is updated to support up to MCS 31.
> >
> > These values are calculated based on the equation 9.1 from the IEEE 802.11n standard.
> > These values are rounded off to multiple of 4 bytes. If the final value is greater than 65535 then it is changed to 65532, since the maximum A-MPDU length is 65535 bytes.
>
> Thanks for this patch, but can you elaborate on what your patch does other
> than the above. For example, you may want to mention that although the current
> ath9k driver does not support up to MCS31 your patch would enable such support
> for future chipsets which could be supported. You also forgot to Signed-off-by
> it, and provide a clearer commit log entry. Check git log for examples on
> wireless-testing, also read:
>
Also, rate control component of ath9k is not generic, it is specific
to ath9k. So this sort of patches make sense only when ath9k supports hw with
three chains, currently ath9k does not have support for those hw.
Vasanth