On Thu, May 22, 2014 at 11:05 AM, Dylan Reid <dgreid@xxxxxxxxxxxx> wrote: > On Thu, May 22, 2014 at 10:54 AM, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote: >> On 05/22/2014 11:48 AM, Dylan Reid wrote: >>> On Thu, May 22, 2014 at 2:50 AM, Thierry Reding >>> <thierry.reding@xxxxxxxxx> 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@xxxxxxxxx> 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@xxxxxxxxxxxxx> wrote: >>>>>>>> On 05/20/2014 02:37 PM, Dylan Reid wrote: >>>>>>>>> On Tue, May 20, 2014 at 11:31 AM, Stephen Warren <swarren@xxxxxxxxxxxxx> 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. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
