All of lore.kernel.org
 help / color / mirror / Atom feed
* [linux-sunxi][PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes
@ 2017-09-03 15:08 ` codekipper at gmail.com
  0 siblings, 0 replies; 8+ messages in thread
From: codekipper-Re5JQEeQqe8AvxtiuMwx3w @ 2017-09-03 15:08 UTC (permalink / raw)
  To: maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8
  Cc: linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-sunxi-/JYPxA39Uh5TLH3MbocFFw, Marcus Cooper

From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

Add the new DAI blocks to the device tree. I2S0 and I2S1 are for
connecting to an external codec whereas I2S2 is used for HDMI
audio.

Signed-off-by: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 arch/arm/boot/dts/sunxi-h3-h5.dtsi | 39 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
index 11240a8313c2..cf6fcc857324 100644
--- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
+++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
@@ -450,6 +450,45 @@
 			status = "disabled";
 		};
 
+		i2s0: i2s@01c22000 {
+			#sound-dai-cells = <0>;
+			compatible = "allwinner,sun8i-h3-i2s";
+			reg = <0x01c22000 0x400>;
+			interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&ccu CLK_BUS_I2S0>, <&ccu CLK_I2S0>;
+			clock-names = "apb", "mod";
+			dmas = <&dma 3>, <&dma 3>;
+			resets = <&ccu RST_BUS_I2S0>;
+			dma-names = "rx", "tx";
+			status = "disabled";
+		};
+
+		i2s1: i2s@01c22400 {
+			#sound-dai-cells = <0>;
+			compatible = "allwinner,sun8i-h3-i2s";
+			reg = <0x01c22400 0x400>;
+			interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&ccu CLK_BUS_I2S1>, <&ccu CLK_I2S1>;
+			clock-names = "apb", "mod";
+			dmas = <&dma 4>, <&dma 4>;
+			resets = <&ccu RST_BUS_I2S1>;
+			dma-names = "rx", "tx";
+			status = "disabled";
+		};
+
+		i2s2: i2s@01c22800 {
+			#sound-dai-cells = <0>;
+			compatible = "allwinner,sun8i-h3-i2s";
+			reg = <0x01c22800 0x400>;
+			interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&ccu CLK_BUS_I2S2>, <&ccu CLK_I2S2>;
+			clock-names = "apb", "mod";
+			dmas = <&dma 5>, <&dma 5>;
+			resets = <&ccu RST_BUS_I2S2>;
+			dma-names = "rx", "tx";
+			status = "disabled";
+		};
+
 		codec: codec@01c22c00 {
 			#sound-dai-cells = <0>;
 			compatible = "allwinner,sun8i-h3-codec";
-- 
2.14.1

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* [linux-sunxi][PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes
@ 2017-09-03 15:08 ` codekipper at gmail.com
  0 siblings, 0 replies; 8+ messages in thread
From: codekipper at gmail.com @ 2017-09-03 15:08 UTC (permalink / raw)
  To: linux-arm-kernel

From: Marcus Cooper <codekipper@gmail.com>

Add the new DAI blocks to the device tree. I2S0 and I2S1 are for
connecting to an external codec whereas I2S2 is used for HDMI
audio.

Signed-off-by: Marcus Cooper <codekipper@gmail.com>
---
 arch/arm/boot/dts/sunxi-h3-h5.dtsi | 39 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
index 11240a8313c2..cf6fcc857324 100644
--- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
+++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
@@ -450,6 +450,45 @@
 			status = "disabled";
 		};
 
+		i2s0: i2s at 01c22000 {
+			#sound-dai-cells = <0>;
+			compatible = "allwinner,sun8i-h3-i2s";
+			reg = <0x01c22000 0x400>;
+			interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&ccu CLK_BUS_I2S0>, <&ccu CLK_I2S0>;
+			clock-names = "apb", "mod";
+			dmas = <&dma 3>, <&dma 3>;
+			resets = <&ccu RST_BUS_I2S0>;
+			dma-names = "rx", "tx";
+			status = "disabled";
+		};
+
+		i2s1: i2s at 01c22400 {
+			#sound-dai-cells = <0>;
+			compatible = "allwinner,sun8i-h3-i2s";
+			reg = <0x01c22400 0x400>;
+			interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&ccu CLK_BUS_I2S1>, <&ccu CLK_I2S1>;
+			clock-names = "apb", "mod";
+			dmas = <&dma 4>, <&dma 4>;
+			resets = <&ccu RST_BUS_I2S1>;
+			dma-names = "rx", "tx";
+			status = "disabled";
+		};
+
+		i2s2: i2s at 01c22800 {
+			#sound-dai-cells = <0>;
+			compatible = "allwinner,sun8i-h3-i2s";
+			reg = <0x01c22800 0x400>;
+			interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>;
+			clocks = <&ccu CLK_BUS_I2S2>, <&ccu CLK_I2S2>;
+			clock-names = "apb", "mod";
+			dmas = <&dma 5>, <&dma 5>;
+			resets = <&ccu RST_BUS_I2S2>;
+			dma-names = "rx", "tx";
+			status = "disabled";
+		};
+
 		codec: codec at 01c22c00 {
 			#sound-dai-cells = <0>;
 			compatible = "allwinner,sun8i-h3-codec";
-- 
2.14.1

^ permalink raw reply related	[flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes
  2017-09-03 15:08 ` codekipper at gmail.com
@ 2017-09-03 16:35     ` Jernej Škrabec
  -1 siblings, 0 replies; 8+ messages in thread
From: Jernej Škrabec @ 2017-09-03 16:35 UTC (permalink / raw)
  To: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw, codekipper-Re5JQEeQqe8AvxtiuMwx3w
  Cc: maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r,
	devicetree-u79uwXL29TY76Z2rM5mHXA

Hi Marcus!

Dne nedelja, 03. september 2017 ob 17:08:06 CEST je codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org 
napisal(a):
> From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> 
> Add the new DAI blocks to the device tree. I2S0 and I2S1 are for
> connecting to an external codec whereas I2S2 is used for HDMI
> audio.
> 
> Signed-off-by: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  arch/arm/boot/dts/sunxi-h3-h5.dtsi | 39
> ++++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 11240a8313c2..cf6fcc857324
> 100644
> --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> @@ -450,6 +450,45 @@
>  			status = "disabled";
>  		};
> 
> +		i2s0: i2s@01c22000 {
> +			#sound-dai-cells = <0>;
> +			compatible = "allwinner,sun8i-h3-i2s";
> +			reg = <0x01c22000 0x400>;
> +			interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&ccu CLK_BUS_I2S0>, <&ccu CLK_I2S0>;
> +			clock-names = "apb", "mod";
> +			dmas = <&dma 3>, <&dma 3>;
> +			resets = <&ccu RST_BUS_I2S0>;
> +			dma-names = "rx", "tx";
> +			status = "disabled";
> +		};
> +
> +		i2s1: i2s@01c22400 {
> +			#sound-dai-cells = <0>;
> +			compatible = "allwinner,sun8i-h3-i2s";
> +			reg = <0x01c22400 0x400>;
> +			interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&ccu CLK_BUS_I2S1>, <&ccu CLK_I2S1>;
> +			clock-names = "apb", "mod";
> +			dmas = <&dma 4>, <&dma 4>;
> +			resets = <&ccu RST_BUS_I2S1>;
> +			dma-names = "rx", "tx";
> +			status = "disabled";
> +		};
> +
> +		i2s2: i2s@01c22800 {
> +			#sound-dai-cells = <0>;
> +			compatible = "allwinner,sun8i-h3-i2s";
> +			reg = <0x01c22800 0x400>;
> +			interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&ccu CLK_BUS_I2S2>, <&ccu CLK_I2S2>;
> +			clock-names = "apb", "mod";
> +			dmas = <&dma 5>, <&dma 5>;
> +			resets = <&ccu RST_BUS_I2S2>;
> +			dma-names = "rx", "tx";
> +			status = "disabled";
> +		};
> +

I don't want to be a pain, but shouldn't we skip this one for now? It is only 
for HDMI and has different capabilities (8 channels supported and slighlty 
different formula to calculate oversample rate.

Best regards,
Jernej

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [linux-sunxi][PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes
@ 2017-09-03 16:35     ` Jernej Škrabec
  0 siblings, 0 replies; 8+ messages in thread
From: Jernej Škrabec @ 2017-09-03 16:35 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Marcus!

Dne nedelja, 03. september 2017 ob 17:08:06 CEST je codekipper at gmail.com 
napisal(a):
> From: Marcus Cooper <codekipper@gmail.com>
> 
> Add the new DAI blocks to the device tree. I2S0 and I2S1 are for
> connecting to an external codec whereas I2S2 is used for HDMI
> audio.
> 
> Signed-off-by: Marcus Cooper <codekipper@gmail.com>
> ---
>  arch/arm/boot/dts/sunxi-h3-h5.dtsi | 39
> ++++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 11240a8313c2..cf6fcc857324
> 100644
> --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> @@ -450,6 +450,45 @@
>  			status = "disabled";
>  		};
> 
> +		i2s0: i2s at 01c22000 {
> +			#sound-dai-cells = <0>;
> +			compatible = "allwinner,sun8i-h3-i2s";
> +			reg = <0x01c22000 0x400>;
> +			interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&ccu CLK_BUS_I2S0>, <&ccu CLK_I2S0>;
> +			clock-names = "apb", "mod";
> +			dmas = <&dma 3>, <&dma 3>;
> +			resets = <&ccu RST_BUS_I2S0>;
> +			dma-names = "rx", "tx";
> +			status = "disabled";
> +		};
> +
> +		i2s1: i2s at 01c22400 {
> +			#sound-dai-cells = <0>;
> +			compatible = "allwinner,sun8i-h3-i2s";
> +			reg = <0x01c22400 0x400>;
> +			interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&ccu CLK_BUS_I2S1>, <&ccu CLK_I2S1>;
> +			clock-names = "apb", "mod";
> +			dmas = <&dma 4>, <&dma 4>;
> +			resets = <&ccu RST_BUS_I2S1>;
> +			dma-names = "rx", "tx";
> +			status = "disabled";
> +		};
> +
> +		i2s2: i2s at 01c22800 {
> +			#sound-dai-cells = <0>;
> +			compatible = "allwinner,sun8i-h3-i2s";
> +			reg = <0x01c22800 0x400>;
> +			interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>;
> +			clocks = <&ccu CLK_BUS_I2S2>, <&ccu CLK_I2S2>;
> +			clock-names = "apb", "mod";
> +			dmas = <&dma 5>, <&dma 5>;
> +			resets = <&ccu RST_BUS_I2S2>;
> +			dma-names = "rx", "tx";
> +			status = "disabled";
> +		};
> +

I don't want to be a pain, but shouldn't we skip this one for now? It is only 
for HDMI and has different capabilities (8 channels supported and slighlty 
different formula to calculate oversample rate.

Best regards,
Jernej

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes
  2017-09-03 16:35     ` [linux-sunxi][PATCH] " Jernej Škrabec
@ 2017-09-04 10:51       ` Code Kipper
  -1 siblings, 0 replies; 8+ messages in thread
From: Code Kipper @ 2017-09-04 10:51 UTC (permalink / raw)
  To: Jernej Škrabec
  Cc: linux-sunxi, Maxime Ripard, linux-arm-kernel, devicetree

On 3 September 2017 at 18:35, Jernej Škrabec <jernej.skrabec-gGgVlfcn5nU@public.gmane.org> wrote:
> Hi Marcus!
>
> Dne nedelja, 03. september 2017 ob 17:08:06 CEST je codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
> napisal(a):
>> From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>>
>> Add the new DAI blocks to the device tree. I2S0 and I2S1 are for
>> connecting to an external codec whereas I2S2 is used for HDMI
>> audio.
>>
>> Signed-off-by: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
>> ---
>>  arch/arm/boot/dts/sunxi-h3-h5.dtsi | 39
>> ++++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
>> b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 11240a8313c2..cf6fcc857324
>> 100644
>> --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
>> +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
>> @@ -450,6 +450,45 @@
>>                       status = "disabled";
>>               };
>>
>> +             i2s0: i2s@01c22000 {
>> +                     #sound-dai-cells = <0>;
>> +                     compatible = "allwinner,sun8i-h3-i2s";
>> +                     reg = <0x01c22000 0x400>;
>> +                     interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> +                     clocks = <&ccu CLK_BUS_I2S0>, <&ccu CLK_I2S0>;
>> +                     clock-names = "apb", "mod";
>> +                     dmas = <&dma 3>, <&dma 3>;
>> +                     resets = <&ccu RST_BUS_I2S0>;
>> +                     dma-names = "rx", "tx";
>> +                     status = "disabled";
>> +             };
>> +
>> +             i2s1: i2s@01c22400 {
>> +                     #sound-dai-cells = <0>;
>> +                     compatible = "allwinner,sun8i-h3-i2s";
>> +                     reg = <0x01c22400 0x400>;
>> +                     interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> +                     clocks = <&ccu CLK_BUS_I2S1>, <&ccu CLK_I2S1>;
>> +                     clock-names = "apb", "mod";
>> +                     dmas = <&dma 4>, <&dma 4>;
>> +                     resets = <&ccu RST_BUS_I2S1>;
>> +                     dma-names = "rx", "tx";
>> +                     status = "disabled";
>> +             };
>> +
>> +             i2s2: i2s@01c22800 {
>> +                     #sound-dai-cells = <0>;
>> +                     compatible = "allwinner,sun8i-h3-i2s";
>> +                     reg = <0x01c22800 0x400>;
>> +                     interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>;
>> +                     clocks = <&ccu CLK_BUS_I2S2>, <&ccu CLK_I2S2>;
>> +                     clock-names = "apb", "mod";
>> +                     dmas = <&dma 5>, <&dma 5>;
>> +                     resets = <&ccu RST_BUS_I2S2>;
>> +                     dma-names = "rx", "tx";
>> +                     status = "disabled";
>> +             };
>> +
>
> I don't want to be a pain, but shouldn't we skip this one for now? It is only
> for HDMI and has different capabilities (8 channels supported and slighlty
> different formula to calculate oversample rate.
I'll have a look into this..maybe this can be handled by the
simple-audio-card binding that we use. The other thing with the
padding which you mentioned I can add a field for. Maxime, what's your
thoughts on this?..I can re-submit without i2s2.
BR,
CK
>
> Best regards,
> Jernej

-- 
You received this message because you are subscribed to the Google Groups "linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org
For more options, visit https://groups.google.com/d/optout.

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [linux-sunxi][PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes
@ 2017-09-04 10:51       ` Code Kipper
  0 siblings, 0 replies; 8+ messages in thread
From: Code Kipper @ 2017-09-04 10:51 UTC (permalink / raw)
  To: linux-arm-kernel

On 3 September 2017 at 18:35, Jernej ?krabec <jernej.skrabec@siol.net> wrote:
> Hi Marcus!
>
> Dne nedelja, 03. september 2017 ob 17:08:06 CEST je codekipper at gmail.com
> napisal(a):
>> From: Marcus Cooper <codekipper@gmail.com>
>>
>> Add the new DAI blocks to the device tree. I2S0 and I2S1 are for
>> connecting to an external codec whereas I2S2 is used for HDMI
>> audio.
>>
>> Signed-off-by: Marcus Cooper <codekipper@gmail.com>
>> ---
>>  arch/arm/boot/dts/sunxi-h3-h5.dtsi | 39
>> ++++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+)
>>
>> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
>> b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 11240a8313c2..cf6fcc857324
>> 100644
>> --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
>> +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
>> @@ -450,6 +450,45 @@
>>                       status = "disabled";
>>               };
>>
>> +             i2s0: i2s at 01c22000 {
>> +                     #sound-dai-cells = <0>;
>> +                     compatible = "allwinner,sun8i-h3-i2s";
>> +                     reg = <0x01c22000 0x400>;
>> +                     interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
>> +                     clocks = <&ccu CLK_BUS_I2S0>, <&ccu CLK_I2S0>;
>> +                     clock-names = "apb", "mod";
>> +                     dmas = <&dma 3>, <&dma 3>;
>> +                     resets = <&ccu RST_BUS_I2S0>;
>> +                     dma-names = "rx", "tx";
>> +                     status = "disabled";
>> +             };
>> +
>> +             i2s1: i2s at 01c22400 {
>> +                     #sound-dai-cells = <0>;
>> +                     compatible = "allwinner,sun8i-h3-i2s";
>> +                     reg = <0x01c22400 0x400>;
>> +                     interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
>> +                     clocks = <&ccu CLK_BUS_I2S1>, <&ccu CLK_I2S1>;
>> +                     clock-names = "apb", "mod";
>> +                     dmas = <&dma 4>, <&dma 4>;
>> +                     resets = <&ccu RST_BUS_I2S1>;
>> +                     dma-names = "rx", "tx";
>> +                     status = "disabled";
>> +             };
>> +
>> +             i2s2: i2s at 01c22800 {
>> +                     #sound-dai-cells = <0>;
>> +                     compatible = "allwinner,sun8i-h3-i2s";
>> +                     reg = <0x01c22800 0x400>;
>> +                     interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>;
>> +                     clocks = <&ccu CLK_BUS_I2S2>, <&ccu CLK_I2S2>;
>> +                     clock-names = "apb", "mod";
>> +                     dmas = <&dma 5>, <&dma 5>;
>> +                     resets = <&ccu RST_BUS_I2S2>;
>> +                     dma-names = "rx", "tx";
>> +                     status = "disabled";
>> +             };
>> +
>
> I don't want to be a pain, but shouldn't we skip this one for now? It is only
> for HDMI and has different capabilities (8 channels supported and slighlty
> different formula to calculate oversample rate.
I'll have a look into this..maybe this can be handled by the
simple-audio-card binding that we use. The other thing with the
padding which you mentioned I can add a field for. Maxime, what's your
thoughts on this?..I can re-submit without i2s2.
BR,
CK
>
> Best regards,
> Jernej

^ permalink raw reply	[flat|nested] 8+ messages in thread
* Re: [linux-sunxi][PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes
  2017-09-04 10:51       ` [linux-sunxi][PATCH] " Code Kipper
@ 2017-09-04 11:40           ` Maxime Ripard
  -1 siblings, 0 replies; 8+ messages in thread
From: Maxime Ripard @ 2017-09-04 11:40 UTC (permalink / raw)
  To: Code Kipper
  Cc: Jernej Škrabec, linux-sunxi, linux-arm-kernel, devicetree

[-- Attachment #1: Type: text/plain, Size: 3743 bytes --]

On Mon, Sep 04, 2017 at 12:51:38PM +0200, Code Kipper wrote:
> On 3 September 2017 at 18:35, Jernej Škrabec <jernej.skrabec-ix9DCk4F938@public.gmane.orgt> wrote:
> > Hi Marcus!
> >
> > Dne nedelja, 03. september 2017 ob 17:08:06 CEST je codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org
> > napisal(a):
> >> From: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> >>
> >> Add the new DAI blocks to the device tree. I2S0 and I2S1 are for
> >> connecting to an external codec whereas I2S2 is used for HDMI
> >> audio.
> >>
> >> Signed-off-by: Marcus Cooper <codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> >> ---
> >>  arch/arm/boot/dts/sunxi-h3-h5.dtsi | 39
> >> ++++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> >> b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 11240a8313c2..cf6fcc857324
> >> 100644
> >> --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> >> +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> >> @@ -450,6 +450,45 @@
> >>                       status = "disabled";
> >>               };
> >>
> >> +             i2s0: i2s@01c22000 {
> >> +                     #sound-dai-cells = <0>;
> >> +                     compatible = "allwinner,sun8i-h3-i2s";
> >> +                     reg = <0x01c22000 0x400>;
> >> +                     interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> >> +                     clocks = <&ccu CLK_BUS_I2S0>, <&ccu CLK_I2S0>;
> >> +                     clock-names = "apb", "mod";
> >> +                     dmas = <&dma 3>, <&dma 3>;
> >> +                     resets = <&ccu RST_BUS_I2S0>;
> >> +                     dma-names = "rx", "tx";
> >> +                     status = "disabled";
> >> +             };
> >> +
> >> +             i2s1: i2s@01c22400 {
> >> +                     #sound-dai-cells = <0>;
> >> +                     compatible = "allwinner,sun8i-h3-i2s";
> >> +                     reg = <0x01c22400 0x400>;
> >> +                     interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> >> +                     clocks = <&ccu CLK_BUS_I2S1>, <&ccu CLK_I2S1>;
> >> +                     clock-names = "apb", "mod";
> >> +                     dmas = <&dma 4>, <&dma 4>;
> >> +                     resets = <&ccu RST_BUS_I2S1>;
> >> +                     dma-names = "rx", "tx";
> >> +                     status = "disabled";
> >> +             };
> >> +
> >> +             i2s2: i2s@01c22800 {
> >> +                     #sound-dai-cells = <0>;
> >> +                     compatible = "allwinner,sun8i-h3-i2s";
> >> +                     reg = <0x01c22800 0x400>;
> >> +                     interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>;
> >> +                     clocks = <&ccu CLK_BUS_I2S2>, <&ccu CLK_I2S2>;
> >> +                     clock-names = "apb", "mod";
> >> +                     dmas = <&dma 5>, <&dma 5>;
> >> +                     resets = <&ccu RST_BUS_I2S2>;
> >> +                     dma-names = "rx", "tx";
> >> +                     status = "disabled";
> >> +             };
> >> +
> >
> > I don't want to be a pain, but shouldn't we skip this one for now? It is only
> > for HDMI and has different capabilities (8 channels supported and slighlty
> > different formula to calculate oversample rate.
> I'll have a look into this..maybe this can be handled by the
> simple-audio-card binding that we use. The other thing with the
> padding which you mentioned I can add a field for. Maxime, what's your
> thoughts on this?..I can re-submit without i2s2.

Yes, please do so.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread
* [linux-sunxi][PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes
@ 2017-09-04 11:40           ` Maxime Ripard
  0 siblings, 0 replies; 8+ messages in thread
From: Maxime Ripard @ 2017-09-04 11:40 UTC (permalink / raw)
  To: linux-arm-kernel

On Mon, Sep 04, 2017 at 12:51:38PM +0200, Code Kipper wrote:
> On 3 September 2017 at 18:35, Jernej ?krabec <jernej.skrabec@siol.net> wrote:
> > Hi Marcus!
> >
> > Dne nedelja, 03. september 2017 ob 17:08:06 CEST je codekipper at gmail.com
> > napisal(a):
> >> From: Marcus Cooper <codekipper@gmail.com>
> >>
> >> Add the new DAI blocks to the device tree. I2S0 and I2S1 are for
> >> connecting to an external codec whereas I2S2 is used for HDMI
> >> audio.
> >>
> >> Signed-off-by: Marcus Cooper <codekipper@gmail.com>
> >> ---
> >>  arch/arm/boot/dts/sunxi-h3-h5.dtsi | 39
> >> ++++++++++++++++++++++++++++++++++++++ 1 file changed, 39 insertions(+)
> >>
> >> diff --git a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> >> b/arch/arm/boot/dts/sunxi-h3-h5.dtsi index 11240a8313c2..cf6fcc857324
> >> 100644
> >> --- a/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> >> +++ b/arch/arm/boot/dts/sunxi-h3-h5.dtsi
> >> @@ -450,6 +450,45 @@
> >>                       status = "disabled";
> >>               };
> >>
> >> +             i2s0: i2s at 01c22000 {
> >> +                     #sound-dai-cells = <0>;
> >> +                     compatible = "allwinner,sun8i-h3-i2s";
> >> +                     reg = <0x01c22000 0x400>;
> >> +                     interrupts = <GIC_SPI 13 IRQ_TYPE_LEVEL_HIGH>;
> >> +                     clocks = <&ccu CLK_BUS_I2S0>, <&ccu CLK_I2S0>;
> >> +                     clock-names = "apb", "mod";
> >> +                     dmas = <&dma 3>, <&dma 3>;
> >> +                     resets = <&ccu RST_BUS_I2S0>;
> >> +                     dma-names = "rx", "tx";
> >> +                     status = "disabled";
> >> +             };
> >> +
> >> +             i2s1: i2s at 01c22400 {
> >> +                     #sound-dai-cells = <0>;
> >> +                     compatible = "allwinner,sun8i-h3-i2s";
> >> +                     reg = <0x01c22400 0x400>;
> >> +                     interrupts = <GIC_SPI 14 IRQ_TYPE_LEVEL_HIGH>;
> >> +                     clocks = <&ccu CLK_BUS_I2S1>, <&ccu CLK_I2S1>;
> >> +                     clock-names = "apb", "mod";
> >> +                     dmas = <&dma 4>, <&dma 4>;
> >> +                     resets = <&ccu RST_BUS_I2S1>;
> >> +                     dma-names = "rx", "tx";
> >> +                     status = "disabled";
> >> +             };
> >> +
> >> +             i2s2: i2s at 01c22800 {
> >> +                     #sound-dai-cells = <0>;
> >> +                     compatible = "allwinner,sun8i-h3-i2s";
> >> +                     reg = <0x01c22800 0x400>;
> >> +                     interrupts = <GIC_SPI 15 IRQ_TYPE_LEVEL_HIGH>;
> >> +                     clocks = <&ccu CLK_BUS_I2S2>, <&ccu CLK_I2S2>;
> >> +                     clock-names = "apb", "mod";
> >> +                     dmas = <&dma 5>, <&dma 5>;
> >> +                     resets = <&ccu RST_BUS_I2S2>;
> >> +                     dma-names = "rx", "tx";
> >> +                     status = "disabled";
> >> +             };
> >> +
> >
> > I don't want to be a pain, but shouldn't we skip this one for now? It is only
> > for HDMI and has different capabilities (8 channels supported and slighlty
> > different formula to calculate oversample rate.
> I'll have a look into this..maybe this can be handled by the
> simple-audio-card binding that we use. The other thing with the
> padding which you mentioned I can add a field for. Maxime, what's your
> thoughts on this?..I can re-submit without i2s2.

Yes, please do so.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20170904/de4520fc/attachment.sig>

^ permalink raw reply	[flat|nested] 8+ messages in thread
end of thread, other threads:[~2017-09-04 11:40 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-03 15:08 [linux-sunxi][PATCH] ARM: dts: sunxi: h3/h5 :Add DAI nodes codekipper-Re5JQEeQqe8AvxtiuMwx3w
2017-09-03 15:08 ` codekipper at gmail.com
     [not found] ` <20170903150806.3451-1-codekipper-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2017-09-03 16:35   ` [PATCH] " Jernej Škrabec
2017-09-03 16:35     ` [linux-sunxi][PATCH] " Jernej Škrabec
2017-09-04 10:51     ` [PATCH] " Code Kipper
2017-09-04 10:51       ` [linux-sunxi][PATCH] " Code Kipper
     [not found]       ` <CAEKpxB=MF-8BPS_Jc=qgj9sTCfn9EKWxdtJQD4goRGSv0xnHMw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2017-09-04 11:40         ` Maxime Ripard
2017-09-04 11:40           ` Maxime Ripard

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.