On Thu, Apr 21, 2016 at 5:27 AM, Simon Horman <horms@xxxxxxxxxxxx> wrote: > > We are back to one DMA controller this week so I updated your patch and > queued up the following. > > From: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> > Date: Mon, 18 Apr 2016 18:02:56 +0200 > Subject: [PATCH] ARM: dts: r8a7793: Add SDHI controllers > > Same as on r8a7791. > > Signed-off-by: Ulrich Hecht <ulrich.hecht+renesas@xxxxxxxxx> > Signed-off-by: Simon Horman <horms+renesas@xxxxxxxxxxxx> > --- > arch/arm/boot/dts/r8a7793.dtsi | 33 +++++++++++++++++++++++++++++++++ > 1 file changed, 33 insertions(+) > > diff --git a/arch/arm/boot/dts/r8a7793.dtsi b/arch/arm/boot/dts/r8a7793.dtsi > index fc552629ed4a..a93d509732a8 100644 > --- a/arch/arm/boot/dts/r8a7793.dtsi > +++ b/arch/arm/boot/dts/r8a7793.dtsi > @@ -507,6 +507,39 @@ > reg = <0 0xe6060000 0 0x250>; > }; > > + sdhi0: sd@ee100000 { > + compatible = "renesas,sdhi-r8a7793"; > + reg = <0 0xee100000 0 0x328>; > + interrupts = <0 165 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&mstp3_clks R8A7793_CLK_SDHI0>; > + dmas = <&dmac0 0xcd>, <&dmac0 0xce>; > + dma-names = "tx", "rx", "tx", "rx"; If you drop the last two dma values, you should drop the last two dma-names values, too. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx 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