* [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] @ 2019-07-05 13:39 Fabrizio Castro 2019-07-05 13:39 ` [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register Fabrizio Castro ` (3 more replies) 0 siblings, 4 replies; 12+ messages in thread From: Fabrizio Castro @ 2019-07-05 13:39 UTC (permalink / raw) To: Geert Uytterhoeven, Rob Herring, Mark Rutland Cc: Fabrizio Castro, Simon Horman, Magnus Damm, linux-renesas-soc, devicetree, Chris Paterson, Biju Das, xu_shunji Dear All, this series adds HDMI audio support to the HiHope RZ/G2[MN]. Thanks, Fab Fabrizio Castro (3): arm64: dts: renesas: r8a774a1: Use extended audio dmac register arm64: dts: renesas: r8a774a1: Add SSIU support for sound arm64: dts: renesas: hihope-common: Add HDMI audio support arch/arm64/boot/dts/renesas/hihope-common.dtsi | 48 +++++ arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 253 +++++++++++++++++++++++-- 2 files changed, 280 insertions(+), 21 deletions(-) -- 2.7.4 ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register 2019-07-05 13:39 [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] Fabrizio Castro @ 2019-07-05 13:39 ` Fabrizio Castro 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:14 ` Geert Uytterhoeven 2019-07-05 13:39 ` [PATCH 2/3] arm64: dts: renesas: r8a774a1: Add SSIU support for sound Fabrizio Castro ` (2 subsequent siblings) 3 siblings, 2 replies; 12+ messages in thread From: Fabrizio Castro @ 2019-07-05 13:39 UTC (permalink / raw) To: Geert Uytterhoeven, Rob Herring, Mark Rutland Cc: Fabrizio Castro, Simon Horman, Magnus Damm, linux-renesas-soc, devicetree, Chris Paterson, Biju Das Basic audio dmac register only supports busif from 0 to 3, in order to use busif4 ~ busif7 extended audio dmac registers need to be used. Based on similar work from Jiada Wang: 7a516e49d975 ("arm64: dts: renesas: use extended audio dmac register") Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> --- arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi index f209457..2c5173e 100644 --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi @@ -1653,7 +1653,7 @@ <0 0xec5a0000 0 0x100>, /* ADG */ <0 0xec540000 0 0x1000>, /* SSIU */ <0 0xec541000 0 0x280>, /* SSI */ - <0 0xec740000 0 0x200>; /* Audio DMAC peri peri*/ + <0 0xec760000 0 0x200>; /* Audio DMAC peri peri*/ reg-names = "scu", "adg", "ssiu", "ssi", "audmapp"; clocks = <&cpg CPG_MOD 1005>, -- 2.7.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register 2019-07-05 13:39 ` [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register Fabrizio Castro @ 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:14 ` Geert Uytterhoeven 1 sibling, 0 replies; 12+ messages in thread From: Simon Horman @ 2019-07-09 8:46 UTC (permalink / raw) To: Fabrizio Castro Cc: Geert Uytterhoeven, Rob Herring, Mark Rutland, Magnus Damm, linux-renesas-soc, devicetree, Chris Paterson, Biju Das On Fri, Jul 05, 2019 at 02:39:37PM +0100, Fabrizio Castro wrote: > Basic audio dmac register only supports busif from 0 to 3, > in order to use busif4 ~ busif7 extended audio dmac registers > need to be used. > > Based on similar work from Jiada Wang: > 7a516e49d975 ("arm64: dts: renesas: use extended audio dmac register") > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> Reviewed-by: Simon Horman <horms+renesas@verge.net.au> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register 2019-07-05 13:39 ` [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register Fabrizio Castro 2019-07-09 8:46 ` Simon Horman @ 2019-07-12 13:14 ` Geert Uytterhoeven 1 sibling, 0 replies; 12+ messages in thread From: Geert Uytterhoeven @ 2019-07-12 13:14 UTC (permalink / raw) To: Fabrizio Castro Cc: Geert Uytterhoeven, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm, Linux-Renesas, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Chris Paterson, Biju Das On Fri, Jul 5, 2019 at 3:39 PM Fabrizio Castro <fabrizio.castro@bp.renesas.com> wrote: > Basic audio dmac register only supports busif from 0 to 3, > in order to use busif4 ~ busif7 extended audio dmac registers > need to be used. > > Based on similar work from Jiada Wang: > 7a516e49d975 ("arm64: dts: renesas: use extended audio dmac register") > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.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] 12+ messages in thread
* [PATCH 2/3] arm64: dts: renesas: r8a774a1: Add SSIU support for sound 2019-07-05 13:39 [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] Fabrizio Castro 2019-07-05 13:39 ` [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register Fabrizio Castro @ 2019-07-05 13:39 ` Fabrizio Castro 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:27 ` Geert Uytterhoeven 2019-07-05 13:39 ` [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support Fabrizio Castro 2019-07-12 13:54 ` [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] Geert Uytterhoeven 3 siblings, 2 replies; 12+ messages in thread From: Fabrizio Castro @ 2019-07-05 13:39 UTC (permalink / raw) To: Geert Uytterhoeven, Rob Herring, Mark Rutland Cc: Fabrizio Castro, Simon Horman, Magnus Damm, linux-renesas-soc, devicetree, Chris Paterson, Biju Das Add SSIU support to the SoC DT as the sound driver supports it now, and also since the sound driver can now handle BUSIF0-7 via SSIU remove the no longer needed "rxu" and "txu" properties. Based on similar work from Kuninori Morimoto and Simon Horman: 8d14bfa074db ("arm64: dts: renesas: r8a7796: add SSIU support for sound") 10bd03fa896e ("arm64: dts: renesas: r8a7796: remove BUSIF0 settings from rcar_sound,ssi") Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> --- arch/arm64/boot/dts/renesas/r8a774a1.dtsi | 251 +++++++++++++++++++++++++++--- 1 file changed, 231 insertions(+), 20 deletions(-) diff --git a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi index 2c5173e..aa27097 100644 --- a/arch/arm64/boot/dts/renesas/r8a774a1.dtsi +++ b/arch/arm64/boot/dts/renesas/r8a774a1.dtsi @@ -1777,56 +1777,267 @@ }; }; + rcar_sound,ssiu { + ssiu00: ssiu-0 { + dmas = <&audma0 0x15>, <&audma1 0x16>; + dma-names = "rx", "tx"; + }; + ssiu01: ssiu-1 { + dmas = <&audma0 0x35>, <&audma1 0x36>; + dma-names = "rx", "tx"; + }; + ssiu02: ssiu-2 { + dmas = <&audma0 0x37>, <&audma1 0x38>; + dma-names = "rx", "tx"; + }; + ssiu03: ssiu-3 { + dmas = <&audma0 0x47>, <&audma1 0x48>; + dma-names = "rx", "tx"; + }; + ssiu04: ssiu-4 { + dmas = <&audma0 0x3F>, <&audma1 0x40>; + dma-names = "rx", "tx"; + }; + ssiu05: ssiu-5 { + dmas = <&audma0 0x43>, <&audma1 0x44>; + dma-names = "rx", "tx"; + }; + ssiu06: ssiu-6 { + dmas = <&audma0 0x4F>, <&audma1 0x50>; + dma-names = "rx", "tx"; + }; + ssiu07: ssiu-7 { + dmas = <&audma0 0x53>, <&audma1 0x54>; + dma-names = "rx", "tx"; + }; + ssiu10: ssiu-8 { + dmas = <&audma0 0x49>, <&audma1 0x4a>; + dma-names = "rx", "tx"; + }; + ssiu11: ssiu-9 { + dmas = <&audma0 0x4B>, <&audma1 0x4C>; + dma-names = "rx", "tx"; + }; + ssiu12: ssiu-10 { + dmas = <&audma0 0x57>, <&audma1 0x58>; + dma-names = "rx", "tx"; + }; + ssiu13: ssiu-11 { + dmas = <&audma0 0x59>, <&audma1 0x5A>; + dma-names = "rx", "tx"; + }; + ssiu14: ssiu-12 { + dmas = <&audma0 0x5F>, <&audma1 0x60>; + dma-names = "rx", "tx"; + }; + ssiu15: ssiu-13 { + dmas = <&audma0 0xC3>, <&audma1 0xC4>; + dma-names = "rx", "tx"; + }; + ssiu16: ssiu-14 { + dmas = <&audma0 0xC7>, <&audma1 0xC8>; + dma-names = "rx", "tx"; + }; + ssiu17: ssiu-15 { + dmas = <&audma0 0xCB>, <&audma1 0xCC>; + dma-names = "rx", "tx"; + }; + ssiu20: ssiu-16 { + dmas = <&audma0 0x63>, <&audma1 0x64>; + dma-names = "rx", "tx"; + }; + ssiu21: ssiu-17 { + dmas = <&audma0 0x67>, <&audma1 0x68>; + dma-names = "rx", "tx"; + }; + ssiu22: ssiu-18 { + dmas = <&audma0 0x6B>, <&audma1 0x6C>; + dma-names = "rx", "tx"; + }; + ssiu23: ssiu-19 { + dmas = <&audma0 0x6D>, <&audma1 0x6E>; + dma-names = "rx", "tx"; + }; + ssiu24: ssiu-20 { + dmas = <&audma0 0xCF>, <&audma1 0xCE>; + dma-names = "rx", "tx"; + }; + ssiu25: ssiu-21 { + dmas = <&audma0 0xEB>, <&audma1 0xEC>; + dma-names = "rx", "tx"; + }; + ssiu26: ssiu-22 { + dmas = <&audma0 0xED>, <&audma1 0xEE>; + dma-names = "rx", "tx"; + }; + ssiu27: ssiu-23 { + dmas = <&audma0 0xEF>, <&audma1 0xF0>; + dma-names = "rx", "tx"; + }; + ssiu30: ssiu-24 { + dmas = <&audma0 0x6f>, <&audma1 0x70>; + dma-names = "rx", "tx"; + }; + ssiu31: ssiu-25 { + dmas = <&audma0 0x21>, <&audma1 0x22>; + dma-names = "rx", "tx"; + }; + ssiu32: ssiu-26 { + dmas = <&audma0 0x23>, <&audma1 0x24>; + dma-names = "rx", "tx"; + }; + ssiu33: ssiu-27 { + dmas = <&audma0 0x25>, <&audma1 0x26>; + dma-names = "rx", "tx"; + }; + ssiu34: ssiu-28 { + dmas = <&audma0 0x27>, <&audma1 0x28>; + dma-names = "rx", "tx"; + }; + ssiu35: ssiu-29 { + dmas = <&audma0 0x29>, <&audma1 0x2A>; + dma-names = "rx", "tx"; + }; + ssiu36: ssiu-30 { + dmas = <&audma0 0x2B>, <&audma1 0x2C>; + dma-names = "rx", "tx"; + }; + ssiu37: ssiu-31 { + dmas = <&audma0 0x2D>, <&audma1 0x2E>; + dma-names = "rx", "tx"; + }; + ssiu40: ssiu-32 { + dmas = <&audma0 0x71>, <&audma1 0x72>; + dma-names = "rx", "tx"; + }; + ssiu41: ssiu-33 { + dmas = <&audma0 0x17>, <&audma1 0x18>; + dma-names = "rx", "tx"; + }; + ssiu42: ssiu-34 { + dmas = <&audma0 0x19>, <&audma1 0x1A>; + dma-names = "rx", "tx"; + }; + ssiu43: ssiu-35 { + dmas = <&audma0 0x1B>, <&audma1 0x1C>; + dma-names = "rx", "tx"; + }; + ssiu44: ssiu-36 { + dmas = <&audma0 0x1D>, <&audma1 0x1E>; + dma-names = "rx", "tx"; + }; + ssiu45: ssiu-37 { + dmas = <&audma0 0x1F>, <&audma1 0x20>; + dma-names = "rx", "tx"; + }; + ssiu46: ssiu-38 { + dmas = <&audma0 0x31>, <&audma1 0x32>; + dma-names = "rx", "tx"; + }; + ssiu47: ssiu-39 { + dmas = <&audma0 0x33>, <&audma1 0x34>; + dma-names = "rx", "tx"; + }; + ssiu50: ssiu-40 { + dmas = <&audma0 0x73>, <&audma1 0x74>; + dma-names = "rx", "tx"; + }; + ssiu60: ssiu-41 { + dmas = <&audma0 0x75>, <&audma1 0x76>; + dma-names = "rx", "tx"; + }; + ssiu70: ssiu-42 { + dmas = <&audma0 0x79>, <&audma1 0x7a>; + dma-names = "rx", "tx"; + }; + ssiu80: ssiu-43 { + dmas = <&audma0 0x7b>, <&audma1 0x7c>; + dma-names = "rx", "tx"; + }; + ssiu90: ssiu-44 { + dmas = <&audma0 0x7d>, <&audma1 0x7e>; + dma-names = "rx", "tx"; + }; + ssiu91: ssiu-45 { + dmas = <&audma0 0x7F>, <&audma1 0x80>; + dma-names = "rx", "tx"; + }; + ssiu92: ssiu-46 { + dmas = <&audma0 0x81>, <&audma1 0x82>; + dma-names = "rx", "tx"; + }; + ssiu93: ssiu-47 { + dmas = <&audma0 0x83>, <&audma1 0x84>; + dma-names = "rx", "tx"; + }; + ssiu94: ssiu-48 { + dmas = <&audma0 0xA3>, <&audma1 0xA4>; + dma-names = "rx", "tx"; + }; + ssiu95: ssiu-49 { + dmas = <&audma0 0xA5>, <&audma1 0xA6>; + dma-names = "rx", "tx"; + }; + ssiu96: ssiu-50 { + dmas = <&audma0 0xA7>, <&audma1 0xA8>; + dma-names = "rx", "tx"; + }; + ssiu97: ssiu-51 { + dmas = <&audma0 0xA9>, <&audma1 0xAA>; + dma-names = "rx", "tx"; + }; + }; + rcar_sound,ssi { ssi0: ssi-0 { interrupts = <GIC_SPI 370 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x01>, <&audma1 0x02>, <&audma0 0x15>, <&audma1 0x16>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x01>, <&audma1 0x02>; + dma-names = "rx", "tx"; }; ssi1: ssi-1 { interrupts = <GIC_SPI 371 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x03>, <&audma1 0x04>, <&audma0 0x49>, <&audma1 0x4a>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x03>, <&audma1 0x04>; + dma-names = "rx", "tx"; }; ssi2: ssi-2 { interrupts = <GIC_SPI 372 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x05>, <&audma1 0x06>, <&audma0 0x63>, <&audma1 0x64>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x05>, <&audma1 0x06>; + dma-names = "rx", "tx"; }; ssi3: ssi-3 { interrupts = <GIC_SPI 373 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x07>, <&audma1 0x08>, <&audma0 0x6f>, <&audma1 0x70>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x07>, <&audma1 0x08>; + dma-names = "rx", "tx"; }; ssi4: ssi-4 { interrupts = <GIC_SPI 374 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x09>, <&audma1 0x0a>, <&audma0 0x71>, <&audma1 0x72>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x09>, <&audma1 0x0a>; + dma-names = "rx", "tx"; }; ssi5: ssi-5 { interrupts = <GIC_SPI 375 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x0b>, <&audma1 0x0c>, <&audma0 0x73>, <&audma1 0x74>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x0b>, <&audma1 0x0c>; + dma-names = "rx", "tx"; }; ssi6: ssi-6 { interrupts = <GIC_SPI 376 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x0d>, <&audma1 0x0e>, <&audma0 0x75>, <&audma1 0x76>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x0d>, <&audma1 0x0e>; + dma-names = "rx", "tx"; }; ssi7: ssi-7 { interrupts = <GIC_SPI 377 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x0f>, <&audma1 0x10>, <&audma0 0x79>, <&audma1 0x7a>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x0f>, <&audma1 0x10>; + dma-names = "rx", "tx"; }; ssi8: ssi-8 { interrupts = <GIC_SPI 378 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x11>, <&audma1 0x12>, <&audma0 0x7b>, <&audma1 0x7c>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x11>, <&audma1 0x12>; + dma-names = "rx", "tx"; }; ssi9: ssi-9 { interrupts = <GIC_SPI 379 IRQ_TYPE_LEVEL_HIGH>; - dmas = <&audma0 0x13>, <&audma1 0x14>, <&audma0 0x7d>, <&audma1 0x7e>; - dma-names = "rx", "tx", "rxu", "txu"; + dmas = <&audma0 0x13>, <&audma1 0x14>; + dma-names = "rx", "tx"; }; }; -- 2.7.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 2/3] arm64: dts: renesas: r8a774a1: Add SSIU support for sound 2019-07-05 13:39 ` [PATCH 2/3] arm64: dts: renesas: r8a774a1: Add SSIU support for sound Fabrizio Castro @ 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:27 ` Geert Uytterhoeven 1 sibling, 0 replies; 12+ messages in thread From: Simon Horman @ 2019-07-09 8:46 UTC (permalink / raw) To: Fabrizio Castro Cc: Geert Uytterhoeven, Rob Herring, Mark Rutland, Magnus Damm, linux-renesas-soc, devicetree, Chris Paterson, Biju Das On Fri, Jul 05, 2019 at 02:39:38PM +0100, Fabrizio Castro wrote: > Add SSIU support to the SoC DT as the sound driver supports > it now, and also since the sound driver can now handle > BUSIF0-7 via SSIU remove the no longer needed "rxu" and "txu" > properties. > > Based on similar work from Kuninori Morimoto and Simon Horman: > 8d14bfa074db ("arm64: dts: renesas: r8a7796: add SSIU support for > sound") > 10bd03fa896e ("arm64: dts: renesas: r8a7796: remove BUSIF0 settings from > rcar_sound,ssi") > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> Reviewed-by: Simon Horman <horms+renesas@verge.net.au> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 2/3] arm64: dts: renesas: r8a774a1: Add SSIU support for sound 2019-07-05 13:39 ` [PATCH 2/3] arm64: dts: renesas: r8a774a1: Add SSIU support for sound Fabrizio Castro 2019-07-09 8:46 ` Simon Horman @ 2019-07-12 13:27 ` Geert Uytterhoeven 1 sibling, 0 replies; 12+ messages in thread From: Geert Uytterhoeven @ 2019-07-12 13:27 UTC (permalink / raw) To: Fabrizio Castro Cc: Geert Uytterhoeven, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm, Linux-Renesas, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Chris Paterson, Biju Das On Fri, Jul 5, 2019 at 3:39 PM Fabrizio Castro <fabrizio.castro@bp.renesas.com> wrote: > Add SSIU support to the SoC DT as the sound driver supports > it now, and also since the sound driver can now handle > BUSIF0-7 via SSIU remove the no longer needed "rxu" and "txu" > properties. > > Based on similar work from Kuninori Morimoto and Simon Horman: > 8d14bfa074db ("arm64: dts: renesas: r8a7796: add SSIU support for > sound") > 10bd03fa896e ("arm64: dts: renesas: r8a7796: remove BUSIF0 settings from > rcar_sound,ssi") > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.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] 12+ messages in thread
* [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support 2019-07-05 13:39 [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] Fabrizio Castro 2019-07-05 13:39 ` [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register Fabrizio Castro 2019-07-05 13:39 ` [PATCH 2/3] arm64: dts: renesas: r8a774a1: Add SSIU support for sound Fabrizio Castro @ 2019-07-05 13:39 ` Fabrizio Castro 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:42 ` Geert Uytterhoeven 2019-07-12 13:54 ` [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] Geert Uytterhoeven 3 siblings, 2 replies; 12+ messages in thread From: Fabrizio Castro @ 2019-07-05 13:39 UTC (permalink / raw) To: Geert Uytterhoeven, Rob Herring, Mark Rutland Cc: Fabrizio Castro, Simon Horman, Magnus Damm, linux-renesas-soc, devicetree, Chris Paterson, Biju Das, xu_shunji This patch adds support for HDMI audio to the device tree common to the HiHope RZ/G2M and the HiHope RZ/G2N. Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> --- arch/arm64/boot/dts/renesas/hihope-common.dtsi | 48 ++++++++++++++++++++++++++ 1 file changed, 48 insertions(+) diff --git a/arch/arm64/boot/dts/renesas/hihope-common.dtsi b/arch/arm64/boot/dts/renesas/hihope-common.dtsi index 52d61c9..67f22ce 100644 --- a/arch/arm64/boot/dts/renesas/hihope-common.dtsi +++ b/arch/arm64/boot/dts/renesas/hihope-common.dtsi @@ -81,6 +81,14 @@ regulator-always-on; }; + sound_card: sound { + compatible = "audio-graph-card"; + + label = "rcar-sound"; + + dais = <&rsnd_port0>; + }; + vbus0_usb2: regulator-vbus0-usb2 { compatible = "regulator-fixed"; @@ -129,6 +137,10 @@ }; }; +&audio_clk_a { + clock-frequency = <22579200>; +}; + &du { clocks = <&cpg CPG_MOD 724>, <&cpg CPG_MOD 723>, @@ -168,6 +180,7 @@ &hdmi0 { status = "okay"; + #sound-dai-cells = <0>; ports { port@1 { @@ -176,6 +189,12 @@ remote-endpoint = <&hdmi0_con>; }; }; + port@2 { + reg = <2>; + dw_hdmi0_snd_in: endpoint { + remote-endpoint = <&rsnd_endpoint0>; + }; + }; }; }; @@ -272,6 +291,11 @@ power-source = <1800>; }; + sound_clk_pins: sound_clk { + groups = "audio_clk_a_a"; + function = "audio_clk"; + }; + usb0_pins: usb0 { groups = "usb0"; function = "usb0"; @@ -295,6 +319,30 @@ }; }; +&rcar_sound { + pinctrl-0 = <&sound_clk_pins>; + pinctrl-names = "default"; + + status = "okay"; + + /* Single DAI */ + #sound-dai-cells = <0>; + + ports { + rsnd_port0: port@0 { + rsnd_endpoint0: endpoint { + remote-endpoint = <&dw_hdmi0_snd_in>; + + dai-format = "i2s"; + bitclock-master = <&rsnd_endpoint0>; + frame-master = <&rsnd_endpoint0>; + + playback = <&ssi2>; + }; + }; + }; +}; + &rwdt { timeout-sec = <60>; status = "okay"; -- 2.7.4 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support 2019-07-05 13:39 ` [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support Fabrizio Castro @ 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:42 ` Geert Uytterhoeven 1 sibling, 0 replies; 12+ messages in thread From: Simon Horman @ 2019-07-09 8:46 UTC (permalink / raw) To: Fabrizio Castro Cc: Geert Uytterhoeven, Rob Herring, Mark Rutland, Magnus Damm, linux-renesas-soc, devicetree, Chris Paterson, Biju Das, xu_shunji On Fri, Jul 05, 2019 at 02:39:39PM +0100, Fabrizio Castro wrote: > This patch adds support for HDMI audio to the device tree > common to the HiHope RZ/G2M and the HiHope RZ/G2N. > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> Reviewed-by: Simon Horman <horms+renesas@verge.net.au> ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support 2019-07-05 13:39 ` [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support Fabrizio Castro 2019-07-09 8:46 ` Simon Horman @ 2019-07-12 13:42 ` Geert Uytterhoeven 2019-07-15 9:29 ` Fabrizio Castro 1 sibling, 1 reply; 12+ messages in thread From: Geert Uytterhoeven @ 2019-07-12 13:42 UTC (permalink / raw) To: Fabrizio Castro Cc: Geert Uytterhoeven, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm, Linux-Renesas, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Chris Paterson, Biju Das, xu_shunji Hi Fabrizio, On Fri, Jul 5, 2019 at 3:40 PM Fabrizio Castro <fabrizio.castro@bp.renesas.com> wrote: > This patch adds support for HDMI audio to the device tree > common to the HiHope RZ/G2M and the HiHope RZ/G2N. > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> Thanks for your patch! > --- a/arch/arm64/boot/dts/renesas/hihope-common.dtsi > +++ b/arch/arm64/boot/dts/renesas/hihope-common.dtsi > @@ -168,6 +180,7 @@ > > &hdmi0 { > status = "okay"; > + #sound-dai-cells = <0>; Why the above line? With the above question answered: 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] 12+ messages in thread
* RE: [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support 2019-07-12 13:42 ` Geert Uytterhoeven @ 2019-07-15 9:29 ` Fabrizio Castro 0 siblings, 0 replies; 12+ messages in thread From: Fabrizio Castro @ 2019-07-15 9:29 UTC (permalink / raw) To: Geert Uytterhoeven Cc: Geert Uytterhoeven, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm, Linux-Renesas, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Chris Paterson, Biju Das, xu_shunji Hi Geert, Thank you for your feedback! > From: Geert Uytterhoeven <geert@linux-m68k.org> > Sent: 12 July 2019 14:42 > Subject: Re: [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support > > Hi Fabrizio, > > On Fri, Jul 5, 2019 at 3:40 PM Fabrizio Castro > <fabrizio.castro@bp.renesas.com> wrote: > > This patch adds support for HDMI audio to the device tree > > common to the HiHope RZ/G2M and the HiHope RZ/G2N. > > > > Signed-off-by: Fabrizio Castro <fabrizio.castro@bp.renesas.com> > > Thanks for your patch! > > > --- a/arch/arm64/boot/dts/renesas/hihope-common.dtsi > > +++ b/arch/arm64/boot/dts/renesas/hihope-common.dtsi > > > @@ -168,6 +180,7 @@ > > > > &hdmi0 { > > status = "okay"; > > + #sound-dai-cells = <0>; > > Why the above line? It doesn't belong there, good catch, I'll send a v2 shortly without it. Thanks, Fab > > With the above question answered: > > 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] 12+ messages in thread
* Re: [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] 2019-07-05 13:39 [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] Fabrizio Castro ` (2 preceding siblings ...) 2019-07-05 13:39 ` [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support Fabrizio Castro @ 2019-07-12 13:54 ` Geert Uytterhoeven 3 siblings, 0 replies; 12+ messages in thread From: Geert Uytterhoeven @ 2019-07-12 13:54 UTC (permalink / raw) To: Fabrizio Castro Cc: Geert Uytterhoeven, Rob Herring, Mark Rutland, Simon Horman, Magnus Damm, Linux-Renesas, open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS, Chris Paterson, Biju Das, xu_shunji Hi Fabrizio, On Fri, Jul 5, 2019 at 3:39 PM Fabrizio Castro <fabrizio.castro@bp.renesas.com> wrote: > this series adds HDMI audio support to the HiHope RZ/G2[MN]. > > Thanks, > Fab > > Fabrizio Castro (3): > arm64: dts: renesas: r8a774a1: Use extended audio dmac register > arm64: dts: renesas: r8a774a1: Add SSIU support for sound > arm64: dts: renesas: hihope-common: Add HDMI audio support Thanks, I have applied and queued for v5.4 the first two patches. 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] 12+ messages in thread
end of thread, other threads:[~2019-07-15 9:29 UTC | newest] Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-07-05 13:39 [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] Fabrizio Castro 2019-07-05 13:39 ` [PATCH 1/3] arm64: dts: renesas: r8a774a1: Use extended audio dmac register Fabrizio Castro 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:14 ` Geert Uytterhoeven 2019-07-05 13:39 ` [PATCH 2/3] arm64: dts: renesas: r8a774a1: Add SSIU support for sound Fabrizio Castro 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:27 ` Geert Uytterhoeven 2019-07-05 13:39 ` [PATCH 3/3] arm64: dts: renesas: hihope-common: Add HDMI audio support Fabrizio Castro 2019-07-09 8:46 ` Simon Horman 2019-07-12 13:42 ` Geert Uytterhoeven 2019-07-15 9:29 ` Fabrizio Castro 2019-07-12 13:54 ` [PATCH 0/3] Add HDMI audio support to the HiHope RZ/G2[MN] Geert Uytterhoeven
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).