On 06/29/2012 05:30 AM, Laxman Dewangan wrote:Use the dmaengine based Tegra APB DMA driver for...
apbio access in place of legacy Tegra APB DMA.
The new driver is selected if legacy driver is not
selected and new DMA driver is enabled through config
file.
+bool tegra_apb_dma_init(void)
+{
+ tegra_apb_dma_chan = dma_request_channel(mask, NULL, NULL);That fires quite a few times during boot, before the DMA driver is
+ if (!tegra_apb_dma_chan) {
+ pr_err("%s: can not allocate dma channel\n", __func__);
+ goto err_dma_alloc;
+ }
probed. I think the message should be down-graded to a pr_dbg(). Do you
agree? I can fix that myself when applying this if you're OK with the
change.
Aside from that, this looks fine to me.