On Fri, 2007-09-14 at 23:33 +0200, Michael Buesch wrote:

> -       macstat = le32_to_cpu(rxhdr->mac_status);
> +       macstat = le16_to_cpu(rxhdr->mac_status);
>         mactime = le16_to_cpu(rxhdr->mac_time);
>         chanstat = le16_to_cpu(rxhdr->channel);
> 
> But I think in this case this also didn't change the actual
> result in macstat.

I don't think so.  Suppose rxhdr->mac_status is 1.  le32_to_cpu() will
return 0x1000000, which would turn into 0 on conversion to u16.
le16_to_cpu() will return 0x100.  Since rxhdr->mac_status is accessed by
value (not by pointer), it will be 1 whether it's treated as 32-bit or
16-bit.

-- 
Regards,
Pavel Roskin

_______________________________________________
Bcm43xx-dev mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev

Reply via email to