Re: [PATCH v10 4/4] iio: pressure: dps310: assert the lock at runtime too

From: Andy Shevchenko

Date: Wed Sep 30 2026 - 08:08:13 EST


On Wed, Sep 30, 2026 at 01:01:58PM +0300, Rupesh Majhi wrote:
> __must_hold() is clang only and does nothing in GCC builds. Add
> lockdep_assert_held() next to each, so lockdep catches a missing lock
> too.

> Suggested-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxx>

Not really, I just explained the difference and asked if lockdep annotations
would be also good to have. Otherwise if Jonathan thinks it's fine, I have
no objections (and also have no objections on not taking this change).

--
With Best Regards,
Andy Shevchenko