Hennerich, Michael wrote on 2011-10-21:
> Jonathan Cameron wrote on 2011-10-21:
>> This varient was introduced in
>> i2c: boilerplate function for byte swapped smbus_write/read_word_data
>>
>> This also has the side effect of ensuring any errors from the i2c
>> read and no longer mangled.
>>
>> Signed-off-by: Jonathan Cameron <jic23@xxxxxxxxx>
>
> Acked-by: Michael Hennerich <michael.hennerich@xxxxxxxxxx>
I think we missing one case in ad7879_i2c_write()...
>
>>
>> --- The patch introducing this swapped function is working its way
>> through the i2c tree.
>>
>> drivers/input/touchscreen/ad7879-i2c.c | 2 +-
>> 1 files changed, 1 insertions(+), 1 deletions(-)
>> diff --git a/drivers/input/touchscreen/ad7879-i2c.c
>> b/drivers/input/touchscreen/ad7879-i2c.c index 4e4e58c..1c1683e 100644
>> --- a/drivers/input/touchscreen/ad7879-i2c.c +++
>> b/drivers/input/touchscreen/ad7879-i2c.c @@ -47,7 +47,7 @@ static int
>> ad7879_i2c_read(struct device *dev, u8 reg)
>> {
>> struct i2c_client *client = to_i2c_client(dev);
>> - return swab16(i2c_smbus_read_word_data(client, reg));
>> + return i2c_smbus_read_word_swapped(client, reg);
>> }
>>
>> static int ad7879_i2c_multi_read(struct device *dev,
>
> Greetings,
> Michael
>
Greetings,
Michael
--
Analog Devices GmbH Wilhelm-Wagenfeld-Str. 6 80807 Muenchen
Sitz der Gesellschaft: Muenchen; Registergericht: Muenchen HRB 40368;
Geschaeftsfuehrer:Dr.Carsten Suckrow, Thomas Wessel, William A. Martin, Margaret Seif
--
To unsubscribe from this list: send the line "unsubscribe linux-input" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
