Re: [PATCH 0/6] IIO: Tidy up parameters to regmap_bulk_* calls

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



If anyone is bored and fancies sanity checking the ones in this series
that have had no eyes on them yet, it would be much appreciated!

Thanks,

Jonathan

On Sun,  5 Apr 2020 19:03:15 +0100
jic23@xxxxxxxxxx wrote:

> From: Jonathan Cameron <Jonathan.Cameron@xxxxxxxxxx>
> 
> One of Andy Shevchenko's recent patches tidied a case of this up
> and a quick grep showed we had some more.
> 
> The main focus of this set is calls to regmap_bulk_read /
> regmap_bulk_write.  Both are used with buffers of many and varied type,
> but as they take a void * there is never any need to cast a pointer
> to said buffers to anything explicitly.
> 
> Whilst I was here I used sizeof(buf) in various places to reduce
> the use of explicit values when we could derive it from the source
> / destination buffer.
> 
> This isn't remotely urgent, but I'd like to clean these out to avoid
> replication in future.  Not I didn't go after any other unnecessary
> casts in the files because that would have take thought and its Sunday
> evening and about time for a beer.
> 
> Jonathan Cameron (6):
>   iio:accel:mxc4005: Drop unnecessary explicit casts in regmap_bulk_read
>     calls
>   iio:chemical:atlas-sensor: Drop unnecessary explicit casts in
>     regmap_bulk_read calls
>   iio:chemical:bme680: Tidy up parameters to regmap_bulk_read
>   iio:imu:mpu6050: Tidy up parameters to regmap_bulk functions.
>   iio:magn:mmc35240: Drop unnecessary casts of val parameter in
>     regmap_bulk*
>   iio:light:ltr501: Drop unnecessary cast of parameter in
>     regmap_bulk_read
> 
>  drivers/iio/accel/mxc4005.c                |  4 +--
>  drivers/iio/chemical/atlas-sensor.c        |  7 ++---
>  drivers/iio/chemical/bme680_core.c         | 36 +++++++++++++---------
>  drivers/iio/imu/inv_mpu6050/inv_mpu_core.c |  4 +--
>  drivers/iio/light/ltr501.c                 |  2 +-
>  drivers/iio/magnetometer/mmc35240.c        |  4 +--
>  6 files changed, 32 insertions(+), 25 deletions(-)
> 




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux