Hi Emanuele,
On Mon, 12 May 2025 at 11:38, Emanuele Ghidoli ghidoliemanuele@gmail.com wrote:
On 12/05/2025 11:23, Andy Shevchenko wrote:
On Mon, May 12, 2025 at 11:17:48AM +0200, Geert Uytterhoeven wrote:
On Fri, 9 May 2025 at 16:18, Francesco Dolcini francesco@dolcini.it wrote:
An irq can be disabled with disable_irq() and still wake the system as long as the irq has wake enabled, so the wake-up functionality is preserved.
...
While this does not cause the regression seen on Salvator-XS with the earlier approach[1], I expect this will break using a GPIO as a wake-up source?
Good point! Have this code been checked for that kind of scenarios?
[1] https://lore.kernel.org/linux-gpio/CAMuHMdVnKX23yi7ir1LVxfXAMeeWMFzM+cdgSSTN...
Yes, I tested this specific scenario with its GPIOs as wake-up sources, and it worked as expected. I already included the note in the commit message.
Sorry, I missed that.
Then I have no objections from my side.
Tested-by: Geert Uytterhoeven geert+renesas@glider.be
Gr{oetje,eeting}s,
Geert