On 20 February 2011 19:43, Stanislav Fomichev <[email protected]> wrote:
> (...)
> diff --git a/drivers/staging/brcm80211/util/bcmsrom.c 
> b/drivers/staging/brcm80211/util/bcmsrom.c
> index b26877c..c4cfd96 100644
> --- a/drivers/staging/brcm80211/util/bcmsrom.c
> +++ b/drivers/staging/brcm80211/util/bcmsrom.c
> @@ -25,7 +25,6 @@
>  #include <hndsoc.h>
>  #include <sbchipc.h>
>  #include <bcmdevs.h>
> -#include <bcmendian.h>
>  #include <pcicfg.h>
>  #include <siutils.h>
>  #include <bcmsrom.h>
> @@ -1438,6 +1437,18 @@ srom_cc_cmd(si_t *sih, struct osl_info *osh, void 
> *ccregs, u32 cmd,
>                return 0xffff;
>  }
>
> +static inline ltoh16_buf(u16 *buf, unsigned int size)
> +{
> +       for (size /= 2; size; size--)
> +               le16_to_cpu(*(buf + size));

Shouldn't this be something like *(buf + size) = le16_to_cpu(*(buf +
size));? Else the actual buffer content won't be changed.

> +}
> +
> +static inline htol16_buf(u16 *buf, unsigned int size)
> +{
> +       for (size /= 2; size; size--)
> +               cpu_to_le16(*(buf + size));
> +}
> +

Same here.


Regards,
Jonas Gorski
_______________________________________________
devel mailing list
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to