diff options
author | Daniel Baumann <daniel.baumann@progress-linux.org> | 2024-06-19 21:00:54 +0000 |
---|---|---|
committer | Daniel Baumann <daniel.baumann@progress-linux.org> | 2024-06-19 21:01:03 +0000 |
commit | 62db400e83102695233497884d16a19c7e34fcd4 (patch) | |
tree | 62d1c1a2105118357e772c53c56478baf87bf946 /debian/patches-rt/pinctrl-renesas-rzg2l-Use-spin_-lock-unlock-_irq-sav.patch | |
parent | Merging upstream version 6.9.2. (diff) | |
download | linux-62db400e83102695233497884d16a19c7e34fcd4.tar.xz linux-62db400e83102695233497884d16a19c7e34fcd4.zip |
Merging debian version 6.9.2-1~exp1.
Signed-off-by: Daniel Baumann <daniel.baumann@progress-linux.org>
Diffstat (limited to 'debian/patches-rt/pinctrl-renesas-rzg2l-Use-spin_-lock-unlock-_irq-sav.patch')
-rw-r--r-- | debian/patches-rt/pinctrl-renesas-rzg2l-Use-spin_-lock-unlock-_irq-sav.patch | 38 |
1 files changed, 38 insertions, 0 deletions
diff --git a/debian/patches-rt/pinctrl-renesas-rzg2l-Use-spin_-lock-unlock-_irq-sav.patch b/debian/patches-rt/pinctrl-renesas-rzg2l-Use-spin_-lock-unlock-_irq-sav.patch new file mode 100644 index 0000000000..3462fa28ef --- /dev/null +++ b/debian/patches-rt/pinctrl-renesas-rzg2l-Use-spin_-lock-unlock-_irq-sav.patch @@ -0,0 +1,38 @@ +From: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> +Date: Wed, 22 May 2024 08:54:21 +0300 +Subject: [PATCH] pinctrl: renesas: rzg2l: Use + spin_{lock,unlock}_irq{save,restore} +Origin: https://www.kernel.org/pub/linux/kernel/projects/rt/6.10/older/patches-6.10-rc1-rt1.tar.xz + +On PREEMPT_RT kernels the spinlock_t maps to an rtmutex. Using +raw_spin_lock_irqsave()/raw_spin_unlock_irqrestore() on +&pctrl->lock.rlock breaks the PREEMPT_RT builds. To fix this use +spin_lock_irqsave()/spin_unlock_irqrestore() on &pctrl->lock. + +Fixes: 02cd2d3be1c3 ("pinctrl: renesas: rzg2l: Configure the interrupt type on resume") +Reported-by: Diederik de Haas <didi.debian@cknow.org> +Closes: https://lore.kernel.org/all/131999629.KQPSlr0Zke@bagend +Signed-off-by: Claudiu Beznea <claudiu.beznea.uj@bp.renesas.com> +Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be> +Link: https://lore.kernel.org/r/20240522055421.2842689-1-claudiu.beznea.uj@bp.renesas.com +Signed-off-by: Sebastian Andrzej Siewior <bigeasy@linutronix.de> +--- + drivers/pinctrl/renesas/pinctrl-rzg2l.c | 4 ++-- + 1 file changed, 2 insertions(+), 2 deletions(-) + +--- a/drivers/pinctrl/renesas/pinctrl-rzg2l.c ++++ b/drivers/pinctrl/renesas/pinctrl-rzg2l.c +@@ -2069,11 +2069,11 @@ static void rzg2l_gpio_irq_restore(struc + * This has to be atomically executed to protect against a concurrent + * interrupt. + */ +- raw_spin_lock_irqsave(&pctrl->lock.rlock, flags); ++ spin_lock_irqsave(&pctrl->lock, flags); + ret = rzg2l_gpio_irq_set_type(data, irqd_get_trigger_type(data)); + if (!ret && !irqd_irq_disabled(data)) + rzg2l_gpio_irq_enable(data); +- raw_spin_unlock_irqrestore(&pctrl->lock.rlock, flags); ++ spin_unlock_irqrestore(&pctrl->lock, flags); + + if (ret) + dev_crit(pctrl->dev, "Failed to set IRQ type for virq=%u\n", virq); |