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

[Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

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