iio:imu:mpu6050: Tidy up parameters to regmap_bulk functions.
authorJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 5 Apr 2020 18:03:19 +0000 (19:03 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 19 Apr 2020 15:56:39 +0000 (16:56 +0100)
A mixture of:
* Casts to u8 * for the val parameter which is void * and so needs no cast.
* Explicit sizes when we can use sizeof on the destination / source variable.

Not broken in anyway, I'm just trying to tidy up the is in the subsystem to
avoid it getting cut and paste into new drivers.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Reviewed-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
drivers/iio/imu/inv_mpu6050/inv_mpu_core.c

index e4b0d368c2f99079c4f39f03d8db8cde95fbf0cd..d3819b516ec6c431a321392aa217d3779328a15e 100644 (file)
@@ -526,7 +526,7 @@ static int inv_mpu6050_sensor_set(struct inv_mpu6050_state  *st, int reg,
        __be16 d = cpu_to_be16(val);
 
        ind = (axis - IIO_MOD_X) * 2;
-       result = regmap_bulk_write(st->map, reg + ind, (u8 *)&d, 2);
+       result = regmap_bulk_write(st->map, reg + ind, &d, sizeof(d));
        if (result)
                return -EINVAL;
 
@@ -540,7 +540,7 @@ static int inv_mpu6050_sensor_show(struct inv_mpu6050_state  *st, int reg,
        __be16 d;
 
        ind = (axis - IIO_MOD_X) * 2;
-       result = regmap_bulk_read(st->map, reg + ind, (u8 *)&d, 2);
+       result = regmap_bulk_read(st->map, reg + ind, &d, sizeof(d));
        if (result)
                return -EINVAL;
        *val = (short)be16_to_cpup(&d);