Skip to content

Commit 03f84b3

Browse files
PCI: dra7xx: Fix threaded IRQ request for "dra7xx-pcie-main" IRQ
Commit da87d35 ("PCI: dra7xx: Use threaded IRQ handler for "dra7xx-pcie-main" IRQ") switched from devm_request_irq() to devm_request_threaded_irq() for the "dra7xx-pcie-main" interrupt. Since the primary handler was set to NULL, the "IRQF_ONESHOT" flag should have also been set. Fix this. Fixes: da87d35 ("PCI: dra7xx: Use threaded IRQ handler for "dra7xx-pcie-main" IRQ") Suggested-by: Vignesh Raghavendra <vigneshr@ti.com> Link: https://lore.kernel.org/linux-pci/20240827122422.985547-2-s-vadapalli@ti.com Reported-by: Udit Kumar <u-kumar1@ti.com> Signed-off-by: Siddharth Vadapalli <s-vadapalli@ti.com> Signed-off-by: Krzysztof Wilczyński <kwilczynski@kernel.org> Reviewed-by: Kevin Hilman <khilman@baylibre.com> Reviewed-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Cc: stable@vger.kernel.org
1 parent 8400291 commit 03f84b3

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

drivers/pci/controller/dwc/pci-dra7xx.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -850,7 +850,8 @@ static int dra7xx_pcie_probe(struct platform_device *pdev)
850850
dra7xx->mode = mode;
851851

852852
ret = devm_request_threaded_irq(dev, irq, NULL, dra7xx_pcie_irq_handler,
853-
IRQF_SHARED, "dra7xx-pcie-main", dra7xx);
853+
IRQF_SHARED | IRQF_ONESHOT,
854+
"dra7xx-pcie-main", dra7xx);
854855
if (ret) {
855856
dev_err(dev, "failed to request irq\n");
856857
goto err_gpio;

0 commit comments

Comments
 (0)