gpio: tegra: Use raw_spinlock
authorDmitry Osipenko <digetx@gmail.com>
Wed, 4 Nov 2020 17:04:23 +0000 (20:04 +0300)
committerBartosz Golaszewski <bgolaszewski@baylibre.com>
Fri, 6 Nov 2020 14:30:47 +0000 (15:30 +0100)
commit37174f3341306eaea7b7f4f5cc624e0040305a98
tree66ad37545691e7312cc042e285419ef807c00d9a
parent6ea68fc0a6049b59ff87f58faac5836e293d2801
gpio: tegra: Use raw_spinlock

Use raw_spinlock in order to fix spurious messages about invalid context
when spinlock debugging is enabled. This happens because there is a legit
nested raw_spinlock->spinlock locking usage within IRQ-related code. IRQ
core uses raw spinlock and then Tegra GPIO driver uses a nested spinlock.
The debug code can't recognize and handle this case, hence we need to use
raw spinlock in the GPIO driver.

 [ BUG: Invalid wait context ]
 ...
  (dump_stack) from (__lock_acquire)
  (__lock_acquire) from (lock_acquire)
  (lock_acquire) from (_raw_spin_lock_irqsave)
  (_raw_spin_lock_irqsave) from (tegra_gpio_irq_set_type)
  (tegra_gpio_irq_set_type) from (__irq_set_trigger)
  (__irq_set_trigger) from (__setup_irq)
  (__setup_irq) from (request_threaded_irq)
  (request_threaded_irq) from (devm_request_threaded_irq)
  (devm_request_threaded_irq) from (elants_i2c_probe)
  (elants_i2c_probe) from (i2c_device_probe)
 ...

Tested-by: Peter Geis <pgwipeout@gmail.com>
Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
Signed-off-by: Bartosz Golaszewski <bgolaszewski@baylibre.com>
drivers/gpio/gpio-tegra.c