Re: [PATCH v4 2/2] caif-driver: Add CAIF-SPI Protocol driver.

[David Miller <davem@davemloft.net>]

From: sjur.brandeland@stericsson.com
Date: Sat, 26 Jun 2010 23:31:29 +0200

> From: Sjur Braendeland <sjur.brandeland@stericsson.com>
> 
> This patch introduces the CAIF SPI Protocol Driver for
> CAIF Link Layer.
> 
> This driver implements a platform driver to accommodate for a
> platform specific SPI device. A general platform driver is not
> possible as there are no SPI Slave side Kernel API defined.
> A sample CAIF SPI Platform device can be found in
> .../Documentation/networking/caif/spi_porting.txt
> 
> Signed-off-by: Sjur Braendeland <sjur.brandeland@stericsson.com>

Applied, but:

> +
> +config CAIF_SPI_SYNC
> +	bool "Next command and length in start of frame"
> +	depends on CAIF_SPI_SLAVE
> +	default n
> +	---help---
> +	Putting the next command and length in the start of the frame can
> +	help to synchronize to the next transfer in case of over or under-runs.
> +	This option also needs to be enabled on the modem.
> +

This adds an extra new-line at the end of the file, which I corrected
when applying this.

GIT warns when trying to apply a patch with this problem,
checkpatch.pl probably does too, so you really can (in an automated
way) prevent these things from showing up in your submission.