On Fri, Oct 15, 2021 at 09:43:32AM -0500, Mario Limonciello wrote: > + if (irq < 0 && !(regval & BIT(WAKE_STS_OFF))) > + return true; Shouldn't be negated. WAKE_STS_OFF is set on wakeup. With that change: Reported-by: Joerie de Gram <j.de.gram@xxxxxxxxx> Tested-by: Joerie de Gram <j.de.gram@xxxxxxxxx>
Re: [PATCH] pinctrl: amd: Fix wakeups when IRQ is shared with SCI
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
- Subject: Re: [PATCH] pinctrl: amd: Fix wakeups when IRQ is shared with SCI
- From: Joerie de Gram <j.de.gram@xxxxxxxxx>
- Date: Sat, 16 Oct 2021 01:53:56 +0200
- In-reply-to: <20211015144332.700-1-mario.limonciello@amd.com>
- References: <20211015144332.700-1-mario.limonciello@amd.com>
- References:
- [PATCH] pinctrl: amd: Fix wakeups when IRQ is shared with SCI
- From: Mario Limonciello
- [PATCH] pinctrl: amd: Fix wakeups when IRQ is shared with SCI
- Prev by Date: Re: [PATCH v5 1/2] dt-bindings: pinctrl: pinctrl-microchip-sgpio: Add reset binding
- Next by Date: [PATCH v3 0/5] pinctrl/GPIO driver for Apple SoCs
- Previous by thread: [PATCH] pinctrl: amd: Fix wakeups when IRQ is shared with SCI
- Next by thread: Re: [PATCH] pinctrl: amd: Fix wakeups when IRQ is shared with SCI
- Index(es):
 |