> +     if ((msg->flags & I2C_M_RD) == 0) {
> +             for (i = 0; i < 4 && i < msg->len; i++)
> +                     data_reg_lo = data_reg_lo |
> +                                     (msg->buf[i] << ((i & 0x3) * 8));
> +
> +             for (i = 4; i < 8 && i < msg->len; i++)
> +                     data_reg_hi = data_reg_hi |
> +                                     (msg->buf[i] << ((i & 0x3) * 8));

Same comment as in the last version: What about be32_to_cpu and friends
instead of the loops (here and later)?

Attachment: signature.asc
Description: Digital signature

Reply via email to