Michael Buesch wrote:
> This patch adds support for new firmware.
> Please test this on old and new firmware.
I have tested this patch with old firmware. It seems to work; however my
testing is not complete as
my computer has started hanging with the "Caps Lock" light flashing. The crash
is not caused by this
patch as it happened with 2.6.24-rc5, which has run for many days. I do have a
suggestion for
changing the patch (see below).
> +static inline
> +size_t b43_txhdr_size(struct b43_wldev *dev)
> +{
> + if (b43_is_old_txhdr_format(dev))
> + return 100 + sizeof(struct b43_plcp_hdr6);
> + return 104 + sizeof(struct b43_plcp_hdr6);
> +}
Why not eliminate most of the magic numbers in this part with
size_t b43_txhdr_size(struct b43_wldev *dev)
{
if (b43_is_old_txhdr_format(dev))
return sizeof(struct b43_txhdr) - 4;
return sizeof(struct b43_txhdr);
}
Larry
_______________________________________________
Bcm43xx-dev mailing list
[email protected]
https://lists.berlios.de/mailman/listinfo/bcm43xx-dev
