RE: [PATCH v3 1/6] pinctrl: renesas: rzg2l: Use raw_spinlock_irqsave() on power source update
From: Biju Das
Date: Thu Jun 04 2026 - 07:03:37 EST
Hi Geert/Claudiu,
> -----Original Message-----
> From: Geert Uytterhoeven <geert@xxxxxxxxxxxxxx>
> Sent: 04 June 2026 10:49
> Subject: Re: [PATCH v3 1/6] pinctrl: renesas: rzg2l: Use raw_spinlock_irqsave() on power source update
>
> On Wed, 3 Jun 2026 at 17:17, Claudiu Beznea <claudiu.beznea@xxxxxxxxxx> wrote:
> > From: Claudiu Beznea <claudiu.beznea.uj@xxxxxxxxxxxxxx>
> >
> > The rest of the driver uses
> > raw_spin_lock_irqsave()/raw_spin_unlock_irqrestore() for locking. To
> > avoid concurrency issues or deadlocks, use raw_spinlock_irqsave() via
> > the scoped_guard() helper for power source updates as well.
Just a question, will rzg2l_set_power_source() called from IRQ context?
This driver does not have IRQ. If any consumer calls rzg2l_set_power_source()
in IRQ contest?
Have we seen any such dead locks/concurrency issue during any testing?
Cheers,
Biju