[PATCH v4 3/3] dma: Add Freescale eDMA engine driver support

-----Original Message-----
From: Vinod Koul [mailto:vinod.koul at intel.com]
Sent: Monday, September 02, 2013 12:51 PM
To: Lu Jingchang-B35083
Cc: shawn.guo at linaro.org; linux-kernel at vger.kernel.org; linux-arm-
kernel at lists.infradead.org; devicetree at vger.kernel.org
Subject: Re: [PATCH v4 3/3] dma: Add Freescale eDMA engine driver support

On Thu, Aug 29, 2013 at 03:32:04AM +0000, Lu Jingchang-B35083 wrote:

Please use a right MUA and wrap your lines at 80chars...

quoted

quoted

quoted

+			return -EINVAL;
+		}
+		return 0;
+
+	default:
+		return -ENXIO;
+	}
+}
+
+static enum dma_status fsl_edma_tx_status(struct dma_chan *chan,
+		dma_cookie_t cookie, struct dma_tx_state *txstate)
+{
+	struct fsl_edma_chan *fsl_chan = to_fsl_edma_chan(chan);
+
+	if (fsl_chan->status == DMA_ERROR)
+		return DMA_ERROR;
+
+	return dma_cookie_status(chan, cookie, txstate);

this will tell if the DMA is completed or not only.
You also need to calculate residue for the pending dma

Since you support cyclic this should be done properly...

also you cna take more help from vchan support to make your life
simpler...

[Lu Jingchang]

No need to put your name :)

[Lu Jingchang-b35083] 
Aha, the Microsoft Outlook adds this automatically with default option.

quoted

Ok, if it is needed, I will add residue calculation in the next version.

Yes this is needed

quoted

quoted

quoted

+static bool fsl_edma_filter_fn(struct dma_chan *chan, void

*fn_param)

quoted

quoted

quoted

+{
+	struct fsl_edma_filter_param *fparam = fn_param;
+	struct fsl_edma_chan *fsl_chan = to_fsl_edma_chan(chan);
+
+	if (fsl_chan->edmamux->mux_id != fparam->mux_id)
+		return false;
+
+	fsl_chan->slot_id = fparam->slot_id;
+	chan->private = fn_param;

why do you need to use chan->private?

[Lu Jingchang]
The private used here is to store the slot_id information, which must

be used

quoted

by the DMAMUX in alloc_chan_resources function. Thanks.

Why dont you pass this in struct dma_slave_config memeber slave_id for
this.

[Lu Jingchang-b35083] 
I will drop this private and setup the slave_id directly in the filter function.
The slave id is only used in first request phase, if filter is true, the channel
Should be to this slave.
Thanks.







Best Regards,
Jingchang