Re: [PATCH 1/2] ARM: tegra: Add Tegra124 HDA support
From: Dylan Reid <hidden>
Date: 2014-05-23 03:55:54
Also in:
linux-devicetree
On Thu, May 22, 2014 at 11:05 AM, Dylan Reid [off-list ref] wrote:
On Thu, May 22, 2014 at 10:54 AM, Stephen Warren [off-list ref] wrote:quoted
On 05/22/2014 11:48 AM, Dylan Reid wrote:quoted
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.799791Thanks 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' onDylan, 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.