On Thu, Sep 17, 2015 at 11:02:57AM +0300, Jyri Sarha wrote:

>       val /= sizeof(u32);
>       for (i = 0; i < val; i++)
> -             if (be32_to_cpup(&of_slot_mask[i]))
> +             if (be32_to_cpup((__be32 *)&of_slot_mask[i]))
>                       *mask |= (1 << i);
>  

There was no changelog and this is setting off alarm bells since the
cast just smashes warnings - are you sure we're not missing some other
annotations and that a cast is the best thing here?

Attachment: signature.asc
Description: Digital signature

Reply via email to