* [PATCH] dmaengine: pxa_dma: use platform_get_irq_optional
@ 2022-09-06 0:07 Doug Brown
2022-10-19 14:26 ` Vinod Koul
0 siblings, 1 reply; 2+ messages in thread
From: Doug Brown @ 2022-09-06 0:07 UTC (permalink / raw)
To: Daniel Mack, Haojian Zhuang, Robert Jarzmik, Vinod Koul
Cc: linux-arm-kernel, dmaengine, Doug Brown
The first IRQ is required, but IRQs 1 through (nb_phy_chans - 1) are
optional, because on some platforms (e.g. PXA168) there is a single IRQ
shared between all channels.
This change inhibits a flood of "IRQ index # not found" messages at
startup. Tested on a PXA168-based device.
Fixes: 7723f4c5ecdb ("driver core: platform: Add an error message to platform_get_irq*()"
Signed-off-by: Doug Brown <doug@schmorgal.com>
---
drivers/dma/pxa_dma.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/dma/pxa_dma.c b/drivers/dma/pxa_dma.c
index e7034f6f3994..22a392fe6d32 100644
--- a/drivers/dma/pxa_dma.c
+++ b/drivers/dma/pxa_dma.c
@@ -1247,14 +1247,14 @@ static int pxad_init_phys(struct platform_device *op,
return -ENOMEM;
for (i = 0; i < nb_phy_chans; i++)
- if (platform_get_irq(op, i) > 0)
+ if (platform_get_irq_optional(op, i) > 0)
nr_irq++;
for (i = 0; i < nb_phy_chans; i++) {
phy = &pdev->phys[i];
phy->base = pdev->base;
phy->idx = i;
- irq = platform_get_irq(op, i);
+ irq = platform_get_irq_optional(op, i);
if ((nr_irq > 1) && (irq > 0))
ret = devm_request_irq(&op->dev, irq,
pxad_chan_handler,
--
2.25.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 2+ messages in thread* Re: [PATCH] dmaengine: pxa_dma: use platform_get_irq_optional
2022-09-06 0:07 [PATCH] dmaengine: pxa_dma: use platform_get_irq_optional Doug Brown
@ 2022-10-19 14:26 ` Vinod Koul
0 siblings, 0 replies; 2+ messages in thread
From: Vinod Koul @ 2022-10-19 14:26 UTC (permalink / raw)
To: Doug Brown
Cc: Daniel Mack, Haojian Zhuang, Robert Jarzmik, linux-arm-kernel,
dmaengine
On 05-09-22, 17:07, Doug Brown wrote:
> The first IRQ is required, but IRQs 1 through (nb_phy_chans - 1) are
> optional, because on some platforms (e.g. PXA168) there is a single IRQ
> shared between all channels.
>
> This change inhibits a flood of "IRQ index # not found" messages at
> startup. Tested on a PXA168-based device.
Applied, thanks
--
~Vinod
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-10-19 14:50 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-09-06 0:07 [PATCH] dmaengine: pxa_dma: use platform_get_irq_optional Doug Brown
2022-10-19 14:26 ` Vinod Koul
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox