Re: [RFC PATCH] gpiolib: remove extra_checks
From: Andy Shevchenko
Date: Wed Dec 20 2023 - 09:04:08 EST
On Tue, Dec 19, 2023 at 09:11:02PM +0100, Bartosz Golaszewski wrote:
> From: Bartosz Golaszewski <bartosz.golaszewski@xxxxxxxxxx>
>
> extra_checks is only used in a few places. It also depends on
> a non-standard DEBUG define one needs to add to the source file.
Huh?!
What then CONFIG_DEBUG_GPIO is about?
> The
> overhead of removing it should be minimal (we already use pure
> might_sleep() in the code anyway) so drop it.
...
I agree on might_sleep() changes, but WARN_ON(), see above why.
--
With Best Regards,
Andy Shevchenko