On Tue, 2008-07-08 at 15:25 -0700, Harvey Harrison wrote:
> Those functions that still use ieee80211_get_hdrlen are moved over
> to use the little endian frame control.
> - switch (fc & (IEEE80211_FCTL_TODS | IEEE80211_FCTL_FROMDS)) {
> - case IEEE80211_FCTL_TODS:
> - /* BSSID SA DA */
> - memcpy(dst, hdr->addr3, ETH_ALEN);
> - memcpy(src, hdr->addr2, ETH_ALEN);
> -
> + switch (hdr->frame_control &
> + cpu_to_le16(IEEE80211_FCTL_TODS | IEEE80211_FCTL_FROMDS)) {
> + case __constant_cpu_to_le16(IEEE80211_FCTL_TODS):
> if (unlikely(sdata->vif.type != IEEE80211_IF_TYPE_AP &&
> sdata->vif.type != IEEE80211_IF_TYPE_VLAN))
> return -1;
> break;
Looks good, but didn't you just lose all the memcpys?
johannes
Attachment:
signature.asc
Description: This is a digitally signed message part
