In preparation for enabling loadable module support for the driver, there is motivation to switch to devm_request_irq() to simplify the cleanup on driver removal. Additionally, since the interrupt handler associated with the "ks-pcie-error-irq" namely "ks_pcie_handle_error_irq() is only printing the error and is clearing the interrupt, there is no necessity to prefer devm_request_threaded_irq() over devm_request_irq(). Hence, switch from request_irq() to devm_request_irq(). Signed-off-by: Siddharth Vadapalli <s-vadapalli@xxxxxx> --- drivers/pci/controller/dwc/pci-keystone.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/pci/controller/dwc/pci-keystone.c b/drivers/pci/controller/dwc/pci-keystone.c index bb93559f6468..02f9a6d0e4a8 100644 --- a/drivers/pci/controller/dwc/pci-keystone.c +++ b/drivers/pci/controller/dwc/pci-keystone.c @@ -1277,8 +1277,8 @@ static int ks_pcie_probe(struct platform_device *pdev) if (irq < 0) return irq; - ret = request_irq(irq, ks_pcie_err_irq_handler, IRQF_SHARED, - "ks-pcie-error-irq", ks_pcie); + ret = devm_request_irq(dev, irq, ks_pcie_err_irq_handler, IRQF_SHARED, + "ks-pcie-error-irq", ks_pcie); if (ret < 0) { dev_err(dev, "failed to request error IRQ %d\n", irq); -- 2.43.0