* [PATCH] dma-mapping: update comment after dmabounce removal
@ 2022-07-20 19:02 ` Lukas Bulwahn
0 siblings, 0 replies; 8+ messages in thread
From: Lukas Bulwahn @ 2022-07-20 19:02 UTC (permalink / raw)
To: Russell King, Christoph Hellwig, Arnd Bergmann, linux-arm-kernel
Cc: kernel-janitors, linux-kernel, Lukas Bulwahn
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(-)
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,
--
2.17.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH] dma-mapping: update comment after dmabounce removal
@ 2022-07-20 19:02 ` Lukas Bulwahn
0 siblings, 0 replies; 8+ messages in thread
From: Lukas Bulwahn @ 2022-07-20 19:02 UTC (permalink / raw)
To: Russell King, Christoph Hellwig, Arnd Bergmann, linux-arm-kernel
Cc: kernel-janitors, linux-kernel, Lukas Bulwahn
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(-)
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,
--
2.17.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH] dma-mapping: update comment after dmabounce removal
2022-07-20 19:02 ` Lukas Bulwahn
@ 2022-07-22 12:50 ` Arnd Bergmann
-1 siblings, 0 replies; 8+ messages in thread
From: Arnd Bergmann @ 2022-07-22 12:50 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Russell King, Christoph Hellwig, Arnd Bergmann, Linux ARM,
kernel-janitors, Linux Kernel Mailing List
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
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dma-mapping: update comment after dmabounce removal
@ 2022-07-22 12:50 ` Arnd Bergmann
0 siblings, 0 replies; 8+ messages in thread
From: Arnd Bergmann @ 2022-07-22 12:50 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Russell King, Christoph Hellwig, Arnd Bergmann, Linux ARM,
kernel-janitors, Linux Kernel Mailing List
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
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dma-mapping: update comment after dmabounce removal
2022-07-22 12:50 ` Arnd Bergmann
@ 2022-07-22 15:04 ` Christoph Hellwig
-1 siblings, 0 replies; 8+ messages in thread
From: Christoph Hellwig @ 2022-07-22 15:04 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Lukas Bulwahn, Russell King, Christoph Hellwig, Linux ARM,
kernel-janitors, Linux Kernel Mailing List
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.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dma-mapping: update comment after dmabounce removal
@ 2022-07-22 15:04 ` Christoph Hellwig
0 siblings, 0 replies; 8+ messages in thread
From: Christoph Hellwig @ 2022-07-22 15:04 UTC (permalink / raw)
To: Arnd Bergmann
Cc: Lukas Bulwahn, Russell King, Christoph Hellwig, Linux ARM,
kernel-janitors, Linux Kernel Mailing List
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.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dma-mapping: update comment after dmabounce removal
2022-07-20 19:02 ` Lukas Bulwahn
@ 2022-07-22 15:15 ` Christoph Hellwig
-1 siblings, 0 replies; 8+ messages in thread
From: Christoph Hellwig @ 2022-07-22 15:15 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Russell King, Christoph Hellwig, Arnd Bergmann, linux-arm-kernel,
kernel-janitors, linux-kernel
Thanks, applied.
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] dma-mapping: update comment after dmabounce removal
@ 2022-07-22 15:15 ` Christoph Hellwig
0 siblings, 0 replies; 8+ messages in thread
From: Christoph Hellwig @ 2022-07-22 15:15 UTC (permalink / raw)
To: Lukas Bulwahn
Cc: Russell King, Christoph Hellwig, Arnd Bergmann, linux-arm-kernel,
kernel-janitors, linux-kernel
Thanks, applied.
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2022-07-22 15:16 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-20 19:02 [PATCH] dma-mapping: update comment after dmabounce removal Lukas Bulwahn
2022-07-20 19:02 ` Lukas Bulwahn
2022-07-22 12:50 ` Arnd Bergmann
2022-07-22 12:50 ` Arnd Bergmann
2022-07-22 15:04 ` Christoph Hellwig
2022-07-22 15:04 ` Christoph Hellwig
2022-07-22 15:15 ` Christoph Hellwig
2022-07-22 15:15 ` Christoph Hellwig
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.