* [PATCH v11 1/9] dt-bindings: dmaengine: Introduce RZN1 dmamux bindings
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-21 8:51 ` [PATCH v11 2/9] dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode Miquel Raynal
` (8 subsequent siblings)
9 siblings, 0 replies; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree, Geert Uytterhoeven
The Renesas RZN1 DMA IP is based on a DW core, with eg. an additional
dmamux register located in the system control area which can take up to
32 requests (16 per DMA controller). Each DMA channel can be wired to
two different peripherals.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Reviewed-by: Rob Herring <robh@kernel.org>
Acked-By: Vinod Koul <vkoul@kernel.org>
---
.../bindings/dma/renesas,rzn1-dmamux.yaml | 51 +++++++++++++++++++
MAINTAINERS | 1 +
2 files changed, 52 insertions(+)
create mode 100644 Documentation/devicetree/bindings/dma/renesas,rzn1-dmamux.yaml
diff --git a/Documentation/devicetree/bindings/dma/renesas,rzn1-dmamux.yaml b/Documentation/devicetree/bindings/dma/renesas,rzn1-dmamux.yaml
new file mode 100644
index 000000000000..d83013b0dd74
--- /dev/null
+++ b/Documentation/devicetree/bindings/dma/renesas,rzn1-dmamux.yaml
@@ -0,0 +1,51 @@
+# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
+%YAML 1.2
+---
+$id: http://devicetree.org/schemas/dma/renesas,rzn1-dmamux.yaml#
+$schema: http://devicetree.org/meta-schemas/core.yaml#
+
+title: Renesas RZ/N1 DMA mux
+
+maintainers:
+ - Miquel Raynal <miquel.raynal@bootlin.com>
+
+allOf:
+ - $ref: "dma-router.yaml#"
+
+properties:
+ compatible:
+ const: renesas,rzn1-dmamux
+
+ reg:
+ maxItems: 1
+ description: DMA mux first register offset within the system control parent.
+
+ '#dma-cells':
+ const: 6
+ description:
+ The first four cells are dedicated to the master DMA controller. The fifth
+ cell gives the DMA mux bit index that must be set starting from 0. The
+ sixth cell gives the binary value that must be written there, ie. 0 or 1.
+
+ dma-masters:
+ minItems: 1
+ maxItems: 2
+
+ dma-requests:
+ const: 32
+
+required:
+ - reg
+ - dma-requests
+
+additionalProperties: false
+
+examples:
+ - |
+ dma-router@a0 {
+ compatible = "renesas,rzn1-dmamux";
+ reg = <0xa0 4>;
+ #dma-cells = <6>;
+ dma-masters = <&dma0 &dma1>;
+ dma-requests = <32>;
+ };
diff --git a/MAINTAINERS b/MAINTAINERS
index fd768d43e048..120d3ae57a4b 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -19040,6 +19040,7 @@ SYNOPSYS DESIGNWARE DMAC DRIVER
M: Viresh Kumar <vireshk@kernel.org>
R: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
S: Maintained
+F: Documentation/devicetree/bindings/dma/renesas,rzn1-dmamux.yaml
F: Documentation/devicetree/bindings/dma/snps,dma-spear1340.yaml
F: drivers/dma/dw/
F: include/dt-bindings/dma/dw-dmac.h
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH v11 2/9] dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
2022-04-21 8:51 ` [PATCH v11 1/9] dt-bindings: dmaengine: Introduce RZN1 dmamux bindings Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-25 16:08 ` Geert Uytterhoeven
2022-04-21 8:51 ` [PATCH v11 3/9] dt-bindings: dmaengine: Introduce RZN1 DMA compatible Miquel Raynal
` (7 subsequent siblings)
9 siblings, 1 reply; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree
This system controller contains several registers that have nothing to
do with the clock handling, like the DMA mux register. Describe this
part of the system controller as a subnode.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Rob Herring <robh@kernel.org>
Acked-by: Stephen Boyd <sboyd@kernel.org>
---
.../bindings/clock/renesas,r9a06g032-sysctrl.yaml | 11 +++++++++++
1 file changed, 11 insertions(+)
diff --git a/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml b/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
index 25dbb0fac065..95bf485c6cec 100644
--- a/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
+++ b/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
@@ -39,6 +39,17 @@ properties:
'#power-domain-cells':
const: 0
+ '#address-cells':
+ const: 1
+
+ '#size-cells':
+ const: 1
+
+patternProperties:
+ "^dma-router@[a-f0-9]+$":
+ type: object
+ $ref: "../dma/renesas,rzn1-dmamux.yaml#"
+
required:
- compatible
- reg
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH v11 2/9] dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode
2022-04-21 8:51 ` [PATCH v11 2/9] dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode Miquel Raynal
@ 2022-04-25 16:08 ` Geert Uytterhoeven
2022-04-27 8:49 ` Miquel Raynal
0 siblings, 1 reply; 24+ messages in thread
From: Geert Uytterhoeven @ 2022-04-25 16:08 UTC (permalink / raw)
To: Miquel Raynal
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
<miquel.raynal@bootlin.com> wrote:
> This system controller contains several registers that have nothing to
> do with the clock handling, like the DMA mux register. Describe this
> part of the system controller as a subnode.
>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> Acked-by: Stephen Boyd <sboyd@kernel.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> --- a/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
> +++ b/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
> @@ -39,6 +39,17 @@ properties:
> '#power-domain-cells':
> const: 0
>
> + '#address-cells':
> + const: 1
> +
> + '#size-cells':
> + const: 1
> +
> +patternProperties:
> + "^dma-router@[a-f0-9]+$":
For now this must be @a0, right?
> + type: object
> + $ref: "../dma/renesas,rzn1-dmamux.yaml#"
> +
> required:
> - compatible
> - reg
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 2/9] dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode
2022-04-25 16:08 ` Geert Uytterhoeven
@ 2022-04-27 8:49 ` Miquel Raynal
0 siblings, 0 replies; 24+ messages in thread
From: Miquel Raynal @ 2022-04-27 8:49 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Geert,
geert@linux-m68k.org wrote on Mon, 25 Apr 2022 18:08:19 +0200:
> On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
> <miquel.raynal@bootlin.com> wrote:
> > This system controller contains several registers that have nothing to
> > do with the clock handling, like the DMA mux register. Describe this
> > part of the system controller as a subnode.
> >
> > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> > Reviewed-by: Rob Herring <robh@kernel.org>
> > Acked-by: Stephen Boyd <sboyd@kernel.org>
>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Thanks for all the feedback received so far!
> > --- a/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
> > +++ b/Documentation/devicetree/bindings/clock/renesas,r9a06g032-sysctrl.yaml
> > @@ -39,6 +39,17 @@ properties:
> > '#power-domain-cells':
> > const: 0
> >
> > + '#address-cells':
> > + const: 1
> > +
> > + '#size-cells':
> > + const: 1
> > +
> > +patternProperties:
> > + "^dma-router@[a-f0-9]+$":
>
> For now this must be @a0, right?
Yes!
Cheers,
Miquèl
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH v11 3/9] dt-bindings: dmaengine: Introduce RZN1 DMA compatible
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
2022-04-21 8:51 ` [PATCH v11 1/9] dt-bindings: dmaengine: Introduce RZN1 dmamux bindings Miquel Raynal
2022-04-21 8:51 ` [PATCH v11 2/9] dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-21 8:51 ` [PATCH v11 4/9] soc: renesas: rzn1-sysc: Export function to set dmamux Miquel Raynal
` (6 subsequent siblings)
9 siblings, 0 replies; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree, Geert Uytterhoeven
Just like for the NAND controller that is also on this SoC, let's
provide a SoC generic and a more specific couple of compatibles for the
DMA controller.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Rob Herring <robh@kernel.org>
Acked-By: Vinod Koul <vkoul@kernel.org>
---
.../devicetree/bindings/dma/snps,dma-spear1340.yaml | 8 +++++++-
1 file changed, 7 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/dma/snps,dma-spear1340.yaml b/Documentation/devicetree/bindings/dma/snps,dma-spear1340.yaml
index 6b35089ac017..c13649bf7f19 100644
--- a/Documentation/devicetree/bindings/dma/snps,dma-spear1340.yaml
+++ b/Documentation/devicetree/bindings/dma/snps,dma-spear1340.yaml
@@ -15,7 +15,13 @@ allOf:
properties:
compatible:
- const: snps,dma-spear1340
+ oneOf:
+ - const: snps,dma-spear1340
+ - items:
+ - enum:
+ - renesas,r9a06g032-dma
+ - const: renesas,rzn1-dma
+
"#dma-cells":
minimum: 3
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH v11 4/9] soc: renesas: rzn1-sysc: Export function to set dmamux
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
` (2 preceding siblings ...)
2022-04-21 8:51 ` [PATCH v11 3/9] dt-bindings: dmaengine: Introduce RZN1 DMA compatible Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-25 16:10 ` Geert Uytterhoeven
2022-04-21 8:51 ` [PATCH v11 5/9] dmaengine: dw: dmamux: Introduce RZN1 DMA router support Miquel Raynal
` (5 subsequent siblings)
9 siblings, 1 reply; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree
The dmamux register is located within the system controller.
Without syscon, we need an extra helper in order to give write access to
this register to a dmamux driver.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Acked-by: Stephen Boyd <sboyd@kernel.org>
---
drivers/clk/renesas/r9a06g032-clocks.c | 35 ++++++++++++++++++-
include/linux/soc/renesas/r9a06g032-sysctrl.h | 11 ++++++
2 files changed, 45 insertions(+), 1 deletion(-)
create mode 100644 include/linux/soc/renesas/r9a06g032-sysctrl.h
diff --git a/drivers/clk/renesas/r9a06g032-clocks.c b/drivers/clk/renesas/r9a06g032-clocks.c
index c99942f0e4d4..052d99059981 100644
--- a/drivers/clk/renesas/r9a06g032-clocks.c
+++ b/drivers/clk/renesas/r9a06g032-clocks.c
@@ -20,9 +20,12 @@
#include <linux/pm_clock.h>
#include <linux/pm_domain.h>
#include <linux/slab.h>
+#include <linux/soc/renesas/r9a06g032-sysctrl.h>
#include <linux/spinlock.h>
#include <dt-bindings/clock/r9a06g032-sysctrl.h>
+#define R9A06G032_SYSCTRL_DMAMUX 0xA0
+
struct r9a06g032_gate {
u16 gate, reset, ready, midle,
scon, mirack, mistat;
@@ -315,6 +318,30 @@ struct r9a06g032_priv {
void __iomem *reg;
};
+static struct r9a06g032_priv *sysctrl_priv;
+
+/* Exported helper to access the DMAMUX register */
+int r9a06g032_sysctrl_set_dmamux(u32 mask, u32 val)
+{
+ unsigned long flags;
+ u32 dmamux;
+
+ if (!sysctrl_priv)
+ return -EPROBE_DEFER;
+
+ spin_lock_irqsave(&sysctrl_priv->lock, flags);
+
+ dmamux = readl(sysctrl_priv->reg + R9A06G032_SYSCTRL_DMAMUX);
+ dmamux &= ~mask;
+ dmamux |= val & mask;
+ writel(dmamux, sysctrl_priv->reg + R9A06G032_SYSCTRL_DMAMUX);
+
+ spin_unlock_irqrestore(&sysctrl_priv->lock, flags);
+
+ return 0;
+}
+EXPORT_SYMBOL_GPL(r9a06g032_sysctrl_set_dmamux);
+
/* register/bit pairs are encoded as an uint16_t */
static void
clk_rdesc_set(struct r9a06g032_priv *clocks,
@@ -963,7 +990,13 @@ static int __init r9a06g032_clocks_probe(struct platform_device *pdev)
if (error)
return error;
- return r9a06g032_add_clk_domain(dev);
+ error = r9a06g032_add_clk_domain(dev);
+ if (error)
+ return error;
+
+ sysctrl_priv = clocks;
+
+ return 0;
}
static const struct of_device_id r9a06g032_match[] = {
diff --git a/include/linux/soc/renesas/r9a06g032-sysctrl.h b/include/linux/soc/renesas/r9a06g032-sysctrl.h
new file mode 100644
index 000000000000..066dfb15cbdd
--- /dev/null
+++ b/include/linux/soc/renesas/r9a06g032-sysctrl.h
@@ -0,0 +1,11 @@
+/* SPDX-License-Identifier: GPL-2.0 */
+#ifndef __LINUX_SOC_RENESAS_R9A06G032_SYSCTRL_H__
+#define __LINUX_SOC_RENESAS_R9A06G032_SYSCTRL_H__
+
+#ifdef CONFIG_CLK_R9A06G032
+int r9a06g032_sysctrl_set_dmamux(u32 mask, u32 val);
+#else
+static inline int r9a06g032_sysctrl_set_dmamux(u32 mask, u32 val) { return -ENODEV; }
+#endif
+
+#endif /* __LINUX_SOC_RENESAS_R9A06G032_SYSCTRL_H__ */
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH v11 4/9] soc: renesas: rzn1-sysc: Export function to set dmamux
2022-04-21 8:51 ` [PATCH v11 4/9] soc: renesas: rzn1-sysc: Export function to set dmamux Miquel Raynal
@ 2022-04-25 16:10 ` Geert Uytterhoeven
0 siblings, 0 replies; 24+ messages in thread
From: Geert Uytterhoeven @ 2022-04-25 16:10 UTC (permalink / raw)
To: Miquel Raynal
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
<miquel.raynal@bootlin.com> wrote:
> The dmamux register is located within the system controller.
>
> Without syscon, we need an extra helper in order to give write access to
> this register to a dmamux driver.
>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> Acked-by: Stephen Boyd <sboyd@kernel.org>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH v11 5/9] dmaengine: dw: dmamux: Introduce RZN1 DMA router support
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
` (3 preceding siblings ...)
2022-04-21 8:51 ` [PATCH v11 4/9] soc: renesas: rzn1-sysc: Export function to set dmamux Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-21 8:51 ` [PATCH v11 6/9] clk: renesas: r9a06g032: Probe possible children Miquel Raynal
` (4 subsequent siblings)
9 siblings, 0 replies; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree
The Renesas RZN1 DMA IP is based on a DW core, with eg. an additional
dmamux register located in the system control area which can take up to
32 requests (16 per DMA controller). Each DMA channel can be wired to
two different peripherals.
We need two additional information from the 'dmas' property: the channel
(bit in the dmamux register) that must be accessed and the value of the
mux for this channel.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
---
drivers/dma/dw/Kconfig | 9 ++
drivers/dma/dw/Makefile | 2 +
drivers/dma/dw/rzn1-dmamux.c | 155 +++++++++++++++++++++++++++++++++++
3 files changed, 166 insertions(+)
create mode 100644 drivers/dma/dw/rzn1-dmamux.c
diff --git a/drivers/dma/dw/Kconfig b/drivers/dma/dw/Kconfig
index db25f9b7778c..a9828ddd6d06 100644
--- a/drivers/dma/dw/Kconfig
+++ b/drivers/dma/dw/Kconfig
@@ -16,6 +16,15 @@ config DW_DMAC
Support the Synopsys DesignWare AHB DMA controller. This
can be integrated in chips such as the Intel Cherrytrail.
+config RZN1_DMAMUX
+ tristate "Renesas RZ/N1 DMAMUX driver"
+ depends on DW_DMAC
+ depends on ARCH_RZN1 || COMPILE_TEST
+ help
+ Support the Renesas RZ/N1 DMAMUX which is located in front of
+ the Synopsys DesignWare AHB DMA controller located on Renesas
+ SoCs.
+
config DW_DMAC_PCI
tristate "Synopsys DesignWare AHB DMA PCI driver"
depends on PCI
diff --git a/drivers/dma/dw/Makefile b/drivers/dma/dw/Makefile
index a6f358ad8591..e1796015f213 100644
--- a/drivers/dma/dw/Makefile
+++ b/drivers/dma/dw/Makefile
@@ -9,3 +9,5 @@ dw_dmac-$(CONFIG_OF) += of.o
obj-$(CONFIG_DW_DMAC_PCI) += dw_dmac_pci.o
dw_dmac_pci-y := pci.o
+
+obj-$(CONFIG_RZN1_DMAMUX) += rzn1-dmamux.o
diff --git a/drivers/dma/dw/rzn1-dmamux.c b/drivers/dma/dw/rzn1-dmamux.c
new file mode 100644
index 000000000000..11d254e450b0
--- /dev/null
+++ b/drivers/dma/dw/rzn1-dmamux.c
@@ -0,0 +1,155 @@
+// SPDX-License-Identifier: GPL-2.0-only
+/*
+ * Copyright (C) 2022 Schneider-Electric
+ * Author: Miquel Raynal <miquel.raynal@bootlin.com
+ * Based on TI crossbar driver written by Peter Ujfalusi <peter.ujfalusi@ti.com>
+ */
+#include <linux/bitops.h>
+#include <linux/of_device.h>
+#include <linux/of_dma.h>
+#include <linux/slab.h>
+#include <linux/soc/renesas/r9a06g032-sysctrl.h>
+#include <linux/types.h>
+
+#define RNZ1_DMAMUX_NCELLS 6
+#define RZN1_DMAMUX_MAX_LINES 64
+#define RZN1_DMAMUX_LINES_PER_CTLR 16
+
+struct rzn1_dmamux_data {
+ struct dma_router dmarouter;
+ DECLARE_BITMAP(used_chans, 2 * RZN1_DMAMUX_LINES_PER_CTLR);
+};
+
+struct rzn1_dmamux_map {
+ unsigned int req_idx;
+};
+
+static void rzn1_dmamux_free(struct device *dev, void *route_data)
+{
+ struct rzn1_dmamux_data *dmamux = dev_get_drvdata(dev);
+ struct rzn1_dmamux_map *map = route_data;
+
+ dev_dbg(dev, "Unmapping DMAMUX request %u\n", map->req_idx);
+
+ clear_bit(map->req_idx, dmamux->used_chans);
+
+ kfree(map);
+}
+
+static void *rzn1_dmamux_route_allocate(struct of_phandle_args *dma_spec,
+ struct of_dma *ofdma)
+{
+ struct platform_device *pdev = of_find_device_by_node(ofdma->of_node);
+ struct rzn1_dmamux_data *dmamux = platform_get_drvdata(pdev);
+ struct rzn1_dmamux_map *map;
+ unsigned int dmac_idx, chan, val;
+ u32 mask;
+ int ret;
+
+ if (dma_spec->args_count != RNZ1_DMAMUX_NCELLS)
+ return ERR_PTR(-EINVAL);
+
+ map = kzalloc(sizeof(*map), GFP_KERNEL);
+ if (!map)
+ return ERR_PTR(-ENOMEM);
+
+ chan = dma_spec->args[0];
+ map->req_idx = dma_spec->args[4];
+ val = dma_spec->args[5];
+ dma_spec->args_count -= 2;
+
+ if (chan >= RZN1_DMAMUX_LINES_PER_CTLR) {
+ dev_err(&pdev->dev, "Invalid DMA request line: %u\n", chan);
+ ret = -EINVAL;
+ goto free_map;
+ }
+
+ if (map->req_idx >= RZN1_DMAMUX_MAX_LINES ||
+ (map->req_idx % RZN1_DMAMUX_LINES_PER_CTLR) != chan) {
+ dev_err(&pdev->dev, "Invalid MUX request line: %u\n", map->req_idx);
+ ret = -EINVAL;
+ goto free_map;
+ }
+
+ dmac_idx = map->req_idx >= RZN1_DMAMUX_LINES_PER_CTLR ? 1 : 0;
+ dma_spec->np = of_parse_phandle(ofdma->of_node, "dma-masters", dmac_idx);
+ if (!dma_spec->np) {
+ dev_err(&pdev->dev, "Can't get DMA master\n");
+ ret = -EINVAL;
+ goto free_map;
+ }
+
+ dev_dbg(&pdev->dev, "Mapping DMAMUX request %u to DMAC%u request %u\n",
+ map->req_idx, dmac_idx, chan);
+
+ if (test_and_set_bit(map->req_idx, dmamux->used_chans)) {
+ ret = -EBUSY;
+ goto free_map;
+ }
+
+ mask = BIT(map->req_idx);
+ ret = r9a06g032_sysctrl_set_dmamux(mask, val ? mask : 0);
+ if (ret)
+ goto clear_bitmap;
+
+ return map;
+
+clear_bitmap:
+ clear_bit(map->req_idx, dmamux->used_chans);
+free_map:
+ kfree(map);
+
+ return ERR_PTR(ret);
+}
+
+static const struct of_device_id rzn1_dmac_match[] = {
+ { .compatible = "renesas,rzn1-dma" },
+ {}
+};
+
+static int rzn1_dmamux_probe(struct platform_device *pdev)
+{
+ struct device_node *mux_node = pdev->dev.of_node;
+ const struct of_device_id *match;
+ struct device_node *dmac_node;
+ struct rzn1_dmamux_data *dmamux;
+
+ dmamux = devm_kzalloc(&pdev->dev, sizeof(*dmamux), GFP_KERNEL);
+ if (!dmamux)
+ return -ENOMEM;
+
+ dmac_node = of_parse_phandle(mux_node, "dma-masters", 0);
+ if (!dmac_node)
+ return dev_err_probe(&pdev->dev, -ENODEV, "Can't get DMA master node\n");
+
+ match = of_match_node(rzn1_dmac_match, dmac_node);
+ of_node_put(dmac_node);
+ if (!match)
+ return dev_err_probe(&pdev->dev, -EINVAL, "DMA master is not supported\n");
+
+ dmamux->dmarouter.dev = &pdev->dev;
+ dmamux->dmarouter.route_free = rzn1_dmamux_free;
+
+ platform_set_drvdata(pdev, dmamux);
+
+ return of_dma_router_register(mux_node, rzn1_dmamux_route_allocate,
+ &dmamux->dmarouter);
+}
+
+static const struct of_device_id rzn1_dmamux_match[] = {
+ { .compatible = "renesas,rzn1-dmamux" },
+ {}
+};
+
+static struct platform_driver rzn1_dmamux_driver = {
+ .driver = {
+ .name = "renesas,rzn1-dmamux",
+ .of_match_table = rzn1_dmamux_match,
+ },
+ .probe = rzn1_dmamux_probe,
+};
+module_platform_driver(rzn1_dmamux_driver);
+
+MODULE_LICENSE("GPL");
+MODULE_AUTHOR("Miquel Raynal <miquel.raynal@bootlin.com");
+MODULE_DESCRIPTION("Renesas RZ/N1 DMAMUX driver");
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH v11 6/9] clk: renesas: r9a06g032: Probe possible children
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
` (4 preceding siblings ...)
2022-04-21 8:51 ` [PATCH v11 5/9] dmaengine: dw: dmamux: Introduce RZN1 DMA router support Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-25 16:18 ` Geert Uytterhoeven
2022-04-21 8:51 ` [PATCH v11 7/9] dmaengine: dw: Add RZN1 compatible Miquel Raynal
` (3 subsequent siblings)
9 siblings, 1 reply; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree
The clock controller device on r9a06g032 takes all the memory range that
is described as being a system controller. This range contains many
different (unrelated?) registers besides the ones belonging to the clock
controller, that can necessitate to be accessed from other peripherals.
For instance, the dmamux registers are there. The dmamux "device" will
be described as a child node of the clock/system controller node, which
means we need the top device driver (the clock controller driver in this
case) to populate its children manually.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Acked-by: Stephen Boyd <sboyd@kernel.org>
---
drivers/clk/renesas/r9a06g032-clocks.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/clk/renesas/r9a06g032-clocks.c b/drivers/clk/renesas/r9a06g032-clocks.c
index 052d99059981..1df56d7ab3e1 100644
--- a/drivers/clk/renesas/r9a06g032-clocks.c
+++ b/drivers/clk/renesas/r9a06g032-clocks.c
@@ -16,6 +16,7 @@
#include <linux/math64.h>
#include <linux/of.h>
#include <linux/of_address.h>
+#include <linux/of_platform.h>
#include <linux/platform_device.h>
#include <linux/pm_clock.h>
#include <linux/pm_domain.h>
@@ -996,7 +997,7 @@ static int __init r9a06g032_clocks_probe(struct platform_device *pdev)
sysctrl_priv = clocks;
- return 0;
+ return of_platform_populate(np, NULL, NULL, dev);
}
static const struct of_device_id r9a06g032_match[] = {
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH v11 6/9] clk: renesas: r9a06g032: Probe possible children
2022-04-21 8:51 ` [PATCH v11 6/9] clk: renesas: r9a06g032: Probe possible children Miquel Raynal
@ 2022-04-25 16:18 ` Geert Uytterhoeven
2022-04-27 9:02 ` Miquel Raynal
0 siblings, 1 reply; 24+ messages in thread
From: Geert Uytterhoeven @ 2022-04-25 16:18 UTC (permalink / raw)
To: Miquel Raynal
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Miquel,
On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
<miquel.raynal@bootlin.com> wrote:
> The clock controller device on r9a06g032 takes all the memory range that
> is described as being a system controller. This range contains many
> different (unrelated?) registers besides the ones belonging to the clock
> controller, that can necessitate to be accessed from other peripherals.
>
> For instance, the dmamux registers are there. The dmamux "device" will
> be described as a child node of the clock/system controller node, which
> means we need the top device driver (the clock controller driver in this
> case) to populate its children manually.
>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> Acked-by: Stephen Boyd <sboyd@kernel.org>
Thanks for your patch!
> --- a/drivers/clk/renesas/r9a06g032-clocks.c
> +++ b/drivers/clk/renesas/r9a06g032-clocks.c
> @@ -996,7 +997,7 @@ static int __init r9a06g032_clocks_probe(struct platform_device *pdev)
>
> sysctrl_priv = clocks;
>
> - return 0;
> + return of_platform_populate(np, NULL, NULL, dev);
This is a bit dangerous: in the (very unlikely) case that
of_platform_populate() fails, the clock driver will fail to probe,
and all managed cleanup will be done (not everything will be cleant
up, though), while sysctrl_priv will still point to the now-freed
r9a06g032_priv structure.
So I think you just want to ignore the failure from
of_platform_populate(), and return zero anyway.
> }
>
> static const struct of_device_id r9a06g032_match[] = {
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 6/9] clk: renesas: r9a06g032: Probe possible children
2022-04-25 16:18 ` Geert Uytterhoeven
@ 2022-04-27 9:02 ` Miquel Raynal
0 siblings, 0 replies; 24+ messages in thread
From: Miquel Raynal @ 2022-04-27 9:02 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Geert,
geert@linux-m68k.org wrote on Mon, 25 Apr 2022 18:18:28 +0200:
> Hi Miquel,
>
> On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
> <miquel.raynal@bootlin.com> wrote:
> > The clock controller device on r9a06g032 takes all the memory range that
> > is described as being a system controller. This range contains many
> > different (unrelated?) registers besides the ones belonging to the clock
> > controller, that can necessitate to be accessed from other peripherals.
> >
> > For instance, the dmamux registers are there. The dmamux "device" will
> > be described as a child node of the clock/system controller node, which
> > means we need the top device driver (the clock controller driver in this
> > case) to populate its children manually.
> >
> > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> > Acked-by: Stephen Boyd <sboyd@kernel.org>
>
> Thanks for your patch!
>
> > --- a/drivers/clk/renesas/r9a06g032-clocks.c
> > +++ b/drivers/clk/renesas/r9a06g032-clocks.c
> > @@ -996,7 +997,7 @@ static int __init r9a06g032_clocks_probe(struct platform_device *pdev)
> >
> > sysctrl_priv = clocks;
> >
> > - return 0;
> > + return of_platform_populate(np, NULL, NULL, dev);
>
> This is a bit dangerous: in the (very unlikely) case that
> of_platform_populate() fails, the clock driver will fail to probe,
> and all managed cleanup will be done (not everything will be cleant
> up, though), while sysctrl_priv will still point to the now-freed
> r9a06g032_priv structure.
>
> So I think you just want to ignore the failure from
> of_platform_populate(), and return zero anyway.
That is a very good point. I've changed the logic to just print an
error message and return 0 anyway.
Thanks,
Miquèl
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH v11 7/9] dmaengine: dw: Add RZN1 compatible
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
` (5 preceding siblings ...)
2022-04-21 8:51 ` [PATCH v11 6/9] clk: renesas: r9a06g032: Probe possible children Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-21 8:51 ` [PATCH v11 8/9] ARM: dts: r9a06g032: Add the two DMA nodes Miquel Raynal
` (2 subsequent siblings)
9 siblings, 0 replies; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree, Geert Uytterhoeven
The Renesas RZN1 DMA IP is very close to the original DW DMA IP, a DMA
router has been introduced to handle the wiring options that have been
added.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Acked-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Acked-By: Vinod Koul <vkoul@kernel.org>
---
drivers/dma/dw/platform.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/dma/dw/platform.c b/drivers/dma/dw/platform.c
index 246118955877..47f2292dba98 100644
--- a/drivers/dma/dw/platform.c
+++ b/drivers/dma/dw/platform.c
@@ -137,6 +137,7 @@ static void dw_shutdown(struct platform_device *pdev)
#ifdef CONFIG_OF
static const struct of_device_id dw_dma_of_id_table[] = {
{ .compatible = "snps,dma-spear1340", .data = &dw_dma_chip_pdata },
+ { .compatible = "renesas,rzn1-dma", .data = &dw_dma_chip_pdata },
{}
};
MODULE_DEVICE_TABLE(of, dw_dma_of_id_table);
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* [PATCH v11 8/9] ARM: dts: r9a06g032: Add the two DMA nodes
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
` (6 preceding siblings ...)
2022-04-21 8:51 ` [PATCH v11 7/9] dmaengine: dw: Add RZN1 compatible Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-25 16:29 ` Geert Uytterhoeven
2022-04-21 8:51 ` [PATCH v11 9/9] ARM: dts: r9a06g032: Describe the DMA router Miquel Raynal
2022-04-25 16:05 ` [PATCH v11 0/9] RZN1 DMA support Geert Uytterhoeven
9 siblings, 1 reply; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree, Geert Uytterhoeven
Describe the two DMA controllers available on this SoC.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
---
arch/arm/boot/dts/r9a06g032.dtsi | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)
diff --git a/arch/arm/boot/dts/r9a06g032.dtsi b/arch/arm/boot/dts/r9a06g032.dtsi
index 636a6ab31c58..839580ec21ee 100644
--- a/arch/arm/boot/dts/r9a06g032.dtsi
+++ b/arch/arm/boot/dts/r9a06g032.dtsi
@@ -200,6 +200,36 @@ nand_controller: nand-controller@40102000 {
status = "disabled";
};
+ dma0: dma-controller@40104000 {
+ compatible = "renesas,r9a06g032-dma", "renesas,rzn1-dma";
+ reg = <0x40104000 0x1000>;
+ interrupts = <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>;
+ clock-names = "hclk";
+ clocks = <&sysctrl R9A06G032_HCLK_DMA0>;
+ dma-channels = <8>;
+ dma-requests = <16>;
+ dma-masters = <1>;
+ #dma-cells = <3>;
+ block_size = <0xfff>;
+ data_width = <3>;
+ status = "disabled";
+ };
+
+ dma1: dma-controller@40105000 {
+ compatible = "renesas,r9a06g032-dma", "renesas,rzn1-dma";
+ reg = <0x40105000 0x1000>;
+ interrupts = <GIC_SPI 57 IRQ_TYPE_LEVEL_HIGH>;
+ clock-names = "hclk";
+ clocks = <&sysctrl R9A06G032_HCLK_DMA1>;
+ dma-channels = <8>;
+ dma-requests = <16>;
+ dma-masters = <1>;
+ #dma-cells = <3>;
+ block_size = <0xfff>;
+ data_width = <3>;
+ status = "disabled";
+ };
+
gic: interrupt-controller@44101000 {
compatible = "arm,gic-400", "arm,cortex-a7-gic";
interrupt-controller;
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH v11 8/9] ARM: dts: r9a06g032: Add the two DMA nodes
2022-04-21 8:51 ` [PATCH v11 8/9] ARM: dts: r9a06g032: Add the two DMA nodes Miquel Raynal
@ 2022-04-25 16:29 ` Geert Uytterhoeven
2022-04-27 9:14 ` Miquel Raynal
0 siblings, 1 reply; 24+ messages in thread
From: Geert Uytterhoeven @ 2022-04-25 16:29 UTC (permalink / raw)
To: Miquel Raynal
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Geert Uytterhoeven
Hi Miquel,
On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
<miquel.raynal@bootlin.com> wrote:
> Describe the two DMA controllers available on this SoC.
>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Still, a few comments below, valid for both instances...
> --- a/arch/arm/boot/dts/r9a06g032.dtsi
> +++ b/arch/arm/boot/dts/r9a06g032.dtsi
> @@ -200,6 +200,36 @@ nand_controller: nand-controller@40102000 {
> status = "disabled";
> };
>
> + dma0: dma-controller@40104000 {
> + compatible = "renesas,r9a06g032-dma", "renesas,rzn1-dma";
> + reg = <0x40104000 0x1000>;
> + interrupts = <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>;
> + clock-names = "hclk";
> + clocks = <&sysctrl R9A06G032_HCLK_DMA0>;
> + dma-channels = <8>;
> + dma-requests = <16>;
> + dma-masters = <1>;
> + #dma-cells = <3>;
> + block_size = <0xfff>;
> + data_width = <3>;
This property is deprecated, in favor of "dma-width".
> + status = "disabled";
Why not keep it enabled?
> + };
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 8/9] ARM: dts: r9a06g032: Add the two DMA nodes
2022-04-25 16:29 ` Geert Uytterhoeven
@ 2022-04-27 9:14 ` Miquel Raynal
2022-04-27 9:36 ` Geert Uytterhoeven
0 siblings, 1 reply; 24+ messages in thread
From: Miquel Raynal @ 2022-04-27 9:14 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Geert Uytterhoeven
Hi Geert,
geert@linux-m68k.org wrote on Mon, 25 Apr 2022 18:29:58 +0200:
> Hi Miquel,
>
> On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
> <miquel.raynal@bootlin.com> wrote:
> > Describe the two DMA controllers available on this SoC.
> >
> > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> Still, a few comments below, valid for both instances...
>
> > --- a/arch/arm/boot/dts/r9a06g032.dtsi
> > +++ b/arch/arm/boot/dts/r9a06g032.dtsi
> > @@ -200,6 +200,36 @@ nand_controller: nand-controller@40102000 {
> > status = "disabled";
> > };
> >
> > + dma0: dma-controller@40104000 {
> > + compatible = "renesas,r9a06g032-dma", "renesas,rzn1-dma";
> > + reg = <0x40104000 0x1000>;
> > + interrupts = <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>;
> > + clock-names = "hclk";
> > + clocks = <&sysctrl R9A06G032_HCLK_DMA0>;
> > + dma-channels = <8>;
> > + dma-requests = <16>;
> > + dma-masters = <1>;
> > + #dma-cells = <3>;
> > + block_size = <0xfff>;
> > + data_width = <3>;
>
> This property is deprecated, in favor of "dma-width".
Indeed,
data_width = <3>;
is deprecated.
However, dma-width does not seem to be described anywhere. Do you mean:
data-width = <8>;
instead?
>
> > + status = "disabled";
>
> Why not keep it enabled?
I'm used to always disable all the nodes from the SoC descriptions,
but it's true that for a DMA controller it might make sense to keep
it enabled.
Would dropping the status property be enough or do you prefer a proper
status = "okay";
instead?
Thanks a lot,
Miquèl
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 8/9] ARM: dts: r9a06g032: Add the two DMA nodes
2022-04-27 9:14 ` Miquel Raynal
@ 2022-04-27 9:36 ` Geert Uytterhoeven
2022-04-27 9:38 ` Miquel Raynal
0 siblings, 1 reply; 24+ messages in thread
From: Geert Uytterhoeven @ 2022-04-27 9:36 UTC (permalink / raw)
To: Miquel Raynal
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Geert Uytterhoeven
Hi Miquel,
On Wed, Apr 27, 2022 at 11:14 AM Miquel Raynal
<miquel.raynal@bootlin.com> wrote:
> geert@linux-m68k.org wrote on Mon, 25 Apr 2022 18:29:58 +0200:
> > On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
> > <miquel.raynal@bootlin.com> wrote:
> > > Describe the two DMA controllers available on this SoC.
> > >
> > > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> >
> > Still, a few comments below, valid for both instances...
> >
> > > --- a/arch/arm/boot/dts/r9a06g032.dtsi
> > > +++ b/arch/arm/boot/dts/r9a06g032.dtsi
> > > @@ -200,6 +200,36 @@ nand_controller: nand-controller@40102000 {
> > > status = "disabled";
> > > };
> > >
> > > + dma0: dma-controller@40104000 {
> > > + compatible = "renesas,r9a06g032-dma", "renesas,rzn1-dma";
> > > + reg = <0x40104000 0x1000>;
> > > + interrupts = <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>;
> > > + clock-names = "hclk";
> > > + clocks = <&sysctrl R9A06G032_HCLK_DMA0>;
> > > + dma-channels = <8>;
> > > + dma-requests = <16>;
> > > + dma-masters = <1>;
> > > + #dma-cells = <3>;
> > > + block_size = <0xfff>;
> > > + data_width = <3>;
> >
> > This property is deprecated, in favor of "dma-width".
>
> Indeed,
> data_width = <3>;
> is deprecated.
>
> However, dma-width does not seem to be described anywhere. Do you mean:
> data-width = <8>;
> instead?
Oops, I did mean "data-width".
> > > + status = "disabled";
> >
> > Why not keep it enabled?
>
> I'm used to always disable all the nodes from the SoC descriptions,
> but it's true that for a DMA controller it might make sense to keep
> it enabled.
>
> Would dropping the status property be enough or do you prefer a proper
> status = "okay";
> instead?
Please just drop the status property, like is done in other Renesas .dtsi
files.
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 8/9] ARM: dts: r9a06g032: Add the two DMA nodes
2022-04-27 9:36 ` Geert Uytterhoeven
@ 2022-04-27 9:38 ` Miquel Raynal
0 siblings, 0 replies; 24+ messages in thread
From: Miquel Raynal @ 2022-04-27 9:38 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS,
Geert Uytterhoeven
Hi Geert,
geert@linux-m68k.org wrote on Wed, 27 Apr 2022 11:36:23 +0200:
> Hi Miquel,
>
> On Wed, Apr 27, 2022 at 11:14 AM Miquel Raynal
> <miquel.raynal@bootlin.com> wrote:
> > geert@linux-m68k.org wrote on Mon, 25 Apr 2022 18:29:58 +0200:
> > > On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
> > > <miquel.raynal@bootlin.com> wrote:
> > > > Describe the two DMA controllers available on this SoC.
> > > >
> > > > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
> > > > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
> > >
> > > Still, a few comments below, valid for both instances...
> > >
> > > > --- a/arch/arm/boot/dts/r9a06g032.dtsi
> > > > +++ b/arch/arm/boot/dts/r9a06g032.dtsi
> > > > @@ -200,6 +200,36 @@ nand_controller: nand-controller@40102000 {
> > > > status = "disabled";
> > > > };
> > > >
> > > > + dma0: dma-controller@40104000 {
> > > > + compatible = "renesas,r9a06g032-dma", "renesas,rzn1-dma";
> > > > + reg = <0x40104000 0x1000>;
> > > > + interrupts = <GIC_SPI 56 IRQ_TYPE_LEVEL_HIGH>;
> > > > + clock-names = "hclk";
> > > > + clocks = <&sysctrl R9A06G032_HCLK_DMA0>;
> > > > + dma-channels = <8>;
> > > > + dma-requests = <16>;
> > > > + dma-masters = <1>;
> > > > + #dma-cells = <3>;
> > > > + block_size = <0xfff>;
> > > > + data_width = <3>;
> > >
> > > This property is deprecated, in favor of "dma-width".
> >
> > Indeed,
> > data_width = <3>;
> > is deprecated.
> >
> > However, dma-width does not seem to be described anywhere. Do you mean:
> > data-width = <8>;
> > instead?
>
> Oops, I did mean "data-width".
>
> > > > + status = "disabled";
> > >
> > > Why not keep it enabled?
> >
> > I'm used to always disable all the nodes from the SoC descriptions,
> > but it's true that for a DMA controller it might make sense to keep
> > it enabled.
> >
> > Would dropping the status property be enough or do you prefer a proper
> > status = "okay";
> > instead?
>
> Please just drop the status property, like is done in other Renesas .dtsi
> files.
Sure.
Thanks,
Miquèl
^ permalink raw reply [flat|nested] 24+ messages in thread
* [PATCH v11 9/9] ARM: dts: r9a06g032: Describe the DMA router
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
` (7 preceding siblings ...)
2022-04-21 8:51 ` [PATCH v11 8/9] ARM: dts: r9a06g032: Add the two DMA nodes Miquel Raynal
@ 2022-04-21 8:51 ` Miquel Raynal
2022-04-25 16:32 ` Geert Uytterhoeven
2022-04-25 16:05 ` [PATCH v11 0/9] RZN1 DMA support Geert Uytterhoeven
9 siblings, 1 reply; 24+ messages in thread
From: Miquel Raynal @ 2022-04-21 8:51 UTC (permalink / raw)
To: Magnus Damm, Gareth Williams, Phil Edworthy, Geert Uytterhoeven,
Vinod Koul
Cc: Miquel Raynal, linux-renesas-soc, dmaengine, Milan Stevanovic,
Jimmy Lalande, Pascal Eberhard, Thomas Petazzoni, Herve Codina,
Clement Leger, Stephen Boyd, Michael Turquette, linux-clk,
Viresh Kumar, Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
devicetree
There is a dmamux on this SoC which allows picking two different sources
for a single DMA request.
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
arch/arm/boot/dts/r9a06g032.dtsi | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/arch/arm/boot/dts/r9a06g032.dtsi b/arch/arm/boot/dts/r9a06g032.dtsi
index 839580ec21ee..c854aa4cfa77 100644
--- a/arch/arm/boot/dts/r9a06g032.dtsi
+++ b/arch/arm/boot/dts/r9a06g032.dtsi
@@ -91,6 +91,16 @@ sysctrl: system-controller@4000c000 {
clocks = <&ext_mclk>, <&ext_rtc_clk>,
<&ext_jtag_clk>, <&ext_rgmii_ref>;
clock-names = "mclk", "rtc", "jtag", "rgmii_ref_ext";
+ #address-cells = <1>;
+ #size-cells = <1>;
+
+ dmamux: dma-router@a0 {
+ compatible = "renesas,rzn1-dmamux";
+ reg = <0xa0 4>;
+ #dma-cells = <6>;
+ dma-requests = <32>;
+ dma-masters = <&dma0 &dma1>;
+ };
};
uart0: serial@40060000 {
--
2.27.0
^ permalink raw reply related [flat|nested] 24+ messages in thread
* Re: [PATCH v11 9/9] ARM: dts: r9a06g032: Describe the DMA router
2022-04-21 8:51 ` [PATCH v11 9/9] ARM: dts: r9a06g032: Describe the DMA router Miquel Raynal
@ 2022-04-25 16:32 ` Geert Uytterhoeven
0 siblings, 0 replies; 24+ messages in thread
From: Geert Uytterhoeven @ 2022-04-25 16:32 UTC (permalink / raw)
To: Miquel Raynal
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
<miquel.raynal@bootlin.com> wrote:
> There is a dmamux on this SoC which allows picking two different sources
> for a single DMA request.
>
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 0/9] RZN1 DMA support
2022-04-21 8:51 [PATCH v11 0/9] RZN1 DMA support Miquel Raynal
` (8 preceding siblings ...)
2022-04-21 8:51 ` [PATCH v11 9/9] ARM: dts: r9a06g032: Describe the DMA router Miquel Raynal
@ 2022-04-25 16:05 ` Geert Uytterhoeven
2022-04-26 7:32 ` Miquel Raynal
9 siblings, 1 reply; 24+ messages in thread
From: Geert Uytterhoeven @ 2022-04-25 16:05 UTC (permalink / raw)
To: Miquel Raynal
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Miquel,
On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
<miquel.raynal@bootlin.com> wrote:
> This is the series bringing DMA support to RZN1 platforms.
> Other series follow with eg. UART and RTC support as well.
Thanks for your series!
> There is no other conflicting dependency with the other series, so this
> series can now entirely be merged in the dmaengine tree I believe.
>
> Changes in v11:
> * Renamed two defines.
> * Changed the way the bitmap is declared.
> * Updated the cover letter: this series can now go in through the
> dmaengine tree.
/me confused
> Miquel Raynal (9):
> dt-bindings: dmaengine: Introduce RZN1 dmamux bindings
> dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode
> dt-bindings: dmaengine: Introduce RZN1 DMA compatible
> soc: renesas: rzn1-sysc: Export function to set dmamux
> dmaengine: dw: dmamux: Introduce RZN1 DMA router support
> clk: renesas: r9a06g032: Probe possible children
> dmaengine: dw: Add RZN1 compatible
> ARM: dts: r9a06g032: Add the two DMA nodes
> ARM: dts: r9a06g032: Describe the DMA router
The last two DTS parts have to go in through the renesas-arm-dt and
soc trees.
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 0/9] RZN1 DMA support
2022-04-25 16:05 ` [PATCH v11 0/9] RZN1 DMA support Geert Uytterhoeven
@ 2022-04-26 7:32 ` Miquel Raynal
2022-04-27 12:50 ` Geert Uytterhoeven
0 siblings, 1 reply; 24+ messages in thread
From: Miquel Raynal @ 2022-04-26 7:32 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Geert,
geert@linux-m68k.org wrote on Mon, 25 Apr 2022 18:05:34 +0200:
> Hi Miquel,
>
> On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
> <miquel.raynal@bootlin.com> wrote:
> > This is the series bringing DMA support to RZN1 platforms.
> > Other series follow with eg. UART and RTC support as well.
>
> Thanks for your series!
>
> > There is no other conflicting dependency with the other series, so this
> > series can now entirely be merged in the dmaengine tree I believe.
> >
> > Changes in v11:
> > * Renamed two defines.
> > * Changed the way the bitmap is declared.
> > * Updated the cover letter: this series can now go in through the
> > dmaengine tree.
>
> /me confused
>
> > Miquel Raynal (9):
> > dt-bindings: dmaengine: Introduce RZN1 dmamux bindings
> > dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode
> > dt-bindings: dmaengine: Introduce RZN1 DMA compatible
> > soc: renesas: rzn1-sysc: Export function to set dmamux
> > dmaengine: dw: dmamux: Introduce RZN1 DMA router support
> > clk: renesas: r9a06g032: Probe possible children
> > dmaengine: dw: Add RZN1 compatible
> > ARM: dts: r9a06g032: Add the two DMA nodes
> > ARM: dts: r9a06g032: Describe the DMA router
>
> The last two DTS parts have to go in through the renesas-arm-dt and
> soc trees.
Yes, DT usually never go in through subsystem trees anyway, of
course they should be taken in through the Renesas tree. For the other
patches I think its simpler if everything goes through the dmaengine
tree, but I'm fine either way, I'll let you discuss this with the DMA
folks if you disagree.
Thanks,
Miquèl
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 0/9] RZN1 DMA support
2022-04-26 7:32 ` Miquel Raynal
@ 2022-04-27 12:50 ` Geert Uytterhoeven
2022-04-27 15:45 ` Miquel Raynal
0 siblings, 1 reply; 24+ messages in thread
From: Geert Uytterhoeven @ 2022-04-27 12:50 UTC (permalink / raw)
To: Miquel Raynal
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Miquel,
On Tue, Apr 26, 2022 at 9:32 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
> geert@linux-m68k.org wrote on Mon, 25 Apr 2022 18:05:34 +0200:
> > On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
> > <miquel.raynal@bootlin.com> wrote:
> > > This is the series bringing DMA support to RZN1 platforms.
> > > Other series follow with eg. UART and RTC support as well.
> >
> > Thanks for your series!
> >
> > > There is no other conflicting dependency with the other series, so this
> > > series can now entirely be merged in the dmaengine tree I believe.
> > >
> > > Changes in v11:
> > > * Renamed two defines.
> > > * Changed the way the bitmap is declared.
> > > * Updated the cover letter: this series can now go in through the
> > > dmaengine tree.
> >
> > /me confused
> >
> > > Miquel Raynal (9):
> > > dt-bindings: dmaengine: Introduce RZN1 dmamux bindings
> > > dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode
> > > dt-bindings: dmaengine: Introduce RZN1 DMA compatible
> > > soc: renesas: rzn1-sysc: Export function to set dmamux
> > > dmaengine: dw: dmamux: Introduce RZN1 DMA router support
> > > clk: renesas: r9a06g032: Probe possible children
> > > dmaengine: dw: Add RZN1 compatible
> > > ARM: dts: r9a06g032: Add the two DMA nodes
> > > ARM: dts: r9a06g032: Describe the DMA router
> >
> > The last two DTS parts have to go in through the renesas-arm-dt and
> > soc trees.
>
> Yes, DT usually never go in through subsystem trees anyway, of
> course they should be taken in through the Renesas tree. For the other
> patches I think its simpler if everything goes through the dmaengine
> tree, but I'm fine either way, I'll let you discuss this with the DMA
> folks if you disagree.
Fine for me. I've acked the renesas-clk related patches, so they
can go in through the dmaengine tree.
Thanks!
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
^ permalink raw reply [flat|nested] 24+ messages in thread
* Re: [PATCH v11 0/9] RZN1 DMA support
2022-04-27 12:50 ` Geert Uytterhoeven
@ 2022-04-27 15:45 ` Miquel Raynal
0 siblings, 0 replies; 24+ messages in thread
From: Miquel Raynal @ 2022-04-27 15:45 UTC (permalink / raw)
To: Geert Uytterhoeven
Cc: Magnus Damm, Gareth Williams, Phil Edworthy, Vinod Koul,
Linux-Renesas, dmaengine, Milan Stevanovic, Jimmy Lalande,
Pascal Eberhard, Thomas Petazzoni, Herve Codina, Clement Leger,
Stephen Boyd, Michael Turquette, linux-clk, Viresh Kumar,
Andy Shevchenko, Ilpo Jarvinen, Rob Herring,
open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS
Hi Geert,
geert@linux-m68k.org wrote on Wed, 27 Apr 2022 14:50:43 +0200:
> Hi Miquel,
>
> On Tue, Apr 26, 2022 at 9:32 AM Miquel Raynal <miquel.raynal@bootlin.com> wrote:
> > geert@linux-m68k.org wrote on Mon, 25 Apr 2022 18:05:34 +0200:
> > > On Thu, Apr 21, 2022 at 10:51 AM Miquel Raynal
> > > <miquel.raynal@bootlin.com> wrote:
> > > > This is the series bringing DMA support to RZN1 platforms.
> > > > Other series follow with eg. UART and RTC support as well.
> > >
> > > Thanks for your series!
> > >
> > > > There is no other conflicting dependency with the other series, so this
> > > > series can now entirely be merged in the dmaengine tree I believe.
> > > >
> > > > Changes in v11:
> > > > * Renamed two defines.
> > > > * Changed the way the bitmap is declared.
> > > > * Updated the cover letter: this series can now go in through the
> > > > dmaengine tree.
> > >
> > > /me confused
> > >
> > > > Miquel Raynal (9):
> > > > dt-bindings: dmaengine: Introduce RZN1 dmamux bindings
> > > > dt-bindings: clock: r9a06g032-sysctrl: Reference the DMAMUX subnode
> > > > dt-bindings: dmaengine: Introduce RZN1 DMA compatible
> > > > soc: renesas: rzn1-sysc: Export function to set dmamux
> > > > dmaengine: dw: dmamux: Introduce RZN1 DMA router support
> > > > clk: renesas: r9a06g032: Probe possible children
> > > > dmaengine: dw: Add RZN1 compatible
> > > > ARM: dts: r9a06g032: Add the two DMA nodes
> > > > ARM: dts: r9a06g032: Describe the DMA router
> > >
> > > The last two DTS parts have to go in through the renesas-arm-dt and
> > > soc trees.
> >
> > Yes, DT usually never go in through subsystem trees anyway, of
> > course they should be taken in through the Renesas tree. For the other
> > patches I think its simpler if everything goes through the dmaengine
> > tree, but I'm fine either way, I'll let you discuss this with the DMA
> > folks if you disagree.
>
> Fine for me. I've acked the renesas-clk related patches, so they
> can go in through the dmaengine tree.
Perfect. Sounds like v12 is the right one \o/
Thanks for all the feedback you keep providing on all the RZ/N1
(correctly spelled this time;-) ) series.
Cheers,
Miquèl
^ permalink raw reply [flat|nested] 24+ messages in thread