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

[Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>]

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