linux-kernel - RE: [PATCH 1/3] drivers: misc: ad525x_dpot: Fix typo in spi write16 and write24 transfer counts

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <544AC56F16B56944AEC3BD4E3D5917713094373C8C@LIMKCMBX1.ad.analog.com>
Date:	Fri, 15 Oct 2010 08:31:04 +0100
From:	"Hennerich, Michael" <Michael.Hennerich@...log.com>
To:	Andrew Morton <akpm@...ux-foundation.org>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"device-drivers-devel@...ckfin.uclinux.org" 
	<device-drivers-devel@...ckfin.uclinux.org>,
	Drivers <Drivers@...log.com>,
	"Frysinger, Michael" <Michael.Frysinger@...log.com>,
	"chrisv@...erswitching.com" <chrisv@...erswitching.com>
Subject: RE: [PATCH 1/3] drivers: misc: ad525x_dpot: Fix typo in spi write16
 and write24 transfer counts

Andrew Morton wrote on 2010-10-15:
> On Thu, 14 Oct 2010 10:59:46 +0200
> <michael.hennerich@...log.com> wrote:
>
>> From: Michael Hennerich <michael.hennerich@...log.com>
>>
>>
>> Signed-off-by: Michael Hennerich <michael.hennerich@...log.com>
>> ---
>>  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?

Hi Andrew,

This is a bug fix. Some SPI connected devices using
16/24 bit accesses, previously failed, now work.

This typo slipped in after testing, during some restructuring.

Greetings,
Michael

Analog Devices GmbH      Wilhelm-Wagenfeld-Str. 6      80807 Muenchen
Sitz der Gesellschaft Muenchen, Registergericht Muenchen HRB 4036 Geschaeftsfuehrer Thomas Wessel, William A. Martin, Margaret Seif

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