lkml.org 
[lkml]   [2026]   [Apr]   [29]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v3] iio: magnetometer: rm3100: Modernize locking and refactor control flow
On Wed, Apr 29, 2026 at 1:26 PM Andy Shevchenko
<andriy.shevchenko@intel.com> wrote:
>
> Won't fly. I am pretty sure most of the maintainers find that bad. The problem
> is that the caller should be crystal clear to show what the critical section
> is. Hiding that information behind macros drastically reduces understanding of
> the code,
>

Was just a thought. Even if it were to be implemented, it certainly
wouldn't be a macro.

best regards,
maxwell




>
> > I think that would be sound
> > for a separate patch. I'd ordinarily include that in this patch, but I
> > feel at that point we might be deviating from the original purpose of
> > the patch.
>
> I agree.
>
> --
> With Best Regards,
> Andy Shevchenko
>
>

\
 
 \ /
  Last update: 2026-04-29 21:00    [W:0.087 / U:2.968 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and my Meterkast|Read the blog