On 05/22/2014 11:48 AM, Dylan Reid wrote:

On Thu, May 22, 2014 at 2:50 AM, Thierry Reding
[off-list ref] wrote:

quoted

On Wed, May 21, 2014 at 02:10:09PM -0700, Dylan Reid wrote:

quoted

On Wed, May 21, 2014 at 1:55 PM, Thierry Reding
[off-list ref] wrote:

quoted

On Tue, May 20, 2014 at 04:28:45PM -0700, Dylan Reid wrote:

quoted

On Tue, May 20, 2014 at 2:55 PM, Stephen Warren [off-list ref] wrote:

quoted

On 05/20/2014 02:37 PM, Dylan Reid wrote:

quoted

On Tue, May 20, 2014 at 11:31 AM, Stephen Warren [off-list ref] wrote:

quoted

On 05/20/2014 12:24 PM, Stephen Warren wrote:

quoted

On 05/19/2014 08:35 PM, Dylan Reid wrote:

quoted

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

...

quoted

quoted

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.