Re: [PATCH] spi: Only call transfer_one() if we have buffers to transfer

[Doug Anderson <dianders@xxxxxxxxxxxx>]

Mark,

On Sat, Aug 16, 2014 at 8:30 AM, Mark Brown <broonie@xxxxxxxxxx> wrote:
> From: Mark Brown <broonie@xxxxxxxxxx>
>
> Client drivers such as the ChomeOS EC driver sometimes use transfers with
> no buffers and only a delay specified in order to allow a delay after the
> assertion of /CS. Rather than require controller drivers handle this noop
> case gracefully put checks in the core to ensure that we don't call into
> the controller for such transfers.
>
> Reported-by: Addy Ke <addy.ke@xxxxxxxxxxxxxx>
> Signed-off-by: Mark Brown <broonie@xxxxxxxxxx>
> ---
>
> Completely untested at this point.
>
>  drivers/spi/spi.c | 44 ++++++++++++++++++++++++++------------------
>  1 file changed, 26 insertions(+), 18 deletions(-)

Thank you for writing this patch.  It looks great and works for me.

Tested-by: Doug Anderson <dianders@xxxxxxxxxxxx>
Reviewed-by: Doug Anderson <dianders@xxxxxxxxxxxx>
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html