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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




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.
--
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




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux