Re: [PATCH] iio: proximity: sx9310: keep the IRQ balanced across suspend and resume

From: Krzysztof Kozlowski

Date: Tue Oct 27 2026 - 11:36:43 EST


On 26/10/2026 11:03, Arnav Kapoor wrote:
> sx9310_suspend() disables the interrupt first and returns with it still
> disabled if a register access fails. Resume isn't called after a failed
> suspend, so the interrupt is never enabled again. sx9310_resume() also
> returns without enabling it on error. Without an interrupt line,
> client->irq is 0 and both callbacks act on IRQ 0, which the driver
> doesn't own.
>
> Disable the interrupt only once the device is asleep, always enable it
> in resume, and skip both when there is no interrupt.
>
> Fixes: 72ad02b15d63 ("iio: Add SEMTECH SX9310/9311 sensor driver")
> Signed-off-by: Arnav Kapoor <kapoorarnav43@xxxxxxxxx>
> ---
> I'm sending the same fix for sx9310, sx9324 and sx9360 as separate
> patches, as the Fixes tags differ.
>

Please group them in patchset to make reviewing and applying easier. Or
rejecting, or even commenting that something is wrong. Previously you
were sending AI slop and this resembles recent AI-bot spamming.

Best regards,
Krzysztof