diff options
author | Jean Delvare <khali@linux-fr.org> | 2011-11-04 07:00:47 -0400 |
---|---|---|
committer | Jean Delvare <khali@endymion.delvare> | 2011-11-04 07:00:47 -0400 |
commit | 90f4102ce59226954edbe960b2434d8b3da5f086 (patch) | |
tree | 93fd275039932253b16ea125c1ba5eea2995b719 /drivers/hwmon/asb100.c | |
parent | 371f2e083b9b081adf68d04fba4978a27dc4e618 (diff) |
hwmon: Use i2c_smbus_{read,write}_word_swapped
Make use of the new i2c_smbus_{read,write}_word_swapped functions.
This makes the driver code more compact and readable. It also ensures
proper error handling.
Signed-off-by: Jean Delvare <khali@linux-fr.org>
Acked-by: Jonathan Cameron <jic23@cam.ac.uk>
Acked-by: Guenter Roeck <guenter.roeck@ericsson.com>
Cc: Dirk Eibach <eibach@gdsys.de>
Cc: "Mark M. Hoffman" <mhoffman@lightlink.com>
Cc: Guillaume Ligneul <guillaume.ligneul@gmail.com>
Diffstat (limited to 'drivers/hwmon/asb100.c')
-rw-r--r-- | drivers/hwmon/asb100.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/hwmon/asb100.c b/drivers/hwmon/asb100.c index c02a052d308..d7bd1f3f2a3 100644 --- a/drivers/hwmon/asb100.c +++ b/drivers/hwmon/asb100.c | |||
@@ -829,17 +829,17 @@ static int asb100_read_value(struct i2c_client *client, u16 reg) | |||
829 | /* convert from ISA to LM75 I2C addresses */ | 829 | /* convert from ISA to LM75 I2C addresses */ |
830 | switch (reg & 0xff) { | 830 | switch (reg & 0xff) { |
831 | case 0x50: /* TEMP */ | 831 | case 0x50: /* TEMP */ |
832 | res = swab16(i2c_smbus_read_word_data(cl, 0)); | 832 | res = i2c_smbus_read_word_swapped(cl, 0); |
833 | break; | 833 | break; |
834 | case 0x52: /* CONFIG */ | 834 | case 0x52: /* CONFIG */ |
835 | res = i2c_smbus_read_byte_data(cl, 1); | 835 | res = i2c_smbus_read_byte_data(cl, 1); |
836 | break; | 836 | break; |
837 | case 0x53: /* HYST */ | 837 | case 0x53: /* HYST */ |
838 | res = swab16(i2c_smbus_read_word_data(cl, 2)); | 838 | res = i2c_smbus_read_word_swapped(cl, 2); |
839 | break; | 839 | break; |
840 | case 0x55: /* MAX */ | 840 | case 0x55: /* MAX */ |
841 | default: | 841 | default: |
842 | res = swab16(i2c_smbus_read_word_data(cl, 3)); | 842 | res = i2c_smbus_read_word_swapped(cl, 3); |
843 | break; | 843 | break; |
844 | } | 844 | } |
845 | } | 845 | } |
@@ -877,10 +877,10 @@ static void asb100_write_value(struct i2c_client *client, u16 reg, u16 value) | |||
877 | i2c_smbus_write_byte_data(cl, 1, value & 0xff); | 877 | i2c_smbus_write_byte_data(cl, 1, value & 0xff); |
878 | break; | 878 | break; |
879 | case 0x53: /* HYST */ | 879 | case 0x53: /* HYST */ |
880 | i2c_smbus_write_word_data(cl, 2, swab16(value)); | 880 | i2c_smbus_write_word_swapped(cl, 2, value); |
881 | break; | 881 | break; |
882 | case 0x55: /* MAX */ | 882 | case 0x55: /* MAX */ |
883 | i2c_smbus_write_word_data(cl, 3, swab16(value)); | 883 | i2c_smbus_write_word_swapped(cl, 3, value); |
884 | break; | 884 | break; |
885 | } | 885 | } |
886 | } | 886 | } |