Thread (10 messages) 10 messages, 2 authors, 2017-01-25
STALE3465d REVIEWED: 1 (0M)

Re: [PATCH v6 2/4] dmaengine: Forward slave device pointer to of_xlate callback

From: Ulf Hansson <hidden>
Date: 2017-01-24 15:09:15
Also in: linux-arm-kernel, linux-samsung-soc

On 24 January 2017 at 10:27, Marek Szyprowski [off-list ref] wrote:
Add pointer to slave device to of_dma_xlate to let DMA engine driver
to know which slave device is using given DMA channel. This will be
later used to implement non-irq-safe runtime PM for DMA engine driver.

Signed-off-by: Marek Szyprowski <m.szyprowski@samsung.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
[...]
quoted hunk ↗ jump to hunk
diff --git a/include/linux/of_dma.h b/include/linux/of_dma.h
index b90d8ec57c1f..a0a6c8c17669 100644
--- a/include/linux/of_dma.h
+++ b/include/linux/of_dma.h
@@ -22,7 +22,8 @@ struct of_dma {
        struct list_head        of_dma_controllers;
        struct device_node      *of_node;
        struct dma_chan         *(*of_dma_xlate)
-                               (struct of_phandle_args *, struct of_dma *);
+                               (struct of_phandle_args *, struct of_dma *,
+                                struct device *);
        void                    *(*of_dma_route_allocate)
                                (struct of_phandle_args *, struct of_dma *);
        struct dma_router       *dma_router;
@@ -37,7 +38,7 @@ struct of_dma_filter_info {
 #ifdef CONFIG_DMA_OF
 extern int of_dma_controller_register(struct device_node *np,
                struct dma_chan *(*of_dma_xlate)
-               (struct of_phandle_args *, struct of_dma *),
+               (struct of_phandle_args *, struct of_dma *, struct device *),
                void *data);
 extern void of_dma_controller_free(struct device_node *np);
@@ -47,17 +48,17 @@ extern int of_dma_router_register(struct device_node *np,
                struct dma_router *dma_router);
 #define of_dma_router_free of_dma_controller_free

-extern struct dma_chan *of_dma_request_slave_channel(struct device_node *np,
+extern struct dma_chan *of_dma_request_slave_channel(struct device *slave,
                                                     const char *name);
I noticed that this API is being used from sound/soc/sh/rcar/dma.c, so
I assume this change triggers a compiler error. I guess you want to
fold in a change dealing with that as well.

[...]

Reviewed-by: Ulf Hansson <redacted>

Kind regards
Uffe
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help