Re: [PATCH] spi: add spi_sync_single_transfer wrapper for single spi_transfer

From: Mark Brown
Date: Fri Jun 17 2016 - 07:35:07 EST


On Fri, Jun 17, 2016 at 09:43:11AM +0900, Andi Shyti wrote:

> > > The spi_sync_single_transfer function calls spi_sync_transfer
> > > with a single spi_transfer element, instead of an array.

> > So, what's the advantage of using this as opposed to calling
> > spi_sync_transfer with a 1 for the number of transfers?

> Not much, but it keeps the code a bit nicer to read for those
> using spi_sync_transfer with only one spi_transfer. Besides it's
> also more understandable what the function itself does and there
> would not be any need to jump into the spi_sync_transfer to check
> what the number '1' is needed for (for example it's not a boolean
> 'true' value).

I really don't think this has been a big source of confusion for people.

Attachment: signature.asc
Description: PGP signature