]> git.proxmox.com Git - mirror_ubuntu-hirsute-kernel.git/commitdiff
iio:chemical:atlas-sensor: Drop unnecessary explicit casts in regmap_bulk_read calls
authorJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 5 Apr 2020 18:03:17 +0000 (19:03 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 19 Apr 2020 15:56:39 +0000 (16:56 +0100)
regmap_bulk_read takes a void * for its val parameter. It certainly
makes no sense to cast to a (u8 *) + no need to explicitly cast
at all when converting another pointer type to void *.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Acked-by: Matt Ranostay <matt.ranostay@konsulko.com>
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
Reviewed-by: Alexandru Ardelean <alexandru.ardelean@analog.com>
drivers/iio/chemical/atlas-sensor.c

index 82d470561ad3c6359f6aa37bb497f61457375230..973cdb4f1e8343d65c3831ebdcb2bc672dc9e347 100644 (file)
@@ -426,8 +426,7 @@ static irqreturn_t atlas_trigger_handler(int irq, void *private)
        int ret;
 
        ret = regmap_bulk_read(data->regmap, data->chip->data_reg,
-                             (u8 *) &data->buffer,
-                             sizeof(__be32) * channels);
+                             &data->buffer, sizeof(__be32) * channels);
 
        if (!ret)
                iio_push_to_buffers_with_timestamp(indio_dev, data->buffer,
@@ -463,7 +462,7 @@ static int atlas_read_measurement(struct atlas_data *data, int reg, __be32 *val)
        if (suspended)
                msleep(data->chip->delay);
 
-       ret = regmap_bulk_read(data->regmap, reg, (u8 *) val, sizeof(*val));
+       ret = regmap_bulk_read(data->regmap, reg, val, sizeof(*val));
 
        pm_runtime_mark_last_busy(dev);
        pm_runtime_put_autosuspend(dev);
@@ -485,7 +484,7 @@ static int atlas_read_raw(struct iio_dev *indio_dev,
                switch (chan->type) {
                case IIO_TEMP:
                        ret = regmap_bulk_read(data->regmap, chan->address,
-                                             (u8 *) &reg, sizeof(reg));
+                                              &reg, sizeof(reg));
                        break;
                case IIO_PH:
                case IIO_CONCENTRATION: