Skip to content

Commit

Permalink
dw_dmac: check for mapping errors
Browse files Browse the repository at this point in the history
Otherwise we get a warning in case of CONFIG_DMA_API_DEBUG=y

[   45.775943] WARNING: at lib/dma-debug.c:933 check_unmap+0x5d6/0x6ac()
[   45.782369] dw_dmac dw_dmac.0: DMA-API: device driver failed to check map error[device address=0x00000000356efcc0] [size=28 bytes] [mapped as single]

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Vinod Koul <vinod.koul@intel.com>
  • Loading branch information
Andy Shevchenko authored and Vinod Koul committed Jan 9, 2013
1 parent 123de54 commit cbd6531
Showing 1 changed file with 14 additions and 6 deletions.
20 changes: 14 additions & 6 deletions drivers/dma/dw_dmac.c
Original file line number Diff line number Diff line change
Expand Up @@ -1087,6 +1087,7 @@ static int dwc_alloc_chan_resources(struct dma_chan *chan)
struct dw_desc *desc;
int i;
unsigned long flags;
int ret;

dev_vdbg(chan2dev(chan), "%s\n", __func__);

Expand All @@ -1110,19 +1111,19 @@ static int dwc_alloc_chan_resources(struct dma_chan *chan)
spin_unlock_irqrestore(&dwc->lock, flags);

desc = kzalloc(sizeof(struct dw_desc), GFP_KERNEL);
if (!desc) {
dev_info(chan2dev(chan),
"only allocated %d descriptors\n", i);
spin_lock_irqsave(&dwc->lock, flags);
break;
}
if (!desc)
goto err_desc_alloc;

INIT_LIST_HEAD(&desc->tx_list);
dma_async_tx_descriptor_init(&desc->txd, chan);
desc->txd.tx_submit = dwc_tx_submit;
desc->txd.flags = DMA_CTRL_ACK;
desc->txd.phys = dma_map_single(chan2parent(chan), &desc->lli,
sizeof(desc->lli), DMA_TO_DEVICE);
ret = dma_mapping_error(chan2parent(chan), desc->txd.phys);
if (ret)
goto err_desc_alloc;

dwc_desc_put(dwc, desc);

spin_lock_irqsave(&dwc->lock, flags);
Expand All @@ -1133,6 +1134,13 @@ static int dwc_alloc_chan_resources(struct dma_chan *chan)

dev_dbg(chan2dev(chan), "%s: allocated %d descriptors\n", __func__, i);

return i;

err_desc_alloc:
kfree(desc);

dev_info(chan2dev(chan), "only allocated %d descriptors\n", i);

return i;
}

Expand Down

0 comments on commit cbd6531

Please sign in to comment.