diff options
author | Bartosz Golaszewski <bartosz.golaszewski@linaro.org> | 2024-01-29 10:23:27 +0100 |
---|---|---|
committer | Bartosz Golaszewski <bartosz.golaszewski@linaro.org> | 2024-02-12 10:50:34 +0100 |
commit | ccfb6ff4f6c0574e01fb16934fb60a46285c5f3f (patch) | |
tree | 393688dcae1123fbcbcb76d109219531f4519f89 /lib | |
parent | d23dc4a9a88f16d0327e7b4bb03e6f43c1a1b166 (diff) |
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>
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions