Re: [PATCH v4 0/8] iio: accel: convert to guard(mutex)

[Jonathan Cameron <jic23@kernel.org>]

On Sat, 14 Mar 2026 22:36:01 +0530
Rajveer Chaudhari <rajveer.chaudhari.linux@gmail.com> wrote:

> This series converts manual mutex_lock/mutex_unlock pairs to
> guard(mutex) in two ADXL accelerometer drivers. Each conversion

No it doesn't...

> also simplifies error handling by removing goto labels and
> returning directly on error paths.
> 
> v4 : Sort Headers alphabetically
> 
> Link to v3: https://lore.kernel.org/all/20260312081942.23858-1-
> rajveer.chaudhari.linux@gmail.com/

As per the thread on doing this stuff as a GSOC project, 
please don't do too many of these.  I don't mind people doing
one or two as a 'getting started' project but they do take non
trivial review and don't necessarily give sufficient benefit.

So on that note, maybe move on to other topics after this set!
I am in favour of these as part of more significant driver
improvement series though!

> 
> Rajveer Chaudhari (8):
>   iio: accel: bmc150: sort headers alphabetically
>   iio: accel: bmc150: convert to guard(mutex)
>   iio: accel: mma8452: sort headers alphabetically
>   iio: accel: mma8452: convert to guard(mutex)
>   iio: accel: mma9551: sort headers alphabetically
>   iio: accel: mma9551: convert to guard(mutex)
>   iio: accel: sca3000: sort headers alphabetically
>   iio: accel: sca3000: convert to guard(mutex)
> 
>  drivers/iio/accel/bmc150-accel-core.c |  38 +++----
>  drivers/iio/accel/mma8452.c           |  38 +++----
>  drivers/iio/accel/mma9551.c           |  19 ++--
>  drivers/iio/accel/sca3000.c           | 157 ++++++++++++--------------
>  4 files changed, 115 insertions(+), 137 deletions(-)
>