gpio: don't set label from irq helpers
authorBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Mon, 29 Jan 2024 09:23:27 +0000 (10:23 +0100)
committerBartosz Golaszewski <bartosz.golaszewski@linaro.org>
Mon, 12 Feb 2024 09:50:34 +0000 (10:50 +0100)
commitccfb6ff4f6c0574e01fb16934fb60a46285c5f3f
tree393688dcae1123fbcbcb76d109219531f4519f89
parentd23dc4a9a88f16d0327e7b4bb03e6f43c1a1b166
gpio: don't set label from irq helpers

We will soon serialize access to the descriptor label using SRCU. The
write-side of the protection will require calling synchronize_srcu()
which must not be called from atomic context. We have two irq helpers:
gpiochip_lock_as_irq() and gpiochip_unlock_as_irq() that set the label
if the GPIO is not requested but is being used as interrupt. They are
called with a spinlock held from the interrupt subsystem.

They must not do it if we are to use SRCU so instead let's move the
special corner case to a dedicated getter.

Don't actually set the label to "interrupt" in the above case but rather
use the newly added gpiod_get_label() helper to hide the logic that
atomically checks the descriptor flags and returns the address of a
static "interrupt" string.

Signed-off-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Acked-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
drivers/gpio/gpiolib.c