[PATCH 1/2] ARM: tegra: Add Tegra124 HDA support

[PATCH 1/2] ARM: tegra: Add Tegra124 HDA support

[Dylan Reid]

Add a device node for the HDA controller found on Tegra124.

Signed-off-by: Dylan Reid <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
---
 arch/arm/boot/dts/tegra124.dtsi | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi
index 6e6bc4e..7890641 100644
--- a/arch/arm/boot/dts/tegra124.dtsi
+++ b/arch/arm/boot/dts/tegra124.dtsi
@@ -449,6 +449,21 @@
 		clock-names = "pclk", "clk32k_in";
 	};
 
+	hda@0,70030000 {
+		compatible = "nvidia,tegra124-hda", "nvidia,tegra30-hda";
+		reg = <0x0 0x70030000 0x0 0x10000>;
+		interrupts = <GIC_SPI 81 IRQ_TYPE_LEVEL_HIGH>;
+		clocks = <&tegra_car TEGRA124_CLK_HDA>,
+		         <&tegra_car TEGRA124_CLK_HDA2HDMI>,
+			 <&tegra_car TEGRA124_CLK_HDA2CODEC_2X>;
+		clock-names = "hda", "hda2hdmi", "hdacodec_2x";
+		resets = <&tegra_car 125>, /* hda */
+			 <&tegra_car 128>, /* hda2hdmi */
+			 <&tegra_car 111>; /* hda2codec_2x */
+		reset-names = "hda", "hda2hdmi", "hdacodec_2x";
+		status = "disabled";
+	};
+
 	sdhci@0,700b0000 {
 		compatible = "nvidia,tegra124-sdhci";
 		reg = <0x0 0x700b0000 0x0 0x200>;
-- 
1.8.1.3.605.g02339dd

--
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

[PATCH 2/2] ARM: tegra: venice2 - Enable HDA

[Dylan Reid]

Turn on the HDA controller in Venice2, it is used for HDMI audio.

Signed-off-by: Dylan Reid <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
---
 arch/arm/boot/dts/tegra124-venice2.dts | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/arch/arm/boot/dts/tegra124-venice2.dts b/arch/arm/boot/dts/tegra124-venice2.dts
index f0bb842..f2a69cf 100644
--- a/arch/arm/boot/dts/tegra124-venice2.dts
+++ b/arch/arm/boot/dts/tegra124-venice2.dts
@@ -940,6 +940,10 @@
 		nvidia,sys-clock-req-active-high;
 	};
 
+	hda@0,70030000 {
+		status = "okay";
+	};
+
 	sdhci@0,700b0400 {
 		cd-gpios = <&gpio TEGRA_GPIO(V, 2) GPIO_ACTIVE_HIGH>;
 		power-gpios = <&gpio TEGRA_GPIO(R, 0) GPIO_ACTIVE_HIGH>;
-- 
1.8.1.3.605.g02339dd

Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support

[Stephen Warren]

On 05/19/2014 08:35 PM, Dylan Reid wrote:
> Add a device node for the HDA controller found on Tegra124.

> diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi

> +	hda@0,70030000 {
> +		compatible = "nvidia,tegra124-hda", "nvidia,tegra30-hda";
> +		reg = <0x0 0x70030000 0x0 0x10000>;
> +		interrupts = <GIC_SPI 81 IRQ_TYPE_LEVEL_HIGH>;
> +		clocks = <&tegra_car TEGRA124_CLK_HDA>,
> +		         <&tegra_car TEGRA124_CLK_HDA2HDMI>,
> +			 <&tegra_car TEGRA124_CLK_HDA2CODEC_2X>;
> +		clock-names = "hda", "hda2hdmi", "hdacodec_2x";

I think that should be "hda2codec_2x". Same for the reset-names property
below. This fix also needs to be propagated into the DT binding
documentation.

Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support

[Stephen Warren]

On 05/20/2014 12:24 PM, Stephen Warren wrote:
> On 05/19/2014 08:35 PM, Dylan Reid wrote:
>> Add a device node for the HDA controller found on Tegra124.
> 
>> diff --git a/arch/arm/boot/dts/tegra124.dtsi b/arch/arm/boot/dts/tegra124.dtsi
> 
>> +	hda@0,70030000 {
>> +		compatible = "nvidia,tegra124-hda", "nvidia,tegra30-hda";
>> +		reg = <0x0 0x70030000 0x0 0x10000>;
>> +		interrupts = <GIC_SPI 81 IRQ_TYPE_LEVEL_HIGH>;
>> +		clocks = <&tegra_car TEGRA124_CLK_HDA>,
>> +		         <&tegra_car TEGRA124_CLK_HDA2HDMI>,
>> +			 <&tegra_car TEGRA124_CLK_HDA2CODEC_2X>;
>> +		clock-names = "hda", "hda2hdmi", "hdacodec_2x";
> 
> I think that should be "hda2codec_2x". Same for the reset-names property
> below. This fix also needs to be propagated into the DT binding
> documentation.

Even after fixing that, I can't play audio on Venice2:

root@localhost:~# speaker-test -Dhw:0,3 -c 2

speaker-test 1.0.27.1

Playback device is hw:0,3
Stream parameters are 48000Hz, S16_LE, 2 channels
Using 16 octaves of pink noise
Rate set to 48000Hz (requested 48000Hz)
Buffer size range from 64 to 16384
Period size range from 32 to 8192
Using max buffer size 16384
Periods = 4
was set period_size = 4096
was set buffer_size = 16384
 0 - Front Left
Write error: -32,Broken pipe
Write error: -32,Broken pipe
Write error: -32,Broken pipe
^C 1 - Front Right
Time per period = 1.799791

Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support

[Stephen Warren]

On 05/19/2014 08:35 PM, Dylan Reid wrote:
> Add a device node for the HDA controller found on Tegra124.

I've applied the series to Tegra's for-3.17/dt branch.