Hi Matt,

On Tue, 21 Feb 2012 11:32:13 +0000 Matt Fleming <matt.fleming@intel.com> wrote:
>
> +static inline u32 read32_le(u8 *src)
> +{
> +	u32 data;
> +
> +	data = *src++;
> +	data |= *src++ >> 8;
> +	data |= *src++ >> 16;
> +	data |= *src++ >> 24;

Shouldn't these be "<<" ?

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/