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>

1 files changed, 5 insertions, 5 deletions

diff --git a/drivers/hwmon/w83781d.c b/drivers/hwmon/w83781d.c
index eed43a008be1..65b685e2c7b7 100644
--- a/drivers/hwmon/w83781d.c
+++ b/drivers/hwmon/w83781d.c
@@ -1245,17 +1245,17 @@ w83781d_read_value_i2c(struct w83781d_data *data, u16 reg)
                 /* convert from ISA to LM75 I2C addresses */
                 switch (reg & 0xff) {
                 case 0x50:      /* TEMP */
-                        res = swab16(i2c_smbus_read_word_data(cl, 0));
+                        res = i2c_smbus_read_word_swapped(cl, 0);
                         break;
                 case 0x52:      /* CONFIG */
                         res = i2c_smbus_read_byte_data(cl, 1);
                         break;
                 case 0x53:      /* HYST */
-                        res = swab16(i2c_smbus_read_word_data(cl, 2));
+                        res = i2c_smbus_read_word_swapped(cl, 2);
                         break;
                 case 0x55:      /* OVER */
                 default:
-                        res = swab16(i2c_smbus_read_word_data(cl, 3));
+                        res = i2c_smbus_read_word_swapped(cl, 3);
                         break;
                 }
         }
@@ -1289,10 +1289,10 @@ w83781d_write_value_i2c(struct w83781d_data *data, u16 reg, u16 value)
                         i2c_smbus_write_byte_data(cl, 1, value & 0xff);
                         break;
                 case 0x53:      /* HYST */
-                        i2c_smbus_write_word_data(cl, 2, swab16(value));
+                        i2c_smbus_write_word_swapped(cl, 2, value);
                         break;
                 case 0x55:      /* OVER */
-                        i2c_smbus_write_word_data(cl, 3, swab16(value));
+                        i2c_smbus_write_word_swapped(cl, 3, value);
                         break;
                 }
         }