Subject: Re: [PATCH] fsldma: print correct IRQ on mpc83xx

From: Timur Tabi <[email-masked]>
Date: Wed, 14 Jan 2009 10:42:16 -0600

Peter Korsgaard wrote:

> Documentation/powerpc/dts-bindings/fsl/dma.txt and
> The NO_IRQ check in fsldma.c:fsl_dma_chan_probe()
> 
> And it makes sense, there's no per-channel DMAC interrupts on mpc83xx.

But the device trees do have IRQs in the channels:

arch/powerpc/boot/dts/mpc836x_mds.dts:

dma@82a8 {
	...
	interrupt-parent = <&ipic>;
	interrupts = <71 8>;
	dma-channel@0 {
		compatible = "fsl,mpc8360-dma-channel", "fsl,elo-dma-channel";
		reg = <0 0x80>;
		cell-index = <0>;
		interrupt-parent = <&ipic>;
		interrupts = <71 8>;
	};
	...

So I don't see what bug you're trying to fix.  If you're saying that the
interrupts should not be specific DMA channels, then the proper fix is to have
the DMA driver pick up the interrupts from the DMA controller's node when
necessary.  Hacking up a printk doesn't fix anything.

-- 
Timur Tabi
Linux kernel developer at Freescale