diff mbox series

dma-mapping: update comment after dmabounce removal

Message ID 20220720190208.11270-1-lukas.bulwahn@gmail.com (mailing list archive)
State New, archived
Headers show
Series dma-mapping: update comment after dmabounce removal | expand

Commit Message

Lukas Bulwahn July 20, 2022, 7:02 p.m. UTC
Commit e3217540c271 ("ARM/dma-mapping: remove dmabounce") removes the
config DMABOUNCE. A comment to the function __dma_page_cpu_to_dev() refers
to this removed config DMABOUNCE.

Remove the obsolete explanation, but keep the recommendation not to use
__dma_page_cpu_to_dev() and use dma_sync_* functions instead.

Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
---
 arch/arm/mm/dma-mapping.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

Comments

Arnd Bergmann July 22, 2022, 12:50 p.m. UTC | #1
On Wed, Jul 20, 2022 at 9:02 PM Lukas Bulwahn <lukas.bulwahn@gmail.com> wrote:
>
> Commit e3217540c271 ("ARM/dma-mapping: remove dmabounce") removes the
> config DMABOUNCE. A comment to the function __dma_page_cpu_to_dev() refers
> to this removed config DMABOUNCE.
>
> Remove the obsolete explanation, but keep the recommendation not to use
> __dma_page_cpu_to_dev() and use dma_sync_* functions instead.
>
> Signed-off-by: Lukas Bulwahn <lukas.bulwahn@gmail.com>
> ---

Thanks!

Acked-by: Arnd Bergmann <arnd@arndb.de>

The e3217540c271 commit is only in Christoph's dma-mapping tree, so this would
ideally get applied on top before the merge window.

        Arnd
Christoph Hellwig July 22, 2022, 3:04 p.m. UTC | #2
On Fri, Jul 22, 2022 at 02:50:20PM +0200, Arnd Bergmann wrote:
> The e3217540c271 commit is only in Christoph's dma-mapping tree, so this would
> ideally get applied on top before the merge window.

I'll pick it up.
Christoph Hellwig July 22, 2022, 3:15 p.m. UTC | #3
Thanks, applied.
diff mbox series

Patch

diff --git a/arch/arm/mm/dma-mapping.c b/arch/arm/mm/dma-mapping.c
index e68d1d2ac4be..16c2cdd04639 100644
--- a/arch/arm/mm/dma-mapping.c
+++ b/arch/arm/mm/dma-mapping.c
@@ -679,8 +679,7 @@  static void dma_cache_maint_page(struct page *page, unsigned long offset,
 
 /*
  * Make an area consistent for devices.
- * Note: Drivers should NOT use this function directly, as it will break
- * platforms with CONFIG_DMABOUNCE.
+ * Note: Drivers should NOT use this function directly.
  * Use the driver DMA support - see dma-mapping.h (dma_sync_*)
  */
 static void __dma_page_cpu_to_dev(struct page *page, unsigned long off,