On Thu, Mar 19, 2015 at 09:01:50AM +0000, ker...@martin.sperl.org wrote:

Looks like I did get these but they took a while to come through...
odd.

> -static int bcm2835_spi_start_transfer(struct spi_device *spi,
> -             struct spi_transfer *tfr)
> +static int bcm2835_spi_start_transfer(
> +     struct spi_device *spi,
> +     struct spi_transfer *tfr)

Like Stephen says I'm not 100% sure what issues this is fixing but I
have to say I find the overall result worse.  It's possible the
warning is something reasonable but without knowing what it is it's hard
to say, I do like to see at least the first argument on the same line as
the function name though.

Attachment: signature.asc
Description: Digital signature

Reply via email to