[<prev] [next>] [day] [month] [year] [list]
Message-ID: <20101005100649.GL28242@pengutronix.de>
Date: Tue, 5 Oct 2010 12:06:49 +0200
From: Sascha Hauer <s.hauer@...gutronix.de>
To: javier Martin <javier.martin@...ta-silicon.com>
Cc: linux-kernel@...r.kernel.org,
Dan Williams <dan.j.williams@...el.com>,
linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH 4/4] dmaengine: Add Freescale i.MX1/21/27 DMA driver
Hi Javier,
On Tue, Oct 05, 2010 at 09:45:10AM +0200, javier Martin wrote:
> Hi Sascha,
>
>
> +
> > +struct imxdma_channel {
> > + struct imxdma_engine *imxdma;
> > + unsigned int channel;
> > + unsigned int imxdma_channel;
> > +
> > + enum dma_data_direction direction;
> > + enum dma_slave_buswidth word_size;
> > + dma_addr_t bd_phys;
> > + unsigned long flags;
> > + dma_addr_t per_address;
> > + u32 watermark_level;
> > + struct dma_chan chan;
> > + spinlock_t lock;
> > + struct dma_async_tx_descriptor desc;
> > + dma_cookie_t last_completed;
> > + enum dma_status status;
> > + int dma_request;
> > + struct scatterlist *sg_list;
> > +};
> > +
> > +#define MAX_DMA_CHANNELS 8
> >
>
> i.MX27 has 16 DMA channels. Maybe an #ifdef would solve the issue.
This is by intention. I want to reserve 8 channels for the old driver
and the remaining 8 for the dmaengine driver. This way we can have both
in the tree and use both drivers parallel. This can be changed to the
real value once the old driver is gone.
Sascha
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists