Re: [PATCH 1/3] drivers: misc: ad525x_dpot: Fix typo in spi write16and write24 transfer counts

From: Andrew Morton
Date: Thu Oct 14 2010 - 18:20:59 EST


On Thu, 14 Oct 2010 10:59:46 +0200
<michael.hennerich@xxxxxxxxxx> wrote:

> From: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
>
>
> Signed-off-by: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
> ---
> drivers/misc/ad525x_dpot-spi.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/misc/ad525x_dpot-spi.c b/drivers/misc/ad525x_dpot-spi.c
> index b8c6df9..6cfcb63 100644
> --- a/drivers/misc/ad525x_dpot-spi.c
> +++ b/drivers/misc/ad525x_dpot-spi.c
> @@ -53,13 +53,13 @@ static int write8(void *client, u8 val)
> static int write16(void *client, u8 reg, u8 val)
> {
> u8 data[2] = {reg, val};
> - return spi_write(client, data, 1);
> + return spi_write(client, data, 2);
> }
>
> static int write24(void *client, u8 reg, u16 val)
> {
> u8 data[3] = {reg, val >> 8, val};
> - return spi_write(client, data, 1);
> + return spi_write(client, data, 3);
> }
>
> static int read8(void *client)

What are the runtime effects of this change?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/