[PATCH] ARM: add PrimeCell generic DMA to MMCI/PL180
Linus Walleij
linus.ml.walleij at gmail.com
Wed Dec 22 16:55:38 EST 2010
2010/12/21 Russell King - ARM Linux <linux at arm.linux.org.uk>:
> This also contains a few other tweaks on top of your original patch too.
> All the DMA conditionals get optimized out when we're not building with
> a DMA engine enabled platform.
Thanks.
> Note that in your original code, mmci_dma_data_error() is always followed
> by a call to mmci_dma_data_end(), and that resulted in dma_unmap_sg()
> being called twice for the same buffer.
Ulf Hansson notified me on this too, last week or so.
Sorry for not fixing in time.
> Also, dma_map_sg() returns the number of physical entries mapped, or zero
> on failure. Should we have a DMA mapping implementation which coalesces
> entries, the returned number will be less than data->nr_sg, and it is the
> number of coalesced entries which should be passed to the DMA engine driver.
Per Förlin reported this one as well.
Sorry again.
> You have this bug in the PL022 driver too, so I suspect that your other
> drivers need fixing for this.
Fixing patches for PL022 now, thanks a lot Russell.
Yours,
Linus Walleij
More information about the linux-arm-kernel
mailing list