Shawn Guo wrote:
- if (of_device_is_compatible(pdev->dev.of_node, "fsl,imx1-ssi")) {
I wonder if the i.MX code should be protected by an #ifdef. There's no point in compiling this code on PowerPC.
/** FIXME: The dma burst size must equal to ssi watermark* setting on imx. We have burstsize be "fifo_depth - 2"* here because "fifo_depth - 2" rather than fifo_depth is* being written into watermark register in fsl_ssi_startup().* Is this something needs to be fixed for PowerPC?
Setting the watermark to fifo_depth-2 allows the SSI to continue storing data into the FIFO (during capture) in the case the DMA engine isn't fast enough. I don't want the FIFO to be completely full before we start the DMA.
For playback, it's the same thing. I can't be sure the FIFO is completely empty when it's time to DMA more data.
*/ssi_private->dma_params_tx.burstsize =ssi_private->fifo_depth - 2;ssi_private->dma_params_rx.burstsize =ssi_private->fifo_depth - 2;ssi_private->dma_params_tx.dma_addr =ssi_private->ssi_phys + offsetof(struct ccsr_ssi, stx0);ssi_private->dma_params_rx.dma_addr =ssi_private->ssi_phys + offsetof(struct ccsr_ssi, srx0);
So I'm a little confused by this new binding. I went through a lot of work to remove DMA-specific data from the SSI driver, and here you are adding a bunch of it back. What you're doing here -- I'm doing it in the DMA driver where it belongs.
/** TODO: This is a temporary solution and should be changed* to use generic DMA binding later when the helplers get in.*/ret = of_property_read_u32_array(pdev->dev.of_node,"fsl,ssi-dma-events", dma_events, 2);if (ret) {dev_err(&pdev->dev, "could not get dma events\n");goto error_irq;}ssi_private->dma_params_tx.dma = dma_events[0];ssi_private->dma_params_rx.dma = dma_events[1];
of_property_read_u32_array seems overkill for just two integers.
ssi_private->dma_params_tx.dma = be32_to_cpu(prop[0]); ssi_private->dma_params_rx.dma = be32_to_cpu(prop[1]);