* [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
@ 2014-05-20 2:35 Dylan Reid
[not found] ` <1400553346-23238-1-git-send-email-dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-20 2:35 UTC (permalink / raw)
To: linux-tegra-u79uwXL29TY76Z2rM5mHXA
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA, swarren-3lzwWm7+Weoh9ZMKESR00Q,
thierry.reding-Re5JQEeQqe8AvxtiuMwx3w, 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
^ permalink raw reply related [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <1400553346-23238-1-git-send-email-dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
@ 2014-05-20 18:24 ` Stephen Warren
[not found] ` <537B9DE3.5020807-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-06-16 19:10 ` Stephen Warren
1 sibling, 1 reply; 23+ messages in thread
From: Stephen Warren @ 2014-05-20 18:24 UTC (permalink / raw)
To: Dylan Reid, linux-tegra-u79uwXL29TY76Z2rM5mHXA
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
thierry.reding-Re5JQEeQqe8AvxtiuMwx3w
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.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <537B9DE3.5020807-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
@ 2014-05-20 18:31 ` Stephen Warren
0 siblings, 0 replies; 23+ messages in thread
From: Stephen Warren @ 2014-05-20 18:31 UTC (permalink / raw)
To: Dylan Reid
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA,
devicetree-u79uwXL29TY76Z2rM5mHXA,
thierry.reding-Re5JQEeQqe8AvxtiuMwx3w
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
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
@ 2014-05-20 20:37 Dylan Reid
[not found] ` <CAEUnVG7+=1ko8NL1Z3ovSpGveUADUidy4sBhU_dTUe1zbe-bMA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-20 20:37 UTC (permalink / raw)
To: Stephen Warren
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA, Thierry Reding
On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> 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
>
Thanks for checking Stephen. I'll fix the reset name.
This is the state of the tree I used when I tested on Venice2 just now:
* ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
* 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
* 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
* 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
* 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
* 6b6ba7b - (next-20140520, linux-next/master) Add linux-next specific
files for 20140520 (11 hours ago)
My speaker-test output is similar to yours, minus the errors, and I
hear noise from the attached monitor.
I tried with the reset name fixed to hda2codec_2x as well and it still
works for me.
Were you testing on one of your public trees? I couldn't get your
github tree to boot.
Dylan
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG7+=1ko8NL1Z3ovSpGveUADUidy4sBhU_dTUe1zbe-bMA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-05-20 21:55 ` Stephen Warren
[not found] ` <537BCF39.6050909-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Stephen Warren @ 2014-05-20 21:55 UTC (permalink / raw)
To: Dylan Reid, Peter De Schrijver
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA, Thierry Reding
On 05/20/2014 02:37 PM, Dylan Reid wrote:
> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> 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.
...
>> 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
>>
>
> Thanks for checking Stephen. I'll fix the reset name.
>
> This is the state of the tree I used when I tested on Venice2 just now:
> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
Ah, I was missing that clock patch. Hopefully Peter will pick it up for
the Tegra clock tree soon.
Now I have the same results as Thierry; speaker-test looks like it
should be working, yet I don't hear any audio from the monitor. I know
the monitor works, since I've used it extensively for testing GeForce
GPU HDMI audio.
> * 6b6ba7b - (next-20140520, linux-next/master) Add linux-next specific
> files for 20140520 (11 hours ago)
>
> My speaker-test output is similar to yours, minus the errors, and I
> hear noise from the attached monitor.
>
> I tried with the reset name fixed to hda2codec_2x as well and it still
> works for me.
>
> Were you testing on one of your public trees? I couldn't get your
> github tree to boot.
I am using:
git://github.com/swarren/linux-tegra.git tegra_dev
You might also need the latest version of my github U-Boot to use my
github kernel, since I fiddle with the pinmux initialization a bit:
git://github.com/swarren/u-boot.git tegra_dev
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <537BCF39.6050909-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
@ 2014-05-20 23:28 ` Dylan Reid
[not found] ` <CAEUnVG7CRWx5Y3RsgPzrHgYRGAx=bS4WY=w1xBUw0Ukt0DTdUQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-20 23:28 UTC (permalink / raw)
To: Stephen Warren
Cc: Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA, Thierry Reding
On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> On 05/20/2014 02:37 PM, Dylan Reid wrote:
>> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>> 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.
> ...
>>> 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
>>>
>>
>> Thanks for checking Stephen. I'll fix the reset name.
>>
>> This is the state of the tree I used when I tested on Venice2 just now:
>> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
>> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
>> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
>> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
>> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
>
> Ah, I was missing that clock patch. Hopefully Peter will pick it up for
> the Tegra clock tree soon.
>
> Now I have the same results as Thierry; speaker-test looks like it
> should be working, yet I don't hear any audio from the monitor. I know
> the monitor works, since I've used it extensively for testing GeForce
> GPU HDMI audio.
Probably a silly question: Is it un-muted?
amixer -c0 cset name='IEC958 Playback Switch' on
>
>> * 6b6ba7b - (next-20140520, linux-next/master) Add linux-next specific
>> files for 20140520 (11 hours ago)
>>
>> My speaker-test output is similar to yours, minus the errors, and I
>> hear noise from the attached monitor.
>>
>> I tried with the reset name fixed to hda2codec_2x as well and it still
>> works for me.
>>
>> Were you testing on one of your public trees? I couldn't get your
>> github tree to boot.
>
> I am using:
>
> git://github.com/swarren/linux-tegra.git tegra_dev
>
> You might also need the latest version of my github U-Boot to use my
> github kernel, since I fiddle with the pinmux initialization a bit:
>
> git://github.com/swarren/u-boot.git tegra_dev
That might be it. I'll give it a try.
-Dylan
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG7CRWx5Y3RsgPzrHgYRGAx=bS4WY=w1xBUw0Ukt0DTdUQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-05-20 23:42 ` Stephen Warren
2014-05-21 20:55 ` Thierry Reding
1 sibling, 0 replies; 23+ messages in thread
From: Stephen Warren @ 2014-05-20 23:42 UTC (permalink / raw)
To: Dylan Reid
Cc: Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA, Thierry Reding
On 05/20/2014 05:28 PM, Dylan Reid wrote:
> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> On 05/20/2014 02:37 PM, Dylan Reid wrote:
>>> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>> 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.
>> ...
>>>> 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
>>>>
>>>
>>> Thanks for checking Stephen. I'll fix the reset name.
>>>
>>> This is the state of the tree I used when I tested on Venice2 just now:
>>> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
>>> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
>>> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
>>> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
>>> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
>>
>> Ah, I was missing that clock patch. Hopefully Peter will pick it up for
>> the Tegra clock tree soon.
>>
>> Now I have the same results as Thierry; speaker-test looks like it
>> should be working, yet I don't hear any audio from the monitor. I know
>> the monitor works, since I've used it extensively for testing GeForce
>> GPU HDMI audio.
>
> Probably a silly question: Is it un-muted?
> amixer -c0 cset name='IEC958 Playback Switch' on
Yes (well I used alsamixer interactively, but it's the same thing, I
believe)
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG7CRWx5Y3RsgPzrHgYRGAx=bS4WY=w1xBUw0Ukt0DTdUQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-20 23:42 ` Stephen Warren
@ 2014-05-21 20:55 ` Thierry Reding
2014-05-21 21:10 ` Dylan Reid
1 sibling, 1 reply; 23+ messages in thread
From: Thierry Reding @ 2014-05-21 20:55 UTC (permalink / raw)
To: Dylan Reid
Cc: Stephen Warren, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 2754 bytes --]
On Tue, May 20, 2014 at 04:28:45PM -0700, Dylan Reid wrote:
> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> > On 05/20/2014 02:37 PM, Dylan Reid wrote:
> >> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+WeqIh2JisoIxyg@public.gmane.orgg> wrote:
> >>> 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.
> > ...
> >>> 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
> >>>
> >>
> >> Thanks for checking Stephen. I'll fix the reset name.
> >>
> >> This is the state of the tree I used when I tested on Venice2 just now:
> >> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
> >> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
> >> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
> >> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
> >> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
> >
> > Ah, I was missing that clock patch. Hopefully Peter will pick it up for
> > the Tegra clock tree soon.
> >
> > Now I have the same results as Thierry; speaker-test looks like it
> > should be working, yet I don't hear any audio from the monitor. I know
> > the monitor works, since I've used it extensively for testing GeForce
> > GPU HDMI audio.
>
> Probably a silly question: Is it un-muted?
> amixer -c0 cset name='IEC958 Playback Switch' on
Dylan, can you provide more detail about the setup that you're running?
There have been similar issues in the past where a driver would work if
it was run on an identical upstream kernel but with a different version
of the bootloader (or a different bootloader altogether). Can you give
further details about your setup? Also perhaps providing a dmesg log or
the clock tree from /sys/kernel/debug/clk/clk_summary could help
determine the diff between a working and a non-working setup.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
2014-05-21 20:55 ` Thierry Reding
@ 2014-05-21 21:10 ` Dylan Reid
[not found] ` <CAEUnVG7tDjw_6wHc=FN0KsOKsSS1u8H8SzYdxt9VCwUsARRTeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-21 21:10 UTC (permalink / raw)
To: Thierry Reding
Cc: Stephen Warren, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 3238 bytes --]
On Wed, May 21, 2014 at 1:55 PM, Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> On Tue, May 20, 2014 at 04:28:45PM -0700, Dylan Reid wrote:
>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> > On 05/20/2014 02:37 PM, Dylan Reid wrote:
>> >> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> >>> 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.
>> > ...
>> >>> 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
>> >>>
>> >>
>> >> Thanks for checking Stephen. I'll fix the reset name.
>> >>
>> >> This is the state of the tree I used when I tested on Venice2 just now:
>> >> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
>> >> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
>> >> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
>> >> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
>> >> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
>> >
>> > Ah, I was missing that clock patch. Hopefully Peter will pick it up for
>> > the Tegra clock tree soon.
>> >
>> > Now I have the same results as Thierry; speaker-test looks like it
>> > should be working, yet I don't hear any audio from the monitor. I know
>> > the monitor works, since I've used it extensively for testing GeForce
>> > GPU HDMI audio.
>>
>> Probably a silly question: Is it un-muted?
>> amixer -c0 cset name='IEC958 Playback Switch' on
>
> Dylan, can you provide more detail about the setup that you're running?
> There have been similar issues in the past where a driver would work if
> it was run on an identical upstream kernel but with a different version
> of the bootloader (or a different bootloader altogether). Can you give
> further details about your setup? Also perhaps providing a dmesg log or
> the clock tree from /sys/kernel/debug/clk/clk_summary could help
> determine the diff between a working and a non-working setup.
Sure. I'm running this u-boot:
https://chromium.googlesource.com/chromiumos/third_party/u-boot
on commit ddde3e9 of the chromeos-v2013.06 branch.
I haven't had a chance to flash swarren's u-boot yet it's been one of
those days...
I attached dmesg and clk_summary.
I don't have many modifications to the ubuntu-core rootfs.
Thanks for looking at this,
Dylan
>
> Thierry
[-- Attachment #2: venice2_clk_summary --]
[-- Type: application/octet-stream, Size: 17866 bytes --]
clock enable_cnt prepare_cnt rate accuracy
---------------------------------------------------------------------------------
clock 0 0 32768 0
vimclk_sync 0 0 24000000 0
i2s4_sync 0 0 24000000 0
i2s3_sync 0 0 24000000 0
i2s2_sync 0 0 24000000 0
i2s1_sync 0 0 24000000 0
i2s0_sync 0 0 24000000 0
spdif_in_sync 0 0 24000000 0
spdif_mux 0 0 24000000 0
spdif 0 0 24000000 0
spdif_doubler 0 0 48000000 0
spdif_div 0 0 48000000 0
spdif_2x 0 0 48000000 0
audio4_mux 0 0 24000000 0
audio4 0 0 24000000 0
audio4_doubler 0 0 48000000 0
audio4_div 0 0 48000000 0
audio4_2x 0 0 48000000 0
audio3_mux 0 0 24000000 0
audio3 0 0 24000000 0
audio3_doubler 0 0 48000000 0
audio3_div 0 0 48000000 0
audio3_2x 0 0 48000000 0
audio2_mux 0 0 24000000 0
audio2 0 0 24000000 0
audio2_doubler 0 0 48000000 0
audio2_div 0 0 48000000 0
audio2_2x 0 0 48000000 0
audio1_mux 0 0 24000000 0
audio1 0 0 24000000 0
audio1_doubler 0 0 48000000 0
audio1_div 0 0 48000000 0
audio1_2x 0 0 48000000 0
audio0_mux 0 0 24000000 0
audio0 0 0 24000000 0
audio0_doubler 0 0 48000000 0
audio0_div 0 0 48000000 0
audio0_2x 0 0 48000000 0
clk_32k 1 1 32768 0
blink_override 0 0 32768 0
blink 0 0 32768 0
kbc 0 0 32768 0
rtc 1 1 32768 0
tsensor 0 0 32768 0
clk_m 10 10 12000000 0
cclk_lp 0 0 12000000 0
clk_out_3_mux 0 0 12000000 0
clk_out_3 0 0 12000000 0
clk_out_2_mux 0 0 12000000 0
clk_out_2 0 0 12000000 0
dpaux 2 2 12000000 0
pcie 0 0 12000000 0
vim2_clk 0 0 12000000 0
ispb 0 0 12000000 0
sata_cold 0 0 12000000 0
dtv 0 0 12000000 0
dp2 0 0 12000000 0
dds 0 0 12000000 0
csus 0 0 12000000 0
afi 0 0 12000000 0
usb3 1 1 12000000 0
usb2 1 1 12000000 0
usbd 1 1 12000000 0
mipi-cal 0 0 12000000 0
bsev 0 0 12000000 0
bsea 0 0 12000000 0
hda2hdmi 1 1 12000000 0
apbif 0 0 12000000 0
kfuse 0 0 12000000 0
fuse_burn 0 0 12000000 0
fuse 0 0 12000000 0
apbdma 6 6 12000000 0
vcp 0 0 12000000 0
timer 1 1 12000000 0
xusb_dev_src 0 0 12000000 0
xusb_dev 0 0 12000000 0
xusb_ss_src 0 0 12000000 0
xusb_ss 0 0 12000000 0
xusb_fs_src 0 0 12000000 0
xusb_falcon_src 0 0 12000000 0
xusb_host_src 0 0 12000000 0
xusb_host 0 0 12000000 0
sor0_lvds 0 0 12000000 0
entropy 0 0 12000000 0
extern3 0 0 12000000 0
extern2 0 0 12000000 0
sdmmc2 0 0 12000000 0
sdmmc1 0 0 12000000 0
amx1 0 0 12000000 0
adx1 0 0 12000000 0
sata 0 0 12000000 0
sata_oob 0 0 12000000 0
sbc6 0 0 12000000 0
sbc5 0 0 12000000 0
sbc3 0 0 12000000 0
sbc2 0 0 12000000 0
sbc1 0 0 12000000 0
i2cslow 0 0 12000000 0
actmon 0 0 12000000 0
mipi 0 0 12000000 0
nor 0 0 12000000 0
owr 0 0 12000000 0
trace 0 0 12000000 0
la 0 0 12000000 0
vfir 0 0 12000000 0
amx 0 0 12000000 0
adx 0 0 12000000 0
pwm 1 1 12000000 0
spdif_out 0 0 12000000 0
dam2 0 0 12000000 0
dam1 0 0 12000000 0
dam0 0 0 12000000 0
d_audio 0 0 12000000 0
clk_m_div4 0 0 3000000 0
clk_m_div2 0 0 6000000 0
pll_ref 5 5 12000000 0
pll_x 0 0 696000000 0
pll_x_out0 0 0 348000000 0
cclk_g 0 0 696000000 0
gpu 0 0 12000000 0
pll_p 11 11 408000000 0
hda2codec_2x 1 1 48000000 0
hda 1 1 102000000 0
dfll_ref 1 1 51000000 0
dfll_soc 1 1 51000000 0
host1x 2 2 136000000 0
vde 0 0 408000000 0
uartd 0 0 408000000 0
uartc 0 0 408000000 0
uartb 0 0 408000000 0
uarta 1 1 408000000 0
csite 0 0 204000000 0
soc_therm 0 0 408000000 0
sdmmc4 1 1 48000000 0
sdmmc3 1 1 19902440 0
sbc4 1 1 12000000 0
spdif_in 0 0 408000000 0
mselect 0 0 102000000 0
se 0 0 102000000 0
i2c5 0 0 81600000 0
i2c4 0 0 20400000 0
i2c3 0 0 20400000 0
i2c2 0 0 20400000 0
i2c1 0 0 20400000 0
pll_p_out5_div 0 0 408000000 0
pll_p_out5 0 0 408000000 0
pll_p_out4_div 0 0 204000000 0
pll_p_out4 0 0 204000000 0
pll_p_out3_div 0 0 102000000 0
pll_p_out3 0 0 102000000 0
csi 0 0 102000000 0
clk72mhz 0 0 102000000 0
hdmi_audio 0 0 102000000 0
pll_p_out2_div 1 1 102000000 0
pll_p_out2 1 1 102000000 0
sclk 1 1 102000000 0
hclk_div 0 0 51000000 0
hclk 0 0 51000000 0
pclk_div 0 0 51000000 0
pclk 0 0 51000000 0
pll_p_out1_div 1 1 9600000 0
pll_p_out1 1 1 9600000 0
pll_a 2 2 282240000 0
pll_a_out0_div 1 1 11289600 0
pll_a_out0 2 2 11289600 0
i2s4 0 0 11289600 0
i2s3 0 0 11289600 0
i2s2 0 0 11289600 0
i2s1 0 0 11289600 0
i2s0 0 0 11289600 0
extern1 2 2 11289600 0
clk_out_1_mux 2 2 11289600 0
clk_out_1 1 1 11289600 0
emc_mux 0 0 408000000 0
emc 0 0 408000000 0
pll_d2 1 1 146400000 0
pll_d2_out0 2 2 146400000 0
disp2 1 1 146400000 0
hdmi 1 1 146400000 0
pll_dp 4 4 268000000 0
sor0 2 2 268000000 0
pll_c4 0 0 18750000 0
pll_e 0 0 100000000 0
cml1 0 0 100000000 0
cml0 0 0 100000000 0
pll_re_vco 0 0 300000000 0
pll_re_out 0 0 300000000 0
pll_d 1 1 572000000 0
pll_d_out0 2 2 286000000 0
disp1 1 1 286000000 0
dsib_mux 0 0 286000000 0
dsib 0 0 286000000 0
dsia_mux 0 0 286000000 0
dsia 0 0 286000000 0
pll_u 3 3 480000000 0
pll_u_12M 0 0 12000000 0
pll_u_48M 0 0 48000000 0
pll_u_60M 0 0 60000000 0
xusb_hs_src 0 0 60000000 0
pll_u_480M 0 0 480000000 0
pll_m 0 0 408000000 0
isp 0 0 408000000 0
vi_sensor2 0 0 408000000 0
dsiblp 0 0 408000000 0
dsialp 0 0 408000000 0
cile 0 0 408000000 0
cilcd 0 0 408000000 0
cilab 0 0 408000000 0
vi_sensor 0 0 408000000 0
vic03 0 0 408000000 0
vi 0 0 408000000 0
pll_m_ud 0 0 408000000 0
pll_m_out1_div 0 0 408000000 0
pll_m_out1 0 0 408000000 0
pll_c3 0 0 300000000 0
msenc 0 0 300000000 0
tsec 0 0 300000000 0
pll_c2 0 0 300000000 0
pll_c 0 0 768000000 0
pll_c_out1_div 0 0 96000000 0
pll_c_out1 0 0 96000000 0
[-- Attachment #3: venice2_dmesg --]
[-- Type: application/octet-stream, Size: 24647 bytes --]
[ 0.000000] Booting Linux on physical CPU 0x0
[ 0.000000] Initializing cgroup subsys cpu
[ 0.000000] Initializing cgroup subsys cpuacct
[ 0.000000] Linux version 3.15.0-rc5-next-20140520-00005-gae57dbf (dgreid@hojo20.mtv.corp.google.com) (gcc version 4.8.x-google 20131105 (prerelease) (4.8.2_cos_gg_3eaa0b2_4.8.2-r69) ) #64 SMP PREEMPT Wed May 21 10:49:35 PDT 2014
[ 0.000000] CPU: ARMv7 Processor [413fc0f3] revision 3 (ARMv7), cr=10c5387d
[ 0.000000] CPU: PIPT / VIPT nonaliasing data cache, PIPT instruction cache
[ 0.000000] Machine model: NVIDIA Tegra124 Venice2
[ 0.000000] bootconsole [earlycon0] enabled
[ 0.000000] cma: CMA: reserved 64 MiB at ab800000
[ 0.000000] Memory policy: Data cache writealloc
[ 0.000000] On node 0 totalpages: 524032
[ 0.000000] free_area_init_node: node 0, pgdat c08d4280, node_mem_map ea7d7000
[ 0.000000] Normal zone: 1520 pages used for memmap
[ 0.000000] Normal zone: 0 pages reserved
[ 0.000000] Normal zone: 194560 pages, LIFO batch:31
[ 0.000000] HighMem zone: 2574 pages used for memmap
[ 0.000000] HighMem zone: 329472 pages, LIFO batch:31
[ 0.000000] Tegra: unknown chip id 64
[ 0.000000] Tegra Revision: A01 SKU: 0 CPU Process: 0 Core Process: 0
[ 0.000000] PERCPU: Embedded 8 pages/cpu @ea795000 s8640 r8192 d15936 u32768
[ 0.000000] pcpu-alloc: s8640 r8192 d15936 u32768 alloc=8*4096
[ 0.000000] pcpu-alloc: [0] 0 [0] 1 [0] 2 [0] 3
[ 0.000000] Built 1 zonelists in Zone order, mobility grouping on. Total pages: 522512
[ 0.000000] Kernel command line: cros_legacy console=ttyS0,115200n8 noinitrd earlyprintk lp0_vec=0x2000@0xffa3e440 root=/dev/mmcblk0p1 rootwait rw
[ 0.000000] PID hash table entries: 4096 (order: 2, 16384 bytes)
[ 0.000000] Dentry cache hash table entries: 131072 (order: 7, 524288 bytes)
[ 0.000000] Inode-cache hash table entries: 65536 (order: 6, 262144 bytes)
[ 0.000000] Memory: 2003580K/2096128K available (6390K kernel code, 334K rwdata, 2016K rodata, 268K init, 346K bss, 92548K reserved, 1317876K highmem)
[ 0.000000] Virtual kernel memory layout:
[ 0.000000] vector : 0xffff0000 - 0xffff1000 ( 4 kB)
[ 0.000000] fixmap : 0xffc00000 - 0xffe00000 (2048 kB)
[ 0.000000] vmalloc : 0xf0000000 - 0xff000000 ( 240 MB)
[ 0.000000] lowmem : 0xc0000000 - 0xef800000 ( 760 MB)
[ 0.000000] pkmap : 0xbfe00000 - 0xc0000000 ( 2 MB)
[ 0.000000] modules : 0xbf000000 - 0xbfe00000 ( 14 MB)
[ 0.000000] .text : 0xc0008000 - 0xc083dbd8 (8407 kB)
[ 0.000000] .init : 0xc083e000 - 0xc08811c0 ( 269 kB)
[ 0.000000] .data : 0xc0882000 - 0xc08d5b80 ( 335 kB)
[ 0.000000] .bss : 0xc08d5b8c - 0xc092c694 ( 347 kB)
[ 0.000000] Preemptible hierarchical RCU implementation.
[ 0.000000] NR_IRQS:16 nr_irqs:16 16
[ 0.000011] sched_clock: 32 bits at 1000kHz, resolution 1000ns, wraps every 2147483648000ns
[ 0.007260] Architected cp15 timer(s) running at 12.00MHz (virt).
[ 0.012706] sched_clock: 56 bits at 12MHz, resolution 83ns, wraps every 2863311536128ns
[ 0.020750] Switching to timer-based delay loop
[ 0.025874] Console: colour dummy device 80x30
[ 0.029874] Calibrating delay loop (skipped), value calculated using timer frequency.. 24.00 BogoMIPS (lpj=120000)
[ 0.040252] pid_max: default: 32768 minimum: 301
[ 0.045380] Mount-cache hash table entries: 2048 (order: 1, 8192 bytes)
[ 0.051624] Mountpoint-cache hash table entries: 2048 (order: 1, 8192 bytes)
[ 0.060326] Initializing cgroup subsys freezer
[ 0.063259] Initializing cgroup subsys debug
[ 0.067696] CPU: Testing write buffer coherency: ok
[ 0.072813] /cpus/cpu@0 missing clock-frequency property
[ 0.077896] /cpus/cpu@1 missing clock-frequency property
[ 0.083250] /cpus/cpu@2 missing clock-frequency property
[ 0.088640] /cpus/cpu@3 missing clock-frequency property
[ 0.093989] CPU0: thread -1, cpu 0, socket 0, mpidr 80000000
[ 0.099779] Setting up static identity map for 0x80612540 - 0x806125d8
[ 0.160983] CPU1: Booted secondary processor
[ 0.161014] CPU1: thread -1, cpu 1, socket 0, mpidr 80000001
[ 0.180965] CPU2: Booted secondary processor
[ 0.180989] CPU2: thread -1, cpu 2, socket 0, mpidr 80000002
[ 0.201004] CPU3: Booted secondary processor
[ 0.201028] CPU3: thread -1, cpu 3, socket 0, mpidr 80000003
[ 0.201208] Brought up 4 CPUs
[ 0.232518] SMP: Total of 4 processors activated.
[ 0.237295] CPU: All CPU(s) started in SVC mode.
[ 0.242978] devtmpfs: initialized
[ 0.258848] VFP support v0.3: implementor 41 architecture 4 part 30 variant f rev 0
[ 0.265603] pinctrl core: initialized pinctrl subsystem
[ 0.270775] regulator-dummy: no parameters
[ 0.298716] NET: Registered protocol family 16
[ 0.302664] DMA: preallocated 256 KiB pool for atomic coherent allocations
[ 0.309803] cpuidle: using governor ladder
[ 0.312468] cpuidle: using governor menu
[ 0.336129] No ATAGs?
[ 0.336355] hw-breakpoint: found 5 (+1 reserved) breakpoint and 4 watchpoint registers.
[ 0.344659] hw-breakpoint: maximum watchpoint size is 8 bytes.
[ 0.398006] +VDD_MUX: 12000 mV
[ 0.399826] +5V_SYS: 5000 mV
[ 0.402429] +5V_SYS: supplied by +VDD_MUX
[ 0.406926] +3.3V_SYS: 3300 mV
[ 0.409714] +3.3V_SYS: supplied by +VDD_MUX
[ 0.414111] platform regulator.10: Driver reg-fixed-voltage requests probe deferral
[ 0.421967] +3.3V_AVDD_HDMI_AP_GATED: 3300 mV
[ 0.426172] reg-fixed-voltage regulator.11: Failed to find supply vin
[ 0.432810] reg-fixed-voltage regulator.11: Failed to register regulator: -517
[ 0.439903] platform regulator.11: Driver reg-fixed-voltage requests probe deferral
[ 0.447971] +VDD_LED: Failed to request enable GPIO122: -517
[ 0.453473] reg-fixed-voltage regulator.12: Failed to register regulator: -517
[ 0.460633] platform regulator.12: Driver reg-fixed-voltage requests probe deferral
[ 0.468663] +5V_VDD_TS_SW: Failed to request enable GPIO81: -517
[ 0.474550] reg-fixed-voltage regulator.13: Failed to register regulator: -517
[ 0.481676] platform regulator.13: Driver reg-fixed-voltage requests probe deferral
[ 0.489738] +5V_USB_HS: Failed to request enable GPIO108: -517
[ 0.495418] reg-fixed-voltage regulator.14: Failed to register regulator: -517
[ 0.502578] platform regulator.14: Driver reg-fixed-voltage requests probe deferral
[ 0.510608] +5V_USB_SS: Failed to request enable GPIO109: -517
[ 0.516320] reg-fixed-voltage regulator.15: Failed to register regulator: -517
[ 0.523449] platform regulator.15: Driver reg-fixed-voltage requests probe deferral
[ 0.531251] platform regulator.16: Driver reg-fixed-voltage requests probe deferral
[ 0.538946] platform regulator.17: Driver reg-fixed-voltage requests probe deferral
[ 0.546937] +1.05V_RUN_AVDD_HDMI_PLL: Failed to request enable GPIO63: -517
[ 0.553744] reg-fixed-voltage regulator.18: Failed to register regulator: -517
[ 0.560904] platform regulator.18: Driver reg-fixed-voltage requests probe deferral
[ 0.568936] +5V_HDMI_CON: Failed to request enable GPIO86: -517
[ 0.574736] reg-fixed-voltage regulator.19: Failed to register regulator: -517
[ 0.581862] platform regulator.19: Driver reg-fixed-voltage requests probe deferral
[ 0.591428] vgaarb: loaded
[ 0.593814] SCSI subsystem initialized
[ 0.596866] usbcore: registered new interface driver usbfs
[ 0.601880] usbcore: registered new interface driver hub
[ 0.607257] usbcore: registered new device driver usb
[ 0.616234] Linux video capture interface: v2.00
[ 0.620329] Advanced Linux Sound Architecture Driver Initialized.
[ 0.626660] Bluetooth: Core ver 2.19
[ 0.628893] NET: Registered protocol family 31
[ 0.633327] Bluetooth: HCI device and connection manager initialized
[ 0.639782] Bluetooth: HCI socket layer initialized
[ 0.644695] Bluetooth: L2CAP socket layer initialized
[ 0.649866] Bluetooth: SCO socket layer initialized
[ 0.655535] cfg80211: Calling CRDA to update world regulatory domain
[ 0.661966] Switched to clocksource arch_sys_counter
[ 0.687973] NET: Registered protocol family 2
[ 0.691584] TCP established hash table entries: 8192 (order: 3, 32768 bytes)
[ 0.697747] TCP bind hash table entries: 8192 (order: 4, 65536 bytes)
[ 0.704303] TCP: Hash tables configured (established 8192 bind 8192)
[ 0.710975] TCP: reno registered
[ 0.713854] UDP hash table entries: 512 (order: 2, 16384 bytes)
[ 0.719832] UDP-Lite hash table entries: 512 (order: 2, 16384 bytes)
[ 0.726627] NET: Registered protocol family 1
[ 0.731366] RPC: Registered named UNIX socket transport module.
[ 0.736637] RPC: Registered udp transport module.
[ 0.741379] RPC: Registered tcp transport module.
[ 0.746182] RPC: Registered tcp NFSv4.1 backchannel transport module.
[ 0.752679] PCI: CLS 0 bytes, default 64
[ 0.757120] futex hash table entries: 1024 (order: 4, 65536 bytes)
[ 0.762867] bounce: pool size: 64 pages
[ 0.767035] squashfs: version 4.0 (2009/01/31) Phillip Lougher
[ 0.772187] fuse init (API version 7.23)
[ 0.775737] msgmni has been set to 1467
[ 0.781027] io scheduler noop registered (default)
[ 0.791658] platform backlight.1: Driver pwm-backlight requests probe deferral
[ 0.806161] tegra-apbdma 60020000.dma: Tegra20 APB DMA driver register 32 channels
[ 0.813039] Serial: 8250/16550 driver, 4 ports, IRQ sharing disabled
[ 0.821866] console [ttyS0] disabled
[ 0.823681] 70006000.serial: ttyS0 at MMIO 0x70006000 (irq = 68, base_baud = 25500000) is a Tegra
[ 0.832587] console [ttyS0] enabled
[ 0.838876] bootconsole [earlycon0] disabled
[ 0.848846] [drm] Initialized drm 1.1.0 20060810
[ 0.855588] platform 54540000.sor: Driver tegra-sor requests probe deferral
[ 0.862929] tegra-hdmi 54280000.hdmi: failed to get HDMI regulator
[ 0.869121] platform 54280000.hdmi: Driver tegra-hdmi requests probe deferral
[ 0.876942] platform 545c0000.dpaux: Driver tegra-dpaux requests probe deferral
[ 0.885245] panel.5 supply power not found, using dummy regulator
[ 0.891494] platform panel.5: Driver panel-simple requests probe deferral
[ 0.909780] loop: module loaded
[ 0.914263] zram: Created 1 device(s) ...
[ 0.919709] as3722 4-0040: AS3722 with revision 0x1 found
[ 0.930755] +VDD_CPU_AP: 700 <--> 1400 mV at 1200 mV 3500 mA
[ 0.938086] +VDD_CORE: 700 <--> 1350 mV at 1100 mV 2500 mA
[ 0.944344] +1.35V_LP0(sd2): 1350 mV
[ 0.948127] +1.35V_LP0(sd2): supplied by +5V_SYS
[ 0.953350] +1.35V_LP0(sd3): 1350 mV
[ 0.957126] +1.35V_LP0(sd3): supplied by +5V_SYS
[ 0.962332] +1.05V_RUN: 1050 mV
[ 0.965675] +1.05V_RUN: supplied by +5V_SYS
[ 0.970431] +1.8V_VDDIO: 1800 mV
[ 0.973878] +1.8V_VDDIO: supplied by +5V_SYS
[ 0.979625] +VDD_GPU_AP: 650 <--> 1200 mV at 1000 mV 3500 mA
[ 0.986834] +1.05V_RUN_AVDD: 1050 mV at 300 mA
[ 0.991497] +1.05V_RUN_AVDD: supplied by +1.35V_LP0(sd2)
[ 0.997402] +1.8V_RUN_CAM: 1800 mV at 150 mA
[ 1.001899] as3722-regulator as3722-regulator: Failed to find supply vin-ldo1-6
[ 1.009415] as3722-regulator as3722-regulator: regulator 8 register failed -517
[ 1.018657] platform as3722-regulator: Driver as3722-regulator requests probe deferral
[ 1.031498] cros-ec-spi spi32766.0: Chrome EC (SPI)
[ 1.039866] pegasus: v0.9.3 (2013/04/25), Pegasus/Pegasus II USB Ethernet driver
[ 1.047421] usbcore: registered new interface driver pegasus
[ 1.053409] usbcore: registered new interface driver asix
[ 1.058930] usbcore: registered new interface driver ax88179_178a
[ 1.065176] usbcore: registered new interface driver cdc_ether
[ 1.071179] usbcore: registered new interface driver smsc75xx
[ 1.077129] usbcore: registered new interface driver smsc95xx
[ 1.083014] usbcore: registered new interface driver net1080
[ 1.088795] usbcore: registered new interface driver cdc_subset
[ 1.094852] usbcore: registered new interface driver zaurus
[ 1.100618] usbcore: registered new interface driver cdc_ncm
[ 1.106347] ehci_hcd: USB 2.0 'Enhanced' Host Controller (EHCI) Driver
[ 1.112894] ehci-pci: EHCI PCI platform driver
[ 1.117466] tegra-ehci: Tegra EHCI driver
[ 1.121765] platform 7d000000.usb: Driver tegra-ehci requests probe deferral
[ 1.128956] platform 7d004000.usb: Driver tegra-ehci requests probe deferral
[ 1.136137] platform 7d008000.usb: Driver tegra-ehci requests probe deferral
[ 1.143674] usbcore: registered new interface driver cdc_acm
[ 1.149324] cdc_acm: USB Abstract Control Model driver for USB modems and ISDN adapters
[ 1.157473] usbcore: registered new interface driver cdc_wdm
[ 1.163333] usbcore: registered new interface driver usb-storage
[ 1.169600] platform 7d000000.usb-phy: Driver tegra-phy requests probe deferral
[ 1.177053] platform 7d004000.usb-phy: Driver tegra-phy requests probe deferral
[ 1.184505] platform 7d008000.usb-phy: Driver tegra-phy requests probe deferral
[ 1.192733] mousedev: PS/2 mouse device common for all mice
[ 1.199194] input: cros-ec-spi as /devices/soc0/7000d400.spi/spi_master/spi32766/spi32766.0/cros-ec-keyb.1/input/input0
[ 1.213564] as3722-rtc as3722-rtc: rtc core: registered as3722-rtc as rtc0
[ 1.220434] as3722-rtc as3722-rtc: RTC interrupt 464
[ 1.228539] tegra_rtc 7000e000.rtc: rtc core: registered 7000e000.rtc as rtc1
[ 1.235719] tegra_rtc 7000e000.rtc: Tegra internal Real Time Clock
[ 1.242326] i2c /dev entries driver
[ 1.248056] usbcore: registered new interface driver uvcvideo
[ 1.253814] USB Video Class driver (1.1.1)
[ 1.257909] gspca_main: v2.14.0 registered
[ 1.264232] device-mapper: ioctl: 4.27.0-ioctl (2013-10-30) initialised: dm-devel@redhat.com
[ 1.273027] sdhci: Secure Digital Host Controller Interface driver
[ 1.279198] sdhci: Copyright(c) Pierre Ossman
[ 1.283602] sdhci-pltfm: SDHCI platform and OF driver helper
[ 1.289819] sdhci-tegra 700b0400.sdhci: Got CD GPIO #170.
[ 1.295251] sdhci-tegra 700b0400.sdhci: Got WP GPIO #132.
[ 1.300687] mmc0: Unknown controller version (3). You may experience problems.
[ 1.307966] mmc0: no vqmmc regulator found
[ 1.312079] mmc0: no vmmc regulator found
[ 1.316084] mmc0: Invalid maximum block size, assuming 512 bytes
[ 1.361981] mmc0: SDHCI controller on 700b0400.sdhci [700b0400.sdhci] using ADMA
[ 1.369519] mmc1: Unknown controller version (3). You may experience problems.
[ 1.376775] mmc1: no vqmmc regulator found
[ 1.380868] mmc1: no vmmc regulator found
[ 1.384886] mmc1: Invalid maximum block size, assuming 512 bytes
[ 1.431984] mmc1: SDHCI controller on 700b0600.sdhci [700b0600.sdhci] using ADMA
[ 1.440545] usbcore: registered new interface driver usbhid
[ 1.446337] usbhid: USB HID core driver
[ 1.458795] tegra-hda 70030000.hda: CORB reset timeout#1, CORBRP = 0
[ 1.557511] mmc1: BKOPS_EN bit is not set
[ 1.588453] mmc1: new HS200 MMC card at address 0001
[ 1.594230] mmcblk0: mmc1:0001 SEM16G 14.6 GiB
[ 1.599032] mmcblk0boot0: mmc1:0001 SEM16G partition 1 4.00 MiB
[ 1.605265] mmcblk0boot1: mmc1:0001 SEM16G partition 2 4.00 MiB
[ 1.611404] mmcblk0rpmb: mmc1:0001 SEM16G partition 3 4.00 MiB
[ 1.618856] mmcblk0: p1
[ 1.624884] mmcblk0boot1: unknown partition table
[ 1.631390] mmcblk0boot0: unknown partition table
[ 1.947952] tegra30-i2s 70301100.i2s: DMA channels sourced from device 70300000.ahub
[ 1.982504] max98090 0-0010: MAX98090 REVID=0x41
[ 1.989727] max98090 0-0010: No platform data
[ 1.995980] tegra-snd-max98090 sound.20: HiFi <-> 70301100.i2s mapping ok
[ 2.008439] TCP: cubic registered
[ 2.012107] NET: Registered protocol family 10
[ 2.018217] mip6: Mobile IPv6
[ 2.021192] sit: IPv6 over IPv4 tunneling driver
[ 2.028723] NET: Registered protocol family 17
[ 2.033255] NET: Registered protocol family 15
[ 2.037836] Bluetooth: RFCOMM socket layer initialized
[ 2.043006] Bluetooth: RFCOMM ver 1.11
[ 2.046771] Bluetooth: BNEP (Ethernet Emulation) ver 1.3
[ 2.052108] Bluetooth: BNEP socket layer initialized
[ 2.057073] Bluetooth: HIDP (Human Interface Emulation) ver 1.2
[ 2.063014] Bluetooth: HIDP socket layer initialized
[ 2.069455] Registering SWP/SWPB emulation handler
[ 2.078109] +3.3V_RUN: 3300 mV
[ 2.081430] +3.3V_RUN: supplied by +3.3V_SYS
[ 2.086561] +3.3V_AVDD_HDMI_AP_GATED: 3300 mV
[ 2.091128] +3.3V_AVDD_HDMI_AP_GATED: supplied by +3.3V_RUN
[ 2.097418] +VDD_LED: no parameters
[ 2.100968] +VDD_LED: supplied by +VDD_MUX
[ 2.105863] +5V_VDD_TS_SW: 5000 mV
[ 2.109470] +5V_VDD_TS_SW: supplied by +5V_SYS
[ 2.114769] +5V_USB_HS: 5000 mV
[ 2.118122] +5V_USB_HS: supplied by +5V_SYS
[ 2.122852] +5V_USB_SS: 5000 mV
[ 2.126149] +5V_USB_SS: supplied by +5V_SYS
[ 2.132950] +3.3V_PANEL: 3300 mV
[ 2.136446] +3.3V_PANEL: supplied by +3.3V_RUN
[ 2.145285] +3.3V_LP0: 3300 mV
[ 2.148594] +3.3V_LP0: supplied by +3.3V_SYS
[ 2.153594] +1.05V_RUN_AVDD_HDMI_PLL: 1050 mV
[ 2.158112] reg-fixed-voltage regulator.18: Failed to find supply vin
[ 2.164763] reg-fixed-voltage regulator.18: Failed to register regulator: -517
[ 2.172013] platform regulator.18: Driver reg-fixed-voltage requests probe deferral
[ 2.180475] +5V_HDMI_CON: 5000 mV
[ 2.184039] +5V_HDMI_CON: supplied by +5V_SYS
[ 2.189338] platform 54540000.sor: Driver tegra-sor requests probe deferral
[ 2.196681] tegra-hdmi 54280000.hdmi: failed to get PLL regulator
[ 2.202885] platform 54280000.hdmi: Driver tegra-hdmi requests probe deferral
[ 2.211332] panel.5 supply power not found, using dummy regulator
[ 2.220664] +VDD_CPU_AP: 700 <--> 1400 mV at 1200 mV 3500 mA
[ 2.228088] +VDD_CORE: 700 <--> 1350 mV at 1100 mV 2500 mA
[ 2.234310] +1.35V_LP0(sd2): 1350 mV
[ 2.238176] +1.35V_LP0(sd2): supplied by +5V_SYS
[ 2.243489] +1.35V_LP0(sd3): 1350 mV
[ 2.247342] +1.35V_LP0(sd3): supplied by +5V_SYS
[ 2.252694] +1.05V_RUN: 1050 mV
[ 2.256110] +1.05V_RUN: supplied by +5V_SYS
[ 2.260749] +1.8V_VDDIO: 1800 mV
[ 2.264245] +1.8V_VDDIO: supplied by +5V_SYS
[ 2.270227] +VDD_GPU_AP: 650 <--> 1200 mV at 1000 mV 3500 mA
[ 2.277250] +1.05V_RUN_AVDD: 1050 mV at 300 mA
[ 2.281966] +1.05V_RUN_AVDD: supplied by +1.35V_LP0(sd2)
[ 2.287706] +1.8V_RUN_CAM: 1800 mV at 150 mA
[ 2.292238] +1.8V_RUN_CAM: supplied by +3.3V_RUN
[ 2.298740] +1.2V_GEN_AVDD: 1200 mV at 150 mA
[ 2.303438] +1.2V_GEN_AVDD: supplied by +1.8V_VDDIO
[ 2.310785] +1.00V_LP0_VDD_RTC: 1000 mV at 150 mA
[ 2.317141] +3.3V_RUN_CAM: 2800 mV at 150 mA
[ 2.323444] +1.2V_RUN_CAM_FRONT: 1200 mV at 150 mA
[ 2.328529] +1.2V_RUN_CAM_FRONT: supplied by +1.8V_VDDIO
[ 2.335254] +VDDIO_SDMMC3: 1800 <--> 3300 mV at 150 mA
[ 2.340714] +VDDIO_SDMMC3: supplied by +3.3V_RUN
[ 2.346887] +1.05V_RUN_CAM_REAR: 1050 mV at 150 mA
[ 2.352027] +1.05V_RUN_CAM_REAR: supplied by +1.8V_VDDIO
[ 2.359990] +2.8V_RUN_TOUCH: 2800 mV at 150 mA
[ 2.364802] +2.8V_RUN_TOUCH: supplied by +5V_SYS
[ 2.371242] +2.8V_RUN_CAM_AF: 2800 mV at 150 mA
[ 2.376155] +2.8V_RUN_CAM_AF: supplied by +5V_SYS
[ 2.382334] +1.8V_RUN_VPP_FUSE: 1800 mV at 150 mA
[ 2.387362] +1.8V_RUN_VPP_FUSE: supplied by +3.3V_RUN
[ 2.393054] platform 7d000000.usb: Driver tegra-ehci requests probe deferral
[ 2.400497] platform 7d004000.usb: Driver tegra-ehci requests probe deferral
[ 2.407893] platform 7d008000.usb: Driver tegra-ehci requests probe deferral
[ 2.416726] +1.05V_RUN_AVDD_HDMI_PLL: 1050 mV
[ 2.421239] +1.05V_RUN_AVDD_HDMI_PLL: supplied by +1.05V_RUN
[ 2.486831] [drm] Supports vblank timestamp caching Rev 2 (21.10.2013).
[ 2.493491] [drm] No driver support for vblank timestamp query.
[ 2.640007] Console: switching to colour frame buffer device 320x106
[ 2.712031] host1x drm: fb0: frame buffer device
[ 2.716730] host1x drm: registered panic notifier
[ 2.752230] [drm] Initialized tegra 0.0.0 20120330 on minor 0
[ 2.761156] tegra-ehci 7d000000.usb: EHCI Host Controller
[ 2.766620] tegra-ehci 7d000000.usb: new USB bus registered, assigned bus number 1
[ 2.774768] tegra-ehci 7d000000.usb: irq 52, io mem 0x7d000000
[ 2.792028] tegra-ehci 7d000000.usb: USB 2.0 started, EHCI 1.10
[ 2.800122] hub 1-0:1.0: USB hub found
[ 2.804053] hub 1-0:1.0: 1 port detected
[ 2.809567] tegra-ehci 7d004000.usb: EHCI Host Controller
[ 2.815149] tegra-ehci 7d004000.usb: new USB bus registered, assigned bus number 2
[ 2.823340] tegra-ehci 7d004000.usb: irq 53, io mem 0x7d004000
[ 2.842226] tegra-ehci 7d004000.usb: USB 2.0 started, EHCI 1.10
[ 2.850050] hub 2-0:1.0: USB hub found
[ 2.853953] hub 2-0:1.0: 1 port detected
[ 2.859641] tegra-ehci 7d008000.usb: EHCI Host Controller
[ 2.865242] tegra-ehci 7d008000.usb: new USB bus registered, assigned bus number 3
[ 2.873471] tegra-ehci 7d008000.usb: irq 129, io mem 0x7d008000
[ 2.892229] tegra-ehci 7d008000.usb: USB 2.0 started, EHCI 1.10
[ 2.900074] hub 3-0:1.0: USB hub found
[ 2.904190] hub 3-0:1.0: 1 port detected
[ 2.910578] input: gpio-keys.4 as /devices/soc0/gpio-keys.4/input/input1
[ 2.919631] as3722-rtc as3722-rtc: setting system clock to 2000-02-01 01:41:48 UTC (949369308)
[ 2.947180] ALSA device list:
[ 2.950156] #0: tegra-hda at 0x70038000 irq 113
[ 2.955130] #1: NVIDIA Tegra Venice2
[ 2.971255] kjournald starting. Commit interval 5 seconds
[ 2.973299] EXT3-fs (mmcblk0p1): using internal journal
[ 2.973309] EXT3-fs (mmcblk0p1): mounted filesystem with writeback data mode
[ 2.973433] VFS: Mounted root (ext3 filesystem) on device 179:1.
[ 2.975581] devtmpfs: mounted
[ 2.975934] Freeing unused kernel memory: 268K (c083e000 - c0881000)
[ 3.172384] usb 2-1: new high-speed USB device number 2 using tegra-ehci
[ 3.295260] random: init urandom read with 41 bits of entropy available
[ 3.458939] init: plymouth-upstart-bridge main process (87) terminated with status 1
[ 3.466998] init: plymouth-upstart-bridge main process ended, respawning
[ 3.473804] uvcvideo: Found UVC 1.00 device Chromebook HD WebCam (2232:1033)
[ 3.494214] input: Chromebook HD WebCam as /devices/soc0/7d004000.usb/usb2/2-1/2-1:1.0/input/input2
[ 3.528514] init: plymouth-upstart-bridge main process (97) terminated with status 1
[ 3.536904] init: plymouth-upstart-bridge main process ended, respawning
[ 3.588602] init: plymouth-upstart-bridge main process (106) terminated with status 1
[ 3.596876] init: plymouth-upstart-bridge main process ended, respawning
[ 3.622249] usb 3-1: new high-speed USB device number 2 using tegra-ehci
[ 3.788050] hub 3-1:1.0: USB hub found
[ 3.798810] hub 3-1:1.0: 4 ports detected
[ 4.103186] usb 3-1.1: new high-speed USB device number 3 using tegra-ehci
[ 4.298982] usb-storage 3-1.1:1.0: USB Mass Storage device detected
[ 4.308861] scsi0 : usb-storage 3-1.1:1.0
[ 4.412623] usb 3-1.2: new high-speed USB device number 4 using tegra-ehci
[ 4.605250] systemd-udevd[221]: starting version 204
[ 4.868461] asix 3-1.2:1.0 eth0: register 'asix' at usb-7d008000.usb-1.2, ASIX AX88772 USB 2.0 Ethernet, c0:c1:c0:76:38:8d
[ 5.395455] systemd-udevd[253]: renamed network interface eth0 to eth1
[ 5.623560] scsi 0:0:0:0: Direct-Access Sony Storage Media 0100 PQ: 0 ANSI: 0 CCS
[ 5.634037] sd 0:0:0:0: [sda] 7907328 512-byte logical blocks: (4.04 GB/3.77 GiB)
[ 5.634995] sd 0:0:0:0: [sda] Write Protect is off
[ 5.635006] sd 0:0:0:0: [sda] Mode Sense: 43 00 00 00
[ 5.640806] sd 0:0:0:0: [sda] No Caching mode page found
[ 5.640815] sd 0:0:0:0: [sda] Assuming drive cache: write through
[ 5.673711] random: nonblocking pool is initialized
[ 5.686873] GPT:Primary header thinks Alt. header is not at the end of the disk.
[ 5.686882] GPT:5058495 != 7907327
[ 5.686886] GPT:Alternate GPT header not at the end of the disk.
[ 5.686890] GPT:5058495 != 7907327
[ 5.686892] GPT: Use GNU Parted to correct GPT errors.
[ 5.687003] sda: sda1 sda2 sda3 sda4 sda5 sda6 sda7 sda8 sda9 sda10 sda11 sda12
[ 5.745769] sd 0:0:0:0: [sda] Attached SCSI removable disk
[ 7.370030] init: failsafe main process (377) killed by TERM signal
[ 8.551905] init: plymouth-upstart-bridge main process ended, respawning
[11296.602217] tegra-sor 54540000.sor: failed to probe eDP link: -5
[11296.608258] host1x drm: tegra_output_enable(): -5
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG7tDjw_6wHc=FN0KsOKsSS1u8H8SzYdxt9VCwUsARRTeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-05-22 9:50 ` Thierry Reding
2014-05-22 17:48 ` Dylan Reid
0 siblings, 1 reply; 23+ messages in thread
From: Thierry Reding @ 2014-05-22 9:50 UTC (permalink / raw)
To: Dylan Reid
Cc: Stephen Warren, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 3651 bytes --]
On Wed, May 21, 2014 at 02:10:09PM -0700, Dylan Reid wrote:
> On Wed, May 21, 2014 at 1:55 PM, Thierry Reding
> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> > On Tue, May 20, 2014 at 04:28:45PM -0700, Dylan Reid wrote:
> >> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> >> > On 05/20/2014 02:37 PM, Dylan Reid wrote:
> >> >> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren@wwwdotorg.org> wrote:
> >> >>> 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.
> >> > ...
> >> >>> 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
> >> >>>
> >> >>
> >> >> Thanks for checking Stephen. I'll fix the reset name.
> >> >>
> >> >> This is the state of the tree I used when I tested on Venice2 just now:
> >> >> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
> >> >> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
> >> >> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
> >> >> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
> >> >> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
> >> >
> >> > Ah, I was missing that clock patch. Hopefully Peter will pick it up for
> >> > the Tegra clock tree soon.
> >> >
> >> > Now I have the same results as Thierry; speaker-test looks like it
> >> > should be working, yet I don't hear any audio from the monitor. I know
> >> > the monitor works, since I've used it extensively for testing GeForce
> >> > GPU HDMI audio.
> >>
> >> Probably a silly question: Is it un-muted?
> >> amixer -c0 cset name='IEC958 Playback Switch' on
> >
> > Dylan, can you provide more detail about the setup that you're running?
> > There have been similar issues in the past where a driver would work if
> > it was run on an identical upstream kernel but with a different version
> > of the bootloader (or a different bootloader altogether). Can you give
> > further details about your setup? Also perhaps providing a dmesg log or
> > the clock tree from /sys/kernel/debug/clk/clk_summary could help
> > determine the diff between a working and a non-working setup.
>
> Sure. I'm running this u-boot:
> https://chromium.googlesource.com/chromiumos/third_party/u-boot
> on commit ddde3e9 of the chromeos-v2013.06 branch.
>
> I haven't had a chance to flash swarren's u-boot yet it's been one of
> those days...
>
> I attached dmesg and clk_summary.
>
> I don't have many modifications to the ubuntu-core rootfs.
>
> Thanks for looking at this,
Also perhaps you can share any change to tegra_defconfig that you have
for this. Perhaps Stephen and I are simply missing some required option
to make it work.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
2014-05-22 9:50 ` Thierry Reding
@ 2014-05-22 17:48 ` Dylan Reid
[not found] ` <CAEUnVG5pNMzDoXGso+UiAWPpD0hNPhBPyGr8KfV587AuU70QPQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-22 17:48 UTC (permalink / raw)
To: Thierry Reding
Cc: Stephen Warren, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Thu, May 22, 2014 at 2:50 AM, Thierry Reding
<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> On Wed, May 21, 2014 at 02:10:09PM -0700, Dylan Reid wrote:
>> On Wed, May 21, 2014 at 1:55 PM, Thierry Reding
>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> > On Tue, May 20, 2014 at 04:28:45PM -0700, Dylan Reid wrote:
>> >> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> >> > On 05/20/2014 02:37 PM, Dylan Reid wrote:
>> >> >> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> >> >>> 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.
>> >> > ...
>> >> >>> 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
>> >> >>>
>> >> >>
>> >> >> Thanks for checking Stephen. I'll fix the reset name.
>> >> >>
>> >> >> This is the state of the tree I used when I tested on Venice2 just now:
>> >> >> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
>> >> >> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
>> >> >> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
>> >> >> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
>> >> >> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
>> >> >
>> >> > Ah, I was missing that clock patch. Hopefully Peter will pick it up for
>> >> > the Tegra clock tree soon.
>> >> >
>> >> > Now I have the same results as Thierry; speaker-test looks like it
>> >> > should be working, yet I don't hear any audio from the monitor. I know
>> >> > the monitor works, since I've used it extensively for testing GeForce
>> >> > GPU HDMI audio.
>> >>
>> >> Probably a silly question: Is it un-muted?
>> >> amixer -c0 cset name='IEC958 Playback Switch' on
>> >
>> > Dylan, can you provide more detail about the setup that you're running?
>> > There have been similar issues in the past where a driver would work if
>> > it was run on an identical upstream kernel but with a different version
>> > of the bootloader (or a different bootloader altogether). Can you give
>> > further details about your setup? Also perhaps providing a dmesg log or
>> > the clock tree from /sys/kernel/debug/clk/clk_summary could help
>> > determine the diff between a working and a non-working setup.
>>
>> Sure. I'm running this u-boot:
>> https://chromium.googlesource.com/chromiumos/third_party/u-boot
>> on commit ddde3e9 of the chromeos-v2013.06 branch.
>>
>> I haven't had a chance to flash swarren's u-boot yet it's been one of
>> those days...
>>
>> I attached dmesg and clk_summary.
>>
>> I don't have many modifications to the ubuntu-core rootfs.
>>
>> Thanks for looking at this,
>
> Also perhaps you can share any change to tegra_defconfig that you have
> for this. Perhaps Stephen and I are simply missing some required option
> to make it work.
Sure, here is the diff from linux-next's tegra defconfig. Still on
the tree mentioned above.
--- .config 2014-05-22 10:47:15.048673228 -0700
+++ /tmp/tegra_hda.config 2014-05-22 10:47:08.118588548 -0700
@@ -2395,6 +2395,8 @@
CONFIG_SND_VERBOSE_PROCFS=y
# CONFIG_SND_VERBOSE_PRINTK is not set
# CONFIG_SND_DEBUG is not set
+CONFIG_SND_VMASTER=y
+CONFIG_SND_KCTL_JACK=y
# CONFIG_SND_RAWMIDI_SEQ is not set
# CONFIG_SND_OPL3_LIB_SEQ is not set
# CONFIG_SND_OPL4_LIB_SEQ is not set
@@ -2468,8 +2470,28 @@
#
# HD-Audio
#
+CONFIG_SND_HDA=y
# CONFIG_SND_HDA_INTEL is not set
-# CONFIG_SND_HDA_TEGRA is not set
+CONFIG_SND_HDA_TEGRA=y
+CONFIG_SND_HDA_PREALLOC_SIZE=64
+# CONFIG_SND_HDA_HWDEP is not set
+# CONFIG_SND_HDA_RECONFIG is not set
+# CONFIG_SND_HDA_INPUT_BEEP is not set
+# CONFIG_SND_HDA_INPUT_JACK is not set
+# CONFIG_SND_HDA_PATCH_LOADER is not set
+# CONFIG_SND_HDA_CODEC_REALTEK is not set
+# CONFIG_SND_HDA_CODEC_ANALOG is not set
+# CONFIG_SND_HDA_CODEC_SIGMATEL is not set
+# CONFIG_SND_HDA_CODEC_VIA is not set
+CONFIG_SND_HDA_CODEC_HDMI=y
+# CONFIG_SND_HDA_CODEC_CIRRUS is not set
+# CONFIG_SND_HDA_CODEC_CONEXANT is not set
+# CONFIG_SND_HDA_CODEC_CA0110 is not set
+# CONFIG_SND_HDA_CODEC_CA0132 is not set
+# CONFIG_SND_HDA_CODEC_CMEDIA is not set
+# CONFIG_SND_HDA_CODEC_SI3054 is not set
+# CONFIG_SND_HDA_GENERIC is not set
+CONFIG_SND_HDA_POWER_SAVE_DEFAULT=0
# CONFIG_SND_ARM is not set
# CONFIG_SND_SPI is not set
# CONFIG_SND_USB is not set
>
> Thierry
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG5pNMzDoXGso+UiAWPpD0hNPhBPyGr8KfV587AuU70QPQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-05-22 17:54 ` Stephen Warren
[not found] ` <537E39D9.2000304-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Stephen Warren @ 2014-05-22 17:54 UTC (permalink / raw)
To: Dylan Reid, Thierry Reding
Cc: Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On 05/22/2014 11:48 AM, Dylan Reid wrote:
> On Thu, May 22, 2014 at 2:50 AM, Thierry Reding
> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> On Wed, May 21, 2014 at 02:10:09PM -0700, Dylan Reid wrote:
>>> On Wed, May 21, 2014 at 1:55 PM, Thierry Reding
>>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>>> On Tue, May 20, 2014 at 04:28:45PM -0700, Dylan Reid wrote:
>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>>>> On 05/20/2014 02:37 PM, Dylan Reid wrote:
>>>>>>> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>>>>>> 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.
>>>>>> ...
>>>>>>>> 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
>>>>>>>>
>>>>>>>
>>>>>>> Thanks for checking Stephen. I'll fix the reset name.
>>>>>>>
>>>>>>> This is the state of the tree I used when I tested on Venice2 just now:
>>>>>>> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
>>>>>>> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
>>>>>>> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
>>>>>>> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
>>>>>>> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
>>>>>>
>>>>>> Ah, I was missing that clock patch. Hopefully Peter will pick it up for
>>>>>> the Tegra clock tree soon.
>>>>>>
>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>> GPU HDMI audio.
>>>>>
>>>>> Probably a silly question: Is it un-muted?
>>>>> amixer -c0 cset name='IEC958 Playback Switch' on
>>>>
>>>> Dylan, can you provide more detail about the setup that you're running?
>>>> There have been similar issues in the past where a driver would work if
>>>> it was run on an identical upstream kernel but with a different version
>>>> of the bootloader (or a different bootloader altogether). Can you give
>>>> further details about your setup? Also perhaps providing a dmesg log or
>>>> the clock tree from /sys/kernel/debug/clk/clk_summary could help
>>>> determine the diff between a working and a non-working setup.
>>>
>>> Sure. I'm running this u-boot:
>>> https://chromium.googlesource.com/chromiumos/third_party/u-boot
>>> on commit ddde3e9 of the chromeos-v2013.06 branch.
>>>
>>> I haven't had a chance to flash swarren's u-boot yet it's been one of
>>> those days...
>>>
>>> I attached dmesg and clk_summary.
>>>
>>> I don't have many modifications to the ubuntu-core rootfs.
>>>
>>> Thanks for looking at this,
>>
>> Also perhaps you can share any change to tegra_defconfig that you have
>> for this. Perhaps Stephen and I are simply missing some required option
>> to make it work.
>
> Sure, here is the diff from linux-next's tegra defconfig. Still on
> the tree mentioned above.
My .config does include all the options that your defconfig diff adds.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <537E39D9.2000304-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
@ 2014-05-22 18:05 ` Dylan Reid
[not found] ` <CAEUnVG4UettOj0xDy+f6qEhUbhNs=u-wgyCf_Prsd2JpnmFfLw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-22 18:05 UTC (permalink / raw)
To: Stephen Warren
Cc: Thierry Reding, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Thu, May 22, 2014 at 10:54 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> On 05/22/2014 11:48 AM, Dylan Reid wrote:
>> On Thu, May 22, 2014 at 2:50 AM, Thierry Reding
>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>> On Wed, May 21, 2014 at 02:10:09PM -0700, Dylan Reid wrote:
>>>> On Wed, May 21, 2014 at 1:55 PM, Thierry Reding
>>>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>>>> On Tue, May 20, 2014 at 04:28:45PM -0700, Dylan Reid wrote:
>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>>>>> On 05/20/2014 02:37 PM, Dylan Reid wrote:
>>>>>>>> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>>>>>>> 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.
>>>>>>> ...
>>>>>>>>> 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
>>>>>>>>>
>>>>>>>>
>>>>>>>> Thanks for checking Stephen. I'll fix the reset name.
>>>>>>>>
>>>>>>>> This is the state of the tree I used when I tested on Venice2 just now:
>>>>>>>> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
>>>>>>>> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
>>>>>>>> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
>>>>>>>> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
>>>>>>>> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
>>>>>>>
>>>>>>> Ah, I was missing that clock patch. Hopefully Peter will pick it up for
>>>>>>> the Tegra clock tree soon.
>>>>>>>
>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>>> GPU HDMI audio.
>>>>>>
>>>>>> Probably a silly question: Is it un-muted?
>>>>>> amixer -c0 cset name='IEC958 Playback Switch' on
>>>>>
>>>>> Dylan, can you provide more detail about the setup that you're running?
>>>>> There have been similar issues in the past where a driver would work if
>>>>> it was run on an identical upstream kernel but with a different version
>>>>> of the bootloader (or a different bootloader altogether). Can you give
>>>>> further details about your setup? Also perhaps providing a dmesg log or
>>>>> the clock tree from /sys/kernel/debug/clk/clk_summary could help
>>>>> determine the diff between a working and a non-working setup.
>>>>
>>>> Sure. I'm running this u-boot:
>>>> https://chromium.googlesource.com/chromiumos/third_party/u-boot
>>>> on commit ddde3e9 of the chromeos-v2013.06 branch.
>>>>
>>>> I haven't had a chance to flash swarren's u-boot yet it's been one of
>>>> those days...
>>>>
>>>> I attached dmesg and clk_summary.
>>>>
>>>> I don't have many modifications to the ubuntu-core rootfs.
>>>>
>>>> Thanks for looking at this,
>>>
>>> Also perhaps you can share any change to tegra_defconfig that you have
>>> for this. Perhaps Stephen and I are simply missing some required option
>>> to make it work.
>>
>> Sure, here is the diff from linux-next's tegra defconfig. Still on
>> the tree mentioned above.
>
> My .config does include all the options that your defconfig diff adds.
OK. I'll try to dig myself out from these other bugs and give this a
try on the Jetson this afternoon.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG4UettOj0xDy+f6qEhUbhNs=u-wgyCf_Prsd2JpnmFfLw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-05-23 3:55 ` Dylan Reid
2014-05-23 19:05 ` Stephen Warren
0 siblings, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-23 3:55 UTC (permalink / raw)
To: Stephen Warren
Cc: Thierry Reding, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Thu, May 22, 2014 at 11:05 AM, Dylan Reid <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> wrote:
> On Thu, May 22, 2014 at 10:54 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> On 05/22/2014 11:48 AM, Dylan Reid wrote:
>>> On Thu, May 22, 2014 at 2:50 AM, Thierry Reding
>>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>>> On Wed, May 21, 2014 at 02:10:09PM -0700, Dylan Reid wrote:
>>>>> On Wed, May 21, 2014 at 1:55 PM, Thierry Reding
>>>>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>>>>> On Tue, May 20, 2014 at 04:28:45PM -0700, Dylan Reid wrote:
>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>>>>>> On 05/20/2014 02:37 PM, Dylan Reid wrote:
>>>>>>>>> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>>>>>>>> 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.
>>>>>>>> ...
>>>>>>>>>> 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
>>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Thanks for checking Stephen. I'll fix the reset name.
>>>>>>>>>
>>>>>>>>> This is the state of the tree I used when I tested on Venice2 just now:
>>>>>>>>> * ae57dbf - (HEAD) ARM: tegra: venice2 - Enable HDA (2 minutes ago)
>>>>>>>>> * 6224524 - ARM: tegra: Add Tegra124 HDA support (2 minutes ago)
>>>>>>>>> * 468ca57 - ALSA: hda - Add driver for Tegra SoC HDA (2 minutes ago)
>>>>>>>>> * 928099e - ALSA: hda/hdmi - Add Nvidia Tegra124 HDMI support (2 minutes ago)
>>>>>>>>> * 701eac4 - clk: tegra124: Enable hda to hdmi clocks (3 minutes ago)
>>>>>>>>
>>>>>>>> Ah, I was missing that clock patch. Hopefully Peter will pick it up for
>>>>>>>> the Tegra clock tree soon.
>>>>>>>>
>>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>>>> GPU HDMI audio.
>>>>>>>
>>>>>>> Probably a silly question: Is it un-muted?
>>>>>>> amixer -c0 cset name='IEC958 Playback Switch' on
>>>>>>
>>>>>> Dylan, can you provide more detail about the setup that you're running?
>>>>>> There have been similar issues in the past where a driver would work if
>>>>>> it was run on an identical upstream kernel but with a different version
>>>>>> of the bootloader (or a different bootloader altogether). Can you give
>>>>>> further details about your setup? Also perhaps providing a dmesg log or
>>>>>> the clock tree from /sys/kernel/debug/clk/clk_summary could help
>>>>>> determine the diff between a working and a non-working setup.
>>>>>
>>>>> Sure. I'm running this u-boot:
>>>>> https://chromium.googlesource.com/chromiumos/third_party/u-boot
>>>>> on commit ddde3e9 of the chromeos-v2013.06 branch.
>>>>>
>>>>> I haven't had a chance to flash swarren's u-boot yet it's been one of
>>>>> those days...
>>>>>
>>>>> I attached dmesg and clk_summary.
>>>>>
>>>>> I don't have many modifications to the ubuntu-core rootfs.
>>>>>
>>>>> Thanks for looking at this,
>>>>
>>>> Also perhaps you can share any change to tegra_defconfig that you have
>>>> for this. Perhaps Stephen and I are simply missing some required option
>>>> to make it work.
>>>
>>> Sure, here is the diff from linux-next's tegra defconfig. Still on
>>> the tree mentioned above.
>>
>> My .config does include all the options that your defconfig diff adds.
>
> OK. I'll try to dig myself out from these other bugs and give this a
> try on the Jetson this afternoon.
Took all day, but I did get to try this.
on U-Boot commit d7782d0, flashed with Stephen's u-boot flasher from github.
And kernel at 81d0207 - ARM: tegra: enable HD-Audio controller in defconfig
plus the addition of the hda node "okay" to the jetson-tk1 DT.
I can hear the jetson's audio on the TV. This is currently a sample
size of one TV, I'll set up the Quantum Data tomorrow and check that
it works there as well.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
2014-05-23 3:55 ` Dylan Reid
@ 2014-05-23 19:05 ` Stephen Warren
[not found] ` <537F9BEB.5020102-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Stephen Warren @ 2014-05-23 19:05 UTC (permalink / raw)
To: Dylan Reid
Cc: Thierry Reding, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On 05/22/2014 09:55 PM, Dylan Reid wrote:
...
>>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
...
>>>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>>>>> GPU HDMI audio.
...
> Took all day, but I did get to try this.
>
> on U-Boot commit d7782d0, flashed with Stephen's u-boot flasher from github.
>
> And kernel at 81d0207 - ARM: tegra: enable HD-Audio controller in defconfig
> plus the addition of the hda node "okay" to the jetson-tk1 DT.
>
> I can hear the jetson's audio on the TV. This is currently a sample
> size of one TV, I'll set up the Quantum Data tomorrow and check that
> it works there as well.
I tried that same U-Boot commit (with a few device-mode USB patches on
top that shouldn't affect anything since I didn't use USB device mode)
and have the same results.
My monitor is a Dell U2410 with sound bar.
I did notice one strange thing: /proc/asound/card0/eld#3.0 said:
monitor_present 0
eld_valid 0
This is odd, since ELD information really should be present. Perhaps
tegradrm simply doesn't have the code to set up an eld yet though?
Anyway, I thought that in the absence of ELD information, ALSA core fell
back to allowing arbitrary audio formats to be used in the hope they'd
work anyway, so perhaps this isn't an issue.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <537F9BEB.5020102-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
@ 2014-05-23 20:34 ` Dylan Reid
[not found] ` <CAEUnVG63BCgpGuo+qcYkvpnVSSguBy9xwSC_AoZqCt1RgBqm7w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-23 20:34 UTC (permalink / raw)
To: Stephen Warren
Cc: Thierry Reding, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Fri, May 23, 2014 at 12:05 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> On 05/22/2014 09:55 PM, Dylan Reid wrote:
> ...
>>>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> ...
>>>>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>>>>>> GPU HDMI audio.
> ...
>> Took all day, but I did get to try this.
>>
>> on U-Boot commit d7782d0, flashed with Stephen's u-boot flasher from github.
>>
>> And kernel at 81d0207 - ARM: tegra: enable HD-Audio controller in defconfig
>> plus the addition of the hda node "okay" to the jetson-tk1 DT.
>>
>> I can hear the jetson's audio on the TV. This is currently a sample
>> size of one TV, I'll set up the Quantum Data tomorrow and check that
>> it works there as well.
>
> I tried that same U-Boot commit (with a few device-mode USB patches on
> top that shouldn't affect anything since I didn't use USB device mode)
> and have the same results.
>
> My monitor is a Dell U2410 with sound bar.
>
> I did notice one strange thing: /proc/asound/card0/eld#3.0 said:
>
> monitor_present 0
> eld_valid 0
>
> This is odd, since ELD information really should be present. Perhaps
> tegradrm simply doesn't have the code to set up an eld yet though?
> Anyway, I thought that in the absence of ELD information, ALSA core fell
> back to allowing arbitrary audio formats to be used in the hope they'd
> work anyway, so perhaps this isn't an issue.
I see the same thing. I wouldn't be surprised if the ELD isn't yet
configured by tegradrm.
I tried one other monitor that also worked.
I also ran the HDMI audio compliance test suite. It failed the ACR
(clock regeneration packet) test and the audio packet timing jitter
test, I could see that upsetting some monitors. It looks like ACR
packets are generated for 44.1 even when playing 48k. I'm not sure
why the jitter test wouldn't run, the QD didn't give a reason for
failure on that one.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG63BCgpGuo+qcYkvpnVSSguBy9xwSC_AoZqCt1RgBqm7w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-05-31 1:45 ` Dylan Reid
[not found] ` <CAEUnVG52fJhd9213fdydoUW11N0yZ31T2MZ2BPmd5Q2xkZ8XeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-04 17:13 ` Stephen Warren
1 sibling, 1 reply; 23+ messages in thread
From: Dylan Reid @ 2014-05-31 1:45 UTC (permalink / raw)
To: Stephen Warren
Cc: Thierry Reding, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Fri, May 23, 2014 at 1:34 PM, Dylan Reid <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> wrote:
> On Fri, May 23, 2014 at 12:05 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> On 05/22/2014 09:55 PM, Dylan Reid wrote:
>> ...
>>>>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> ...
>>>>>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>>>>>>> GPU HDMI audio.
>> ...
>>> Took all day, but I did get to try this.
>>>
>>> on U-Boot commit d7782d0, flashed with Stephen's u-boot flasher from github.
>>>
>>> And kernel at 81d0207 - ARM: tegra: enable HD-Audio controller in defconfig
>>> plus the addition of the hda node "okay" to the jetson-tk1 DT.
>>>
>>> I can hear the jetson's audio on the TV. This is currently a sample
>>> size of one TV, I'll set up the Quantum Data tomorrow and check that
>>> it works there as well.
>>
>> I tried that same U-Boot commit (with a few device-mode USB patches on
>> top that shouldn't affect anything since I didn't use USB device mode)
>> and have the same results.
>>
>> My monitor is a Dell U2410 with sound bar.
I found one of these. I also didn't hear any audio. In fact I
couldn't get audio out of most monitors, TV sets worked fine.
hdmi.c couldn't configure audio for the pixel clock generated for the
monitor. Luckily I found a patch in our downstream tree that fixes
it.
video: tegra: Calculate HDMI audio CTS/N/AVAL values -- 7/25/11 author
swarren =)
Thanks for fixing this for me!
I can send this up unless you'd rather send it yourself.
I appended the patch moved to the new location of hdmi.c:
video: tegra: Calculate HDMI audio CTS/N/AVAL values
The current table-drive approach for determining the CTS/N/AVAL values
required for HDMI audio limits audio availability to a small set of
pixel clock frequencies. Allow audio in other cases by calculating the
CTS and N value dynamically where the tables don't contain
pre-calculated values.
---
drivers/gpu/drm/tegra/hdmi.c | 143 +++++++++++--------------------------------
1 file changed, 35 insertions(+), 108 deletions(-)
diff --git a/drivers/gpu/drm/tegra/hdmi.c b/drivers/gpu/drm/tegra/hdmi.c
index fec1a63..453e515 100644
--- a/drivers/gpu/drm/tegra/hdmi.c
+++ b/drivers/gpu/drm/tegra/hdmi.c
@@ -9,6 +9,7 @@
#include <linux/clk.h>
#include <linux/debugfs.h>
+#include <linux/gcd.h>
#include <linux/hdmi.h>
#include <linux/regulator/consumer.h>
#include <linux/reset.h>
@@ -104,61 +105,34 @@ struct tegra_hdmi_audio_config {
unsigned int aval;
};
-static const struct tegra_hdmi_audio_config tegra_hdmi_audio_32k[] = {
- { 25200000, 4096, 25200, 24000 },
- { 27000000, 4096, 27000, 24000 },
- { 74250000, 4096, 74250, 24000 },
- { 148500000, 4096, 148500, 24000 },
- { 0, 0, 0, 0 },
-};
-
-static const struct tegra_hdmi_audio_config tegra_hdmi_audio_44_1k[] = {
- { 25200000, 5880, 26250, 25000 },
- { 27000000, 5880, 28125, 25000 },
- { 74250000, 4704, 61875, 20000 },
- { 148500000, 4704, 123750, 20000 },
- { 0, 0, 0, 0 },
-};
-
-static const struct tegra_hdmi_audio_config tegra_hdmi_audio_48k[] = {
- { 25200000, 6144, 25200, 24000 },
- { 27000000, 6144, 27000, 24000 },
- { 74250000, 6144, 74250, 24000 },
- { 148500000, 6144, 148500, 24000 },
- { 0, 0, 0, 0 },
-};
-
-static const struct tegra_hdmi_audio_config tegra_hdmi_audio_88_2k[] = {
- { 25200000, 11760, 26250, 25000 },
- { 27000000, 11760, 28125, 25000 },
- { 74250000, 9408, 61875, 20000 },
- { 148500000, 9408, 123750, 20000 },
- { 0, 0, 0, 0 },
-};
-
-static const struct tegra_hdmi_audio_config tegra_hdmi_audio_96k[] = {
- { 25200000, 12288, 25200, 24000 },
- { 27000000, 12288, 27000, 24000 },
- { 74250000, 12288, 74250, 24000 },
- { 148500000, 12288, 148500, 24000 },
- { 0, 0, 0, 0 },
-};
+static int tegra_hdmi_calc_audio_cts_n(unsigned fs, unsigned pixel_rate,
+ unsigned *hda_cts, unsigned *hda_n,
+ unsigned *hda_aval)
+{
+ /* Ideal ACR interval is 1000 hz (1 ms) */
+ unsigned n_ideal = (128 * fs) / 1000;
+ unsigned cts = pixel_rate;
+ unsigned n = 128 * fs;
+ unsigned common_divisor = gcd(cts, n);
+ unsigned mult;
+
+ cts /= common_divisor;
+ n /= common_divisor;
+ mult = n_ideal / n;
+ if (mult) {
+ n *= mult;
+ cts *= mult;
+ }
-static const struct tegra_hdmi_audio_config tegra_hdmi_audio_176_4k[] = {
- { 25200000, 23520, 26250, 25000 },
- { 27000000, 23520, 28125, 25000 },
- { 74250000, 18816, 61875, 20000 },
- { 148500000, 18816, 123750, 20000 },
- { 0, 0, 0, 0 },
-};
+ if (cts < (1 << 20)) {
+ *hda_cts = cts;
+ *hda_n = n;
+ *hda_aval = 240000UL * n / (128 * fs / 100);
+ return 0;
+ }
-static const struct tegra_hdmi_audio_config tegra_hdmi_audio_192k[] = {
- { 25200000, 24576, 25200, 24000 },
- { 27000000, 24576, 27000, 24000 },
- { 74250000, 24576, 74250, 24000 },
- { 148500000, 24576, 148500, 24000 },
- { 0, 0, 0, 0 },
-};
+ return -EINVAL;
+}
static const struct tmds_config tegra20_tmds_config[] = {
{ /* slow pixel clock modes */
@@ -397,54 +371,6 @@ static const struct tmds_config tegra124_tmds_config[] = {
},
};
-static const struct tegra_hdmi_audio_config *
-tegra_hdmi_get_audio_config(unsigned int audio_freq, unsigned int pclk)
-{
- const struct tegra_hdmi_audio_config *table;
-
- switch (audio_freq) {
- case 32000:
- table = tegra_hdmi_audio_32k;
- break;
-
- case 44100:
- table = tegra_hdmi_audio_44_1k;
- break;
-
- case 48000:
- table = tegra_hdmi_audio_48k;
- break;
-
- case 88200:
- table = tegra_hdmi_audio_88_2k;
- break;
-
- case 96000:
- table = tegra_hdmi_audio_96k;
- break;
-
- case 176400:
- table = tegra_hdmi_audio_176_4k;
- break;
-
- case 192000:
- table = tegra_hdmi_audio_192k;
- break;
-
- default:
- return NULL;
- }
-
- while (table->pclk) {
- if (table->pclk == pclk)
- return table;
-
- table++;
- }
-
- return NULL;
-}
-
static void tegra_hdmi_setup_audio_fs_tables(struct tegra_hdmi *hdmi)
{
const unsigned int freqs[] = {
@@ -475,7 +401,7 @@ static void
tegra_hdmi_setup_audio_fs_tables(struct tegra_hdmi *hdmi)
static int tegra_hdmi_setup_audio(struct tegra_hdmi *hdmi, unsigned int pclk)
{
struct device_node *node = hdmi->dev->of_node;
- const struct tegra_hdmi_audio_config *config;
+ unsigned hda_n, hda_cts, hda_aval;
unsigned int offset = 0;
unsigned long value;
@@ -506,8 +432,9 @@ static int tegra_hdmi_setup_audio(struct
tegra_hdmi *hdmi, unsigned int pclk)
tegra_hdmi_writel(hdmi, value, HDMI_NV_PDISP_AUDIO_CNTRL0);
}
- config = tegra_hdmi_get_audio_config(hdmi->audio_freq, pclk);
- if (!config) {
+ if (tegra_hdmi_calc_audio_cts_n(hdmi->audio_freq,
+ clk_get_rate(hdmi->clk),
+ &hda_cts, &hda_n, &hda_aval) < 0) {
dev_err(hdmi->dev, "cannot set audio to %u at %u pclk\n",
hdmi->audio_freq, pclk);
return -EINVAL;
@@ -516,13 +443,13 @@ static int tegra_hdmi_setup_audio(struct
tegra_hdmi *hdmi, unsigned int pclk)
tegra_hdmi_writel(hdmi, 0, HDMI_NV_PDISP_HDMI_ACR_CTRL);
value = AUDIO_N_RESETF | AUDIO_N_GENERATE_ALTERNATE |
- AUDIO_N_VALUE(config->n - 1);
+ AUDIO_N_VALUE(hda_n - 1);
tegra_hdmi_writel(hdmi, value, HDMI_NV_PDISP_AUDIO_N);
- tegra_hdmi_writel(hdmi, ACR_SUBPACK_N(config->n) | ACR_ENABLE,
+ tegra_hdmi_writel(hdmi, ACR_SUBPACK_N(hda_n) | ACR_ENABLE,
HDMI_NV_PDISP_HDMI_ACR_0441_SUBPACK_HIGH);
- value = ACR_SUBPACK_CTS(config->cts);
+ value = ACR_SUBPACK_CTS(hda_cts);
tegra_hdmi_writel(hdmi, value, HDMI_NV_PDISP_HDMI_ACR_0441_SUBPACK_LOW);
value = SPARE_HW_CTS | SPARE_FORCE_SW_CTS | SPARE_CTS_RESET_VAL(1);
@@ -563,7 +490,7 @@ static int tegra_hdmi_setup_audio(struct
tegra_hdmi *hdmi, unsigned int pclk)
break;
}
- tegra_hdmi_writel(hdmi, config->aval, offset);
+ tegra_hdmi_writel(hdmi, hda_aval, offset);
}
tegra_hdmi_setup_audio_fs_tables(hdmi);
--
1.8.1.3.605.g02339dd
^ permalink raw reply related [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG63BCgpGuo+qcYkvpnVSSguBy9xwSC_AoZqCt1RgBqm7w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-31 1:45 ` Dylan Reid
@ 2014-06-04 17:13 ` Stephen Warren
1 sibling, 0 replies; 23+ messages in thread
From: Stephen Warren @ 2014-06-04 17:13 UTC (permalink / raw)
To: Dylan Reid
Cc: Thierry Reding, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On 05/23/2014 02:34 PM, Dylan Reid wrote:
...
> I also ran the HDMI audio compliance test suite. It failed the ACR
> (clock regeneration packet) test and the audio packet timing jitter
> test, I could see that upsetting some monitors. It looks like ACR
> packets are generated for 44.1 even when playing 48k.
That's because the only thing that sets hdmi->audio_freq is
tegra_hdmi_probe(). There needs to be a callback from the sound/HDA
driver to the display driver telling it what audio frequency is in use.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG52fJhd9213fdydoUW11N0yZ31T2MZ2BPmd5Q2xkZ8XeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-06-04 17:32 ` Stephen Warren
[not found] ` <538F5848.1050803-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
[not found] ` <CAEUnVG7bbcPjRcGxn_GAJ7rKJy4g_5veEsRnU 2yCDVWFY3ncUQ@mail.gmail.com>
0 siblings, 2 replies; 23+ messages in thread
From: Stephen Warren @ 2014-06-04 17:32 UTC (permalink / raw)
To: Dylan Reid
Cc: Thierry Reding, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On 05/30/2014 07:45 PM, Dylan Reid wrote:
> On Fri, May 23, 2014 at 1:34 PM, Dylan Reid <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> wrote:
>> On Fri, May 23, 2014 at 12:05 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>> On 05/22/2014 09:55 PM, Dylan Reid wrote:
>>> ...
>>>>>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>> ...
>>>>>>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>>>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>>>>>>>> GPU HDMI audio.
>>> ...
>>>> Took all day, but I did get to try this.
>>>>
>>>> on U-Boot commit d7782d0, flashed with Stephen's u-boot flasher from github.
>>>>
>>>> And kernel at 81d0207 - ARM: tegra: enable HD-Audio controller in defconfig
>>>> plus the addition of the hda node "okay" to the jetson-tk1 DT.
>>>>
>>>> I can hear the jetson's audio on the TV. This is currently a sample
>>>> size of one TV, I'll set up the Quantum Data tomorrow and check that
>>>> it works there as well.
>>>
>>> I tried that same U-Boot commit (with a few device-mode USB patches on
>>> top that shouldn't affect anything since I didn't use USB device mode)
>>> and have the same results.
>>>
>>> My monitor is a Dell U2410 with sound bar.
>
> I found one of these. I also didn't hear any audio. In fact I
> couldn't get audio out of most monitors, TV sets worked fine.
>
> hdmi.c couldn't configure audio for the pixel clock generated for the
> monitor. Luckily I found a patch in our downstream tree that fixes
> it.
>
> video: tegra: Calculate HDMI audio CTS/N/AVAL values -- 7/25/11 author
> swarren =)
>
> Thanks for fixing this for me!
>
> I can send this up unless you'd rather send it yourself.
>
> I appended the patch moved to the new location of hdmi.c:
That wouldn't apply for me since all the TABs were spaces in the email.
I tracked down the original and manually ported it over. You can find it
in my github if you're interested:
git://github.com/swarren/linux-tegra.git tegra_dev
That patch does make things a bit better, but there are still some issues.
When my (Venice2) system boots, the login screen is displayed at native
LCD resolution, and my HDMI monitor displays a 1920x1200 chunk from the
top-left of the LCD. At this point, I can play 44.1KHz audio without
issue. 48KHz doesn't work due to hdmi->audio_freq not being updated, as
I mentioned in my previous mail.
However, once I log in, the window manager changes the HDMI output to
1024x768@75Hz right-of the LCD panel. I have no idea why it does this,
but still... Without the audio ACR value calculation patch, the HDMI
video output fine at this point. However, with the patch, the monitor
won't sync to the signal; it keeps entering/leaving power-save mode.
I suspect this is simply because without the patch, I see the following
error when setting up HDMI for the login screen:
tegra-hdmi 54280000.hdmi: cannot set audio to 44100 at 154000000 pclk
... and when that happens, tegra_output_hdmi_enable() sets hdmi->dvi =
true, which disables a whole bunch of code. However, with the patch
applied, the audio setup doesn't fail, so all that infoframe/... code
runs now, and I suppose there are some issues with it.
Are you able to look into this, or do you need me to investigate further?
I also couldn't persuade xrandr to switch back to a sane resolution on
the HDMI port to see if that resolved the issue. I suppose that's
probably unrelated to the audio code.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <538F5848.1050803-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
@ 2014-06-04 17:51 ` Dylan Reid
0 siblings, 0 replies; 23+ messages in thread
From: Dylan Reid @ 2014-06-04 17:51 UTC (permalink / raw)
To: Stephen Warren
Cc: Thierry Reding, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On Wed, Jun 4, 2014 at 10:32 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> On 05/30/2014 07:45 PM, Dylan Reid wrote:
>> On Fri, May 23, 2014 at 1:34 PM, Dylan Reid <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> wrote:
>>> On Fri, May 23, 2014 at 12:05 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>> On 05/22/2014 09:55 PM, Dylan Reid wrote:
>>>> ...
>>>>>>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>> ...
>>>>>>>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>>>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>>>>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>>>>>>>>> GPU HDMI audio.
>>>> ...
>>>>> Took all day, but I did get to try this.
>>>>>
>>>>> on U-Boot commit d7782d0, flashed with Stephen's u-boot flasher from github.
>>>>>
>>>>> And kernel at 81d0207 - ARM: tegra: enable HD-Audio controller in defconfig
>>>>> plus the addition of the hda node "okay" to the jetson-tk1 DT.
>>>>>
>>>>> I can hear the jetson's audio on the TV. This is currently a sample
>>>>> size of one TV, I'll set up the Quantum Data tomorrow and check that
>>>>> it works there as well.
>>>>
>>>> I tried that same U-Boot commit (with a few device-mode USB patches on
>>>> top that shouldn't affect anything since I didn't use USB device mode)
>>>> and have the same results.
>>>>
>>>> My monitor is a Dell U2410 with sound bar.
>>
>> I found one of these. I also didn't hear any audio. In fact I
>> couldn't get audio out of most monitors, TV sets worked fine.
>>
>> hdmi.c couldn't configure audio for the pixel clock generated for the
>> monitor. Luckily I found a patch in our downstream tree that fixes
>> it.
>>
>> video: tegra: Calculate HDMI audio CTS/N/AVAL values -- 7/25/11 author
>> swarren =)
>>
>> Thanks for fixing this for me!
>>
>> I can send this up unless you'd rather send it yourself.
>>
>> I appended the patch moved to the new location of hdmi.c:
>
> That wouldn't apply for me since all the TABs were spaces in the email.
> I tracked down the original and manually ported it over. You can find it
> in my github if you're interested:
>
> git://github.com/swarren/linux-tegra.git tegra_dev
>
> That patch does make things a bit better, but there are still some issues.
>
> When my (Venice2) system boots, the login screen is displayed at native
> LCD resolution, and my HDMI monitor displays a 1920x1200 chunk from the
> top-left of the LCD. At this point, I can play 44.1KHz audio without
> issue. 48KHz doesn't work due to hdmi->audio_freq not being updated, as
> I mentioned in my previous mail.
Do you have a good idea how to convey this information from the audio
driver over to HDMI?
Downstream I see we have a hideous if CONFIG_TEGRA_DC in patch_hdmi.
>
> However, once I log in, the window manager changes the HDMI output to
> 1024x768@75Hz right-of the LCD panel. I have no idea why it does this,
> but still... Without the audio ACR value calculation patch, the HDMI
> video output fine at this point. However, with the patch, the monitor
> won't sync to the signal; it keeps entering/leaving power-save mode.
>
> I suspect this is simply because without the patch, I see the following
> error when setting up HDMI for the login screen:
>
> tegra-hdmi 54280000.hdmi: cannot set audio to 44100 at 154000000 pclk
>
> ... and when that happens, tegra_output_hdmi_enable() sets hdmi->dvi =
> true, which disables a whole bunch of code. However, with the patch
> applied, the audio setup doesn't fail, so all that infoframe/... code
> runs now, and I suppose there are some issues with it.
>
> Are you able to look into this, or do you need me to investigate further?
I'm travelling for the rest of the week, but I can set aside time to
look at it on Monday. If you make any more progress before then let
me know.
Thanks for spending the time looking at this.
Dylan
>
> I also couldn't persuade xrandr to switch back to a sane resolution on
> the HDMI port to see if that resolved the issue. I suppose that's
> probably unrelated to the audio code.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <CAEUnVG7bbcPjRcGxn_GAJ7rKJy4g_5veEsRnU2yCDVWFY3ncUQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
@ 2014-06-04 19:04 ` Stephen Warren
[not found] ` <538F6DA5.9040608-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
0 siblings, 1 reply; 23+ messages in thread
From: Stephen Warren @ 2014-06-04 19:04 UTC (permalink / raw)
To: Dylan Reid, Thierry Reding
Cc: Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
On 06/04/2014 11:51 AM, Dylan Reid wrote:
> On Wed, Jun 4, 2014 at 10:32 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>> On 05/30/2014 07:45 PM, Dylan Reid wrote:
>>> On Fri, May 23, 2014 at 1:34 PM, Dylan Reid <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> wrote:
>>>> On Fri, May 23, 2014 at 12:05 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>>> On 05/22/2014 09:55 PM, Dylan Reid wrote:
>>>>> ...
>>>>>>>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
>>>>> ...
>>>>>>>>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
>>>>>>>>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
>>>>>>>>>>>>>> the monitor works, since I've used it extensively for testing GeForce
>>>>>>>>>>>>>> GPU HDMI audio.
>>>>> ...
>>>>>> Took all day, but I did get to try this.
>>>>>>
>>>>>> on U-Boot commit d7782d0, flashed with Stephen's u-boot flasher from github.
>>>>>>
>>>>>> And kernel at 81d0207 - ARM: tegra: enable HD-Audio controller in defconfig
>>>>>> plus the addition of the hda node "okay" to the jetson-tk1 DT.
>>>>>>
>>>>>> I can hear the jetson's audio on the TV. This is currently a sample
>>>>>> size of one TV, I'll set up the Quantum Data tomorrow and check that
>>>>>> it works there as well.
>>>>>
>>>>> I tried that same U-Boot commit (with a few device-mode USB patches on
>>>>> top that shouldn't affect anything since I didn't use USB device mode)
>>>>> and have the same results.
>>>>>
>>>>> My monitor is a Dell U2410 with sound bar.
>>>
>>> I found one of these. I also didn't hear any audio. In fact I
>>> couldn't get audio out of most monitors, TV sets worked fine.
>>>
>>> hdmi.c couldn't configure audio for the pixel clock generated for the
>>> monitor. Luckily I found a patch in our downstream tree that fixes
>>> it.
>>>
>>> video: tegra: Calculate HDMI audio CTS/N/AVAL values -- 7/25/11 author
>>> swarren =)
>>>
>>> Thanks for fixing this for me!
>>>
>>> I can send this up unless you'd rather send it yourself.
>>>
>>> I appended the patch moved to the new location of hdmi.c:
>>
>> That wouldn't apply for me since all the TABs were spaces in the email.
>> I tracked down the original and manually ported it over. You can find it
>> in my github if you're interested:
>>
>> git://github.com/swarren/linux-tegra.git tegra_dev
>>
>> That patch does make things a bit better, but there are still some issues.
>>
>> When my (Venice2) system boots, the login screen is displayed at native
>> LCD resolution, and my HDMI monitor displays a 1920x1200 chunk from the
>> top-left of the LCD. At this point, I can play 44.1KHz audio without
>> issue. 48KHz doesn't work due to hdmi->audio_freq not being updated, as
>> I mentioned in my previous mail.
>
> Do you have a good idea how to convey this information from the audio
> driver over to HDMI?
> Downstream I see we have a hideous if CONFIG_TEGRA_DC in patch_hdmi.
The only way of avoiding that would be to create some kind of common
infra-structure where video drivers can register for notifications from
arbitrary audio devices. Then, the HDA code could always call this, and
there'd be no direct references between the two drivers. To be honest
though, this feels like creating a whole bunch of overhead for something
that's very simple and likely not a maintenance issue, so I'm not sure
there's much benefit doing that.
>> However, once I log in, the window manager changes the HDMI output to
>> 1024x768@75Hz right-of the LCD panel. I have no idea why it does this,
>> but still... Without the audio ACR value calculation patch, the HDMI
>> video output fine at this point. However, with the patch, the monitor
>> won't sync to the signal; it keeps entering/leaving power-save mode.
>>
>> I suspect this is simply because without the patch, I see the following
>> error when setting up HDMI for the login screen:
>>
>> tegra-hdmi 54280000.hdmi: cannot set audio to 44100 at 154000000 pclk
>>
>> ... and when that happens, tegra_output_hdmi_enable() sets hdmi->dvi =
>> true, which disables a whole bunch of code. However, with the patch
>> applied, the audio setup doesn't fail, so all that infoframe/... code
>> runs now, and I suppose there are some issues with it.
>>
>> Are you able to look into this, or do you need me to investigate further?
>
> I'm travelling for the rest of the week, but I can set aside time to
> look at it on Monday. If you make any more progress before then let
> me know.
I don't know how useful this is, but I found that if I apply the
following patch, video still works:
> diff --git a/drivers/gpu/drm/tegra/hdmi.c b/drivers/gpu/drm/tegra/hdmi.c
> index c1853aee3716..bb271efef263 100644
> --- a/drivers/gpu/drm/tegra/hdmi.c
> +++ b/drivers/gpu/drm/tegra/hdmi.c
> @@ -560,7 +560,7 @@ static void tegra_hdmi_setup_avi_infoframe(struct tegra_hdmi *hdmi,
> u8 buffer[17];
> ssize_t err;
>
> - if (hdmi->dvi) {
> + if (1 /*hdmi->dvi*/) {
> tegra_hdmi_writel(hdmi, 0,
> HDMI_NV_PDISP_HDMI_AVI_INFOFRAME_CTRL);
> return;
> @@ -809,7 +809,7 @@ static int tegra_output_hdmi_enable(struct tegra_output *output)
>
> tegra_hdmi_writel(hdmi, value, HDMI_NV_PDISP_HDMI_CTRL);
>
> - if (hdmi->dvi)
> + if (1 /*hdmi->dvi*/)
> tegra_hdmi_writel(hdmi, 0x0,
> HDMI_NV_PDISP_HDMI_GENERIC_CTRL);
> else
With either of those changes missing, I get the problem I described before.
Even with those hacks, something is obviously a bit wrong; my monitor
reports 1026x768 rather than 1024x768 as the resolution!
I'm not sure if this is related, but I notice hdmi.c contains a few:
if (of_device_is_compatible(node, "nvidia,tegra30-hdmi"))
I assume that code should also run on Tegra114/124 since the code is
probably touching HW that was added between Tegra20 and Tegra30, and
then probably not removed later. Still, I can't say for sure. These
tests aren't succeeding now, since the HDMI DT not in tegra124.dtsi only
includes "nvidia,tegra124-hdmi" in the compatible value. "Fixing" those
tests to succeed on Tegra124 breaks more functionality.
It's probably best if I leave further investigation to Thierry since he
knows the HDMI device much more than I, or perhaps you.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <538F6DA5.9040608-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
@ 2014-06-04 20:43 ` Thierry Reding
0 siblings, 0 replies; 23+ messages in thread
From: Thierry Reding @ 2014-06-04 20:43 UTC (permalink / raw)
To: Stephen Warren
Cc: Dylan Reid, Peter De Schrijver,
linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA
[-- Attachment #1: Type: text/plain, Size: 4208 bytes --]
On Wed, Jun 04, 2014 at 01:04:05PM -0600, Stephen Warren wrote:
> On 06/04/2014 11:51 AM, Dylan Reid wrote:
> > On Wed, Jun 4, 2014 at 10:32 AM, Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org> wrote:
> >> On 05/30/2014 07:45 PM, Dylan Reid wrote:
> >>> On Fri, May 23, 2014 at 1:34 PM, Dylan Reid <dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org> wrote:
> >>>> On Fri, May 23, 2014 at 12:05 PM, Stephen Warren <swarren@wwwdotorg.org> wrote:
> >>>>> On 05/22/2014 09:55 PM, Dylan Reid wrote:
> >>>>> ...
> >>>>>>>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren@wwwdotorg.org> wrote:
> >>>>> ...
> >>>>>>>>>>>>>> Now I have the same results as Thierry; speaker-test looks like it
> >>>>>>>>>>>>>> should be working, yet I don't hear any audio from the monitor. I know
> >>>>>>>>>>>>>> the monitor works, since I've used it extensively for testing GeForce
> >>>>>>>>>>>>>> GPU HDMI audio.
> >>>>> ...
> >>>>>> Took all day, but I did get to try this.
> >>>>>>
> >>>>>> on U-Boot commit d7782d0, flashed with Stephen's u-boot flasher from github.
> >>>>>>
> >>>>>> And kernel at 81d0207 - ARM: tegra: enable HD-Audio controller in defconfig
> >>>>>> plus the addition of the hda node "okay" to the jetson-tk1 DT.
> >>>>>>
> >>>>>> I can hear the jetson's audio on the TV. This is currently a sample
> >>>>>> size of one TV, I'll set up the Quantum Data tomorrow and check that
> >>>>>> it works there as well.
> >>>>>
> >>>>> I tried that same U-Boot commit (with a few device-mode USB patches on
> >>>>> top that shouldn't affect anything since I didn't use USB device mode)
> >>>>> and have the same results.
> >>>>>
> >>>>> My monitor is a Dell U2410 with sound bar.
> >>>
> >>> I found one of these. I also didn't hear any audio. In fact I
> >>> couldn't get audio out of most monitors, TV sets worked fine.
> >>>
> >>> hdmi.c couldn't configure audio for the pixel clock generated for the
> >>> monitor. Luckily I found a patch in our downstream tree that fixes
> >>> it.
> >>>
> >>> video: tegra: Calculate HDMI audio CTS/N/AVAL values -- 7/25/11 author
> >>> swarren =)
> >>>
> >>> Thanks for fixing this for me!
> >>>
> >>> I can send this up unless you'd rather send it yourself.
> >>>
> >>> I appended the patch moved to the new location of hdmi.c:
> >>
> >> That wouldn't apply for me since all the TABs were spaces in the email.
> >> I tracked down the original and manually ported it over. You can find it
> >> in my github if you're interested:
> >>
> >> git://github.com/swarren/linux-tegra.git tegra_dev
> >>
> >> That patch does make things a bit better, but there are still some issues.
> >>
> >> When my (Venice2) system boots, the login screen is displayed at native
> >> LCD resolution, and my HDMI monitor displays a 1920x1200 chunk from the
> >> top-left of the LCD. At this point, I can play 44.1KHz audio without
> >> issue. 48KHz doesn't work due to hdmi->audio_freq not being updated, as
> >> I mentioned in my previous mail.
> >
> > Do you have a good idea how to convey this information from the audio
> > driver over to HDMI?
> > Downstream I see we have a hideous if CONFIG_TEGRA_DC in patch_hdmi.
>
> The only way of avoiding that would be to create some kind of common
> infra-structure where video drivers can register for notifications from
> arbitrary audio devices. Then, the HDA code could always call this, and
> there'd be no direct references between the two drivers. To be honest
> though, this feels like creating a whole bunch of overhead for something
> that's very simple and likely not a maintenance issue, so I'm not sure
> there's much benefit doing that.
FWIW, there's currently a discussion going on about this on alsa-devel:
[RFC] set up an sync channel between audio and display driver (i.e. ALSA and DRM)
http://mailman.alsa-project.org/pipermail/alsa-devel/2014-May/076800.html
http://mailman.alsa-project.org/pipermail/alsa-devel/2014-June/077371.html
Some of the ideas discussed there seem to be overengineered, but in
light of this thread it might be good if you guys could join in and
share some of your insights.
Thierry
[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
[not found] ` <1400553346-23238-1-git-send-email-dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2014-05-20 18:24 ` Stephen Warren
@ 2014-06-16 19:10 ` Stephen Warren
1 sibling, 0 replies; 23+ messages in thread
From: Stephen Warren @ 2014-06-16 19:10 UTC (permalink / raw)
To: Dylan Reid, linux-tegra-u79uwXL29TY76Z2rM5mHXA
Cc: devicetree-u79uwXL29TY76Z2rM5mHXA,
thierry.reding-Re5JQEeQqe8AvxtiuMwx3w
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.
^ permalink raw reply [flat|nested] 23+ messages in thread
end of thread, other threads:[~2014-06-16 19:10 UTC | newest]
Thread overview: 23+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-05-20 20:37 [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support Dylan Reid
[not found] ` <CAEUnVG7+=1ko8NL1Z3ovSpGveUADUidy4sBhU_dTUe1zbe-bMA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-20 21:55 ` Stephen Warren
[not found] ` <537BCF39.6050909-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-05-20 23:28 ` Dylan Reid
[not found] ` <CAEUnVG7CRWx5Y3RsgPzrHgYRGAx=bS4WY=w1xBUw0Ukt0DTdUQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-20 23:42 ` Stephen Warren
2014-05-21 20:55 ` Thierry Reding
2014-05-21 21:10 ` Dylan Reid
[not found] ` <CAEUnVG7tDjw_6wHc=FN0KsOKsSS1u8H8SzYdxt9VCwUsARRTeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-22 9:50 ` Thierry Reding
2014-05-22 17:48 ` Dylan Reid
[not found] ` <CAEUnVG5pNMzDoXGso+UiAWPpD0hNPhBPyGr8KfV587AuU70QPQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-22 17:54 ` Stephen Warren
[not found] ` <537E39D9.2000304-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-05-22 18:05 ` Dylan Reid
[not found] ` <CAEUnVG4UettOj0xDy+f6qEhUbhNs=u-wgyCf_Prsd2JpnmFfLw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-23 3:55 ` Dylan Reid
2014-05-23 19:05 ` Stephen Warren
[not found] ` <537F9BEB.5020102-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-05-23 20:34 ` Dylan Reid
[not found] ` <CAEUnVG63BCgpGuo+qcYkvpnVSSguBy9xwSC_AoZqCt1RgBqm7w-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-05-31 1:45 ` Dylan Reid
[not found] ` <CAEUnVG52fJhd9213fdydoUW11N0yZ31T2MZ2BPmd5Q2xkZ8XeQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-04 17:32 ` Stephen Warren
[not found] ` <538F5848.1050803-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-06-04 17:51 ` Dylan Reid
[not found] ` <CAEUnVG7bbcPjRcGxn_GAJ7rKJy4g_5veEsRnU 2yCDVWFY3ncUQ@mail.gmail.com>
[not found] ` <CAEUnVG7bbcPjRcGxn_GAJ7rKJy4g_5veEsRnU2yCDVWFY3ncUQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-06-04 19:04 ` Stephen Warren
[not found] ` <538F6DA5.9040608-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-06-04 20:43 ` Thierry Reding
2014-06-04 17:13 ` Stephen Warren
-- strict thread matches above, loose matches on Subject: below --
2014-05-20 2:35 Dylan Reid
[not found] ` <1400553346-23238-1-git-send-email-dgreid-F7+t8E8rja9g9hUCZPvPmw@public.gmane.org>
2014-05-20 18:24 ` Stephen Warren
[not found] ` <537B9DE3.5020807-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-05-20 18:31 ` Stephen Warren
2014-06-16 19:10 ` Stephen Warren
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).