iio: humidity: hts221: remove warnings in hts221_parse_{temp,rh}_caldata()
authorLorenzo Bianconi <lorenzo.bianconi@redhat.com>
Mon, 8 Jan 2018 22:12:28 +0000 (23:12 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Tue, 30 Jan 2018 18:33:15 +0000 (18:33 +0000)
Remove following sparse warnings in hts221_parse_temp_caldata() and in
hts221_parse_rh_caldata():
drivers/iio/humidity/hts221_core.c:302:19: warning: cast to
restricted __le16
drivers/iio/humidity/hts221_core.c:314:18: warning: cast to
restricted __le16
drivers/iio/humidity/hts221_core.c:320:18: warning: cast to
restricted __le16
drivers/iio/humidity/hts221_core.c:355:18: warning: cast to
restricted __le16
drivers/iio/humidity/hts221_core.c:361:18: warning: cast to
restricted __le16

Fixes: e4a70e3e7d84 ("iio: humidity: add support to hts221 rh/temp combo device")
Signed-off-by: Lorenzo Bianconi <lorenzo.bianconi@redhat.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
drivers/iio/humidity/hts221_core.c

index d3f7904766bd3f477343c91d45d3280e8c23ac1c..b662afb2b35c7c0c7131673761ad3129940790bc 100644 (file)
@@ -289,6 +289,7 @@ static int hts221_parse_temp_caldata(struct hts221_hw *hw)
        int err, *slope, *b_gen;
        s16 cal_x0, cal_x1, cal_y0, cal_y1;
        u8 cal0, cal1;
+       __le16 val;
 
        err = hw->tf->read(hw->dev, HTS221_REG_0T_CAL_Y_H,
                           sizeof(cal0), &cal0);
@@ -299,7 +300,7 @@ static int hts221_parse_temp_caldata(struct hts221_hw *hw)
                           sizeof(cal1), &cal1);
        if (err < 0)
                return err;
-       cal_y0 = (le16_to_cpu(cal1 & 0x3) << 8) | cal0;
+       cal_y0 = ((cal1 & 0x3) << 8) | cal0;
 
        err = hw->tf->read(hw->dev, HTS221_REG_1T_CAL_Y_H,
                           sizeof(cal0), &cal0);
@@ -307,17 +308,17 @@ static int hts221_parse_temp_caldata(struct hts221_hw *hw)
                return err;
        cal_y1 = (((cal1 & 0xc) >> 2) << 8) | cal0;
 
-       err = hw->tf->read(hw->dev, HTS221_REG_0T_CAL_X_L, sizeof(cal_x0),
-                          (u8 *)&cal_x0);
+       err = hw->tf->read(hw->dev, HTS221_REG_0T_CAL_X_L, sizeof(val),
+                          (u8 *)&val);
        if (err < 0)
                return err;
-       cal_x0 = le16_to_cpu(cal_x0);
+       cal_x0 = le16_to_cpu(val);
 
-       err = hw->tf->read(hw->dev, HTS221_REG_1T_CAL_X_L, sizeof(cal_x1),
-                          (u8 *)&cal_x1);
+       err = hw->tf->read(hw->dev, HTS221_REG_1T_CAL_X_L, sizeof(val),
+                          (u8 *)&val);
        if (err < 0)
                return err;
-       cal_x1 = le16_to_cpu(cal_x1);
+       cal_x1 = le16_to_cpu(val);
 
        slope = &hw->sensors[HTS221_SENSOR_T].slope;
        b_gen = &hw->sensors[HTS221_SENSOR_T].b_gen;
@@ -334,6 +335,7 @@ static int hts221_parse_rh_caldata(struct hts221_hw *hw)
 {
        int err, *slope, *b_gen;
        s16 cal_x0, cal_x1, cal_y0, cal_y1;
+       __le16 val;
        u8 data;
 
        err = hw->tf->read(hw->dev, HTS221_REG_0RH_CAL_Y_H, sizeof(data),
@@ -348,17 +350,17 @@ static int hts221_parse_rh_caldata(struct hts221_hw *hw)
                return err;
        cal_y1 = data;
 
-       err = hw->tf->read(hw->dev, HTS221_REG_0RH_CAL_X_H, sizeof(cal_x0),
-                          (u8 *)&cal_x0);
+       err = hw->tf->read(hw->dev, HTS221_REG_0RH_CAL_X_H, sizeof(val),
+                          (u8 *)&val);
        if (err < 0)
                return err;
-       cal_x0 = le16_to_cpu(cal_x0);
+       cal_x0 = le16_to_cpu(val);
 
-       err = hw->tf->read(hw->dev, HTS221_REG_1RH_CAL_X_H, sizeof(cal_x1),
-                          (u8 *)&cal_x1);
+       err = hw->tf->read(hw->dev, HTS221_REG_1RH_CAL_X_H, sizeof(val),
+                          (u8 *)&val);
        if (err < 0)
                return err;
-       cal_x1 = le16_to_cpu(cal_x1);
+       cal_x1 = le16_to_cpu(val);
 
        slope = &hw->sensors[HTS221_SENSOR_H].slope;
        b_gen = &hw->sensors[HTS221_SENSOR_H].b_gen;