Skip to content

Commit 8c04b77

Browse files
Fei Shaobroonie
authored andcommitted
spi: mt65xx: Use IRQF_ONESHOT with threaded IRQ
This driver is migrated to use threaded IRQ since commit 5972eb0 ("spi: spi-mt65xx: Use threaded interrupt for non-SPIMEM transfer"), and we almost always want to disable the interrupt line to avoid excess interrupts while the threaded handler is processing SPI transfer. Use IRQF_ONESHOT for that purpose. In practice, we see MediaTek devices show SPI transfer timeout errors when communicating with ChromeOS EC in certain scenarios, and with IRQF_ONESHOT, the issue goes away. Signed-off-by: Fei Shao <fshao@chromium.org> Link: https://patch.msgid.link/20251217101131.1975131-1-fshao@chromium.org Signed-off-by: Mark Brown <broonie@kernel.org>
1 parent 1d24636 commit 8c04b77

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

drivers/spi/spi-mt65xx.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1320,7 +1320,7 @@ static int mtk_spi_probe(struct platform_device *pdev)
13201320

13211321
ret = devm_request_threaded_irq(dev, irq, mtk_spi_interrupt,
13221322
mtk_spi_interrupt_thread,
1323-
IRQF_TRIGGER_NONE, dev_name(dev), host);
1323+
IRQF_ONESHOT, dev_name(dev), host);
13241324
if (ret)
13251325
return dev_err_probe(dev, ret, "failed to register irq\n");
13261326

0 commit comments

Comments
 (0)