diff options
author | Christoph Hellwig <hch@lst.de> | 2019-03-15 12:56:43 -0400 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2019-09-04 05:13:17 -0400 |
commit | 14451467014b4c8aff6570b44b6d0ee68cd49bc0 (patch) | |
tree | 66eec34dfcf1db31d85ebc9b1387e4ec43fe8a58 /arch/arm/mm/dma-mapping.c | |
parent | 392e879a445008e8e96b872dabaaf5b1eca58729 (diff) |
dma-mapping: move the dma_get_sgtable API comments from arm to common code
The comments are spot on and should be near the central API, not just
near a single implementation.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'arch/arm/mm/dma-mapping.c')
-rw-r--r-- | arch/arm/mm/dma-mapping.c | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c index d27b12f61737..0189a757a8f2 100644 --- a/arch/arm/mm/dma-mapping.c +++ b/arch/arm/mm/dma-mapping.c | |||
@@ -877,17 +877,6 @@ static void arm_coherent_dma_free(struct device *dev, size_t size, void *cpu_add | |||
877 | __arm_dma_free(dev, size, cpu_addr, handle, attrs, true); | 877 | __arm_dma_free(dev, size, cpu_addr, handle, attrs, true); |
878 | } | 878 | } |
879 | 879 | ||
880 | /* | ||
881 | * The whole dma_get_sgtable() idea is fundamentally unsafe - it seems | ||
882 | * that the intention is to allow exporting memory allocated via the | ||
883 | * coherent DMA APIs through the dma_buf API, which only accepts a | ||
884 | * scattertable. This presents a couple of problems: | ||
885 | * 1. Not all memory allocated via the coherent DMA APIs is backed by | ||
886 | * a struct page | ||
887 | * 2. Passing coherent DMA memory into the streaming APIs is not allowed | ||
888 | * as we will try to flush the memory through a different alias to that | ||
889 | * actually being used (and the flushes are redundant.) | ||
890 | */ | ||
891 | int arm_dma_get_sgtable(struct device *dev, struct sg_table *sgt, | 880 | int arm_dma_get_sgtable(struct device *dev, struct sg_table *sgt, |
892 | void *cpu_addr, dma_addr_t handle, size_t size, | 881 | void *cpu_addr, dma_addr_t handle, size_t size, |
893 | unsigned long attrs) | 882 | unsigned long attrs) |