On 10/05/13 08:45, Lars-Peter Clausen wrote:
> Use the spi_sync_transfer() helper function instead of open-coding it. Makes
> the code a bit shorter.
>
> Signed-off-by: Lars-Peter Clausen <lars@xxxxxxxxxx>
Applied to the togreg branch of iio.git
Thanks,
> ---
> drivers/iio/dac/ad5449.c | 7 +------
> 1 file changed, 1 insertion(+), 6 deletions(-)
>
> diff --git a/drivers/iio/dac/ad5449.c b/drivers/iio/dac/ad5449.c
> index fff7d07..82e208f 100644
> --- a/drivers/iio/dac/ad5449.c
> +++ b/drivers/iio/dac/ad5449.c
> @@ -101,7 +101,6 @@ static int ad5449_read(struct iio_dev *indio_dev, unsigned int addr,
> {
> struct ad5449 *st = iio_priv(indio_dev);
> int ret;
> - struct spi_message msg;
> struct spi_transfer t[] = {
> {
> .tx_buf = &st->data[0],
> @@ -114,15 +113,11 @@ static int ad5449_read(struct iio_dev *indio_dev, unsigned int addr,
> },
> };
>
> - spi_message_init(&msg);
> - spi_message_add_tail(&t[0], &msg);
> - spi_message_add_tail(&t[1], &msg);
> -
> mutex_lock(&indio_dev->mlock);
> st->data[0] = cpu_to_be16(addr << 12);
> st->data[1] = cpu_to_be16(AD5449_CMD_NOOP);
>
> - ret = spi_sync(st->spi, &msg);
> + ret = spi_sync_transfer(st->spi, t, ARRAY_SIZE(t));
> if (ret < 0)
> goto out_unlock;
>
>
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
