[PATCH 1/3] ath10k: Fix shared WEP
Kalle Valo
kvalo at qca.qualcomm.com
Mon Nov 24 21:53:12 PST 2014
Sujith Manoharan <sujith at msujith.org> writes:
> Kalle Valo wrote:
>> No magic numbers, please. I didn't find anything for IV internals from
>> ieee80211.h, not sure if we should add them there or just use ath10k
>> internal defines.
>
> We are just retrieving the keyidx from the IV - not sure adding a couple
> of macros for "6" and "3" will make any difference.
It won't make a difference for you but it will for someone else reading
that part of code.
keyidx = skb->data[hdrlen + WEP_HDR_IV_LEN] >> WEP_HDR_KEY_ID_LSB;
versus:
keyidx = skb->data[hdrlen + 3] >> 6;
(I didn't check the standard so tha names can be totally wrong.)
--
Kalle Valo
More information about the ath10k
mailing list