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 linux-tegra" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
