Re: [PATCH v3] iio: magnetometer: rm3100: Modernize locking and refactor control flow

Maxwell Doose
2026-04-29T20:39:02Z

On Wed, Apr 29, 2026 at 2:12 PM Andy Shevchenko
<andriy.shevchenko@intel.com> wrote:
>
> Still the locking wrapper that takes function as a parameter doesn't smell good
> to me. The reason is the same, hiding important details of critical section,
> decreases readability. Also long term maintenance might suffer if one needs to
> alter that piece of code.
>

Good point.

best regards,
maxwell

> --
> With Best Regards,
> Andy Shevchenko
>
>