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

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

 




On Wed, Jun 4, 2014 at 10:32 AM, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote:
> On 05/30/2014 07:45 PM, Dylan Reid wrote:
>> On Fri, May 23, 2014 at 1:34 PM, Dylan Reid <dgreid@xxxxxxxxxxxx> wrote:
>>> On Fri, May 23, 2014 at 12:05 PM, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote:
>>>> On 05/22/2014 09:55 PM, Dylan Reid wrote:
>>>> ...
>>>>>>>>>>>> On Tue, May 20, 2014 at 2:55 PM, Stephen Warren <swarren@xxxxxxxxxxxxx> wrote:
>>>> ...
>>>>>>>>>>>>> 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.
>>>> ...
>>>>> 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.
>>>>
>>>> I tried that same U-Boot commit (with a few device-mode USB patches on
>>>> top that shouldn't affect anything since I didn't use USB device mode)
>>>> and have the same results.
>>>>
>>>> My monitor is a Dell U2410 with sound bar.
>>
>> I found one of these.  I also didn't hear any audio.  In fact I
>> couldn't get audio out of most monitors, TV sets worked fine.
>>
>> hdmi.c couldn't configure audio for the pixel clock generated for the
>> monitor.  Luckily I found a patch in our downstream tree that fixes
>> it.
>>
>> video: tegra: Calculate HDMI audio CTS/N/AVAL values -- 7/25/11 author
>> swarren =)
>>
>> Thanks for fixing this for me!
>>
>> I can send this up unless you'd rather send it yourself.
>>
>> I appended the patch moved to the new location of hdmi.c:
>
> That wouldn't apply for me since all the TABs were spaces in the email.
> I tracked down the original and manually ported it over. You can find it
> in my github if you're interested:
>
> git://github.com/swarren/linux-tegra.git tegra_dev
>
> That patch does make things a bit better, but there are still some issues.
>
> When my (Venice2) system boots, the login screen is displayed at native
> LCD resolution, and my HDMI monitor displays a 1920x1200 chunk from the
> top-left of the LCD. At this point, I can play 44.1KHz audio without
> issue. 48KHz doesn't work due to hdmi->audio_freq not being updated, as
> I mentioned in my previous mail.

Do you have a good idea how to convey this information from the audio
driver over to HDMI?
Downstream I see we have a hideous if CONFIG_TEGRA_DC in patch_hdmi.

>
> However, once I log in, the window manager changes the HDMI output to
> 1024x768@75Hz right-of the LCD panel. I have no idea why it does this,
> but still... Without the audio ACR value calculation patch, the HDMI
> video output fine at this point. However, with the patch, the monitor
> won't sync to the signal; it keeps entering/leaving power-save mode.
>
> I suspect this is simply because without the patch, I see the following
> error when setting up HDMI for the login screen:
>
> tegra-hdmi 54280000.hdmi: cannot set audio to 44100 at 154000000 pclk
>
> ... and when that happens, tegra_output_hdmi_enable() sets hdmi->dvi =
> true, which disables a whole bunch of code. However, with the patch
> applied, the audio setup doesn't fail, so all that infoframe/... code
> runs now, and I suppose there are some issues with it.
>
> Are you able to look into this, or do you need me to investigate further?

I'm travelling for the rest of the week, but I can set aside time to
look at it on Monday.  If you make any more progress before then let
me know.

Thanks for spending the time looking at this.

Dylan

>
> I also couldn't persuade xrandr to switch back to a sane resolution on
> the HDMI port to see if that resolved the issue. I suppose that's
> probably unrelated to the audio code.
--
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