On Tue, Mar 31, 2015 at 03:54:49PM +0200, Michael Grzeschik wrote:

>               if ((flags & SPI_MASTER_NO_TX) == 0)
> -                     setmosi(spi, word & (1 << 31));
> +                     if ((word & (1 << 31)) != oldbit) {
> +                             setmosi(spi, word & (1 << 31));
> +                             oldbit = word & (1 << 31);
> +                     }
>               spidelay(nsecs);        /* T(setup) */

This change is fine but for legibility can you please add braces around
the outer if as well?

Attachment: signature.asc
Description: Digital signature

Reply via email to