From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Menzel Subject: Re: [PATCH v5] drm/i915: pass ELD to HDMI/DP audio driver Date: Wed, 02 Nov 2011 08:35:26 +0100 Message-ID: <1320219326.3560.2.camel@mattotaupa> References: <20110905011400.GB24561@localhost> <20110905123128.GA31852@localhost> <4E64C41B.5090309@pulseforce.com> <20110905124730.GB794@localhost> <4EA82DBD.9020301@pulseforce.com> <4EA9B6EF.9040305@pulseforce.com> <20111101113645.GA19100@localhost> <4EB025B0.4000900@pulseforce.com> <20111102014541.GA16026@localhost> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1621751542==" Return-path: Received: from mail.gw90.de (mail.gw90.de [188.40.100.199]) by gabe.freedesktop.org (Postfix) with ESMTP id 672689E887 for ; Wed, 2 Nov 2011 00:35:36 -0700 (PDT) Received: from f053038194.adsl.alicedsl.de ([78.53.38.194] helo=[192.168.178.21]) by mail.gw90.de with esmtpsa (TLS1.0:RSA_AES_256_CBC_SHA1:32) (Exim 4.69) (envelope-from ) id 1RLVMJ-0007kj-Kx for intel-gfx@lists.freedesktop.org; Wed, 02 Nov 2011 07:35:39 +0000 In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org --===============1621751542== Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-GIHbFqJTnHQVwz1hj+Kq" --=-GIHbFqJTnHQVwz1hj+Kq Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Dear Sander, Am Mittwoch, den 02.11.2011, 01:10 -0500 schrieb Sander Jansen: > On Tue, Nov 1, 2011 at 8:45 PM, Wu Fengguang wro= te: > >> The log does confirm that the drm_edid_to_eld function is running, and > >> that we're not far from a solution: > >> [ 21.061417] [drm:drm_edid_to_eld], ELD monitor TX-SR607 > >> [ 21.061421] [drm:drm_edid_to_eld], ELD size 13, SAD count 8 > > > > It looks all sane to this point. > > > >> As for where I am getting the EDID dump from, I am getting it from > >> /sys/devices/pci0000:00/0000:00:02.0/drm/card0/card0-HDMI-A-2/edid, > >> which provides direct virtual access to the EDID response of the > >> connected device. > >> > >> I'm completely confident that the device doesn't report too small of a > >> buffer size, and that it's completely compliant with the spec: If you > > > > Agreed. > > > >> have a Windows virtual machine (or if you're masochistic enough - a re= al > >> machine) you should download the excellent, free "Monitor Asset Manage= r" > >> by EnTech Taiwan from http://www.entechtaiwan.com/util/moninfo.shtm. I= t > >> will let you analyze EDID + ELD + extended timings, etc from an EDID > >> dump, such as the one taken above. It understands every part of EDID. > >> > >> I've put together a small archive containing my exact EDID binary dump > >> (taken from the above device path), the FULL dmesg log, as well as > >> EnTech's interpretation of the EDID dump, showing the full list of > >> supported channels, formats, etc. > >> > >> I'm guessing there is some tiny bug in your interpretation of how to > >> read ELD, maybe an incorrect 1 byte offset or something like that. > >> > >> Here's the pack: > >> http://www.pulseforce.com/node/edid_to_eld.zip > > > > Thanks! It's great tool and information! > > > >> If you do a hex analysis of my EDID dump and compare it to what the > >> edid_to_eld function is trying to do, it will probably show what's > >> wrong. I'd love to have a look at that myself but am really busy with = a > >> project over here so I can't help out other than to recompile and test > >> as fast as I can. > > > > Would you install the "intel-gpu-tools" package and run its > > intel_audio_dump utility? If not shipped with your distribution, the > > source code is also available in > > > > git://anongit.freedesktop.org/git/xorg/app/intel-gpu-tools > > > > You'll need to install packages "autotools-dev pkg-config > > libpciaccess-dev libdrm-dev libdrm-intel1" in order to build it from > > source. > > > > intel_audio_dump will dump the ELD data in the hardware buffer for use > > by the audio driver. By verifying if that data is correct, we are able > > to analyze whether and how the audio driver goes wrong. > > >=20 > I think I experience similar issues. In my case the multi-channel pcm > playback through HDMI doesn't work. Stereo and passthrough seem to > work fine though !? It's hookedup to my TV via a yamaha receiver. >=20 > I'm currently running Linux 3.1 with a G45 chipset. >=20 > libdrm 2.4.27-1 > xf86-video-intel 2.16.0-1 do you have Fengguang=E2=80=99s patch applied? This thread is about testing= that patch. > The eld seems be incorrectly parsed, though the kernel log didn't give > much info. The eld info from alsa is rather empty: >=20 > cat /proc/asound/Intel/eld#3.0 > monitor_present 1 > eld_valid 0 >=20 > Using the same Monitor Asset Manager I was able to verify that the edid f= rom > (/sys/devices/pci0000\:00/0000\:00\:02.0/drm/card0/card0-HDMI-A-1/edid > ) does seem to contain the correct information. >=20 > I've attached both the edid and the output of Monitor Asset Manager.In > addition I also run the intel_audio_dump. >=20 > Let me know if you need anymore information. It would be great if you could test the patch. Thanks, Paul --=-GIHbFqJTnHQVwz1hj+Kq Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iEYEABECAAYFAk6w8r4ACgkQPX1aK2wOHVi57QCfVaSRFdx3LiYOB4zhRyfSxORn ELEAn2AaIEhlOIz3ScSJugfVyC1BCvOG =XEEE -----END PGP SIGNATURE----- --=-GIHbFqJTnHQVwz1hj+Kq-- --===============1621751542== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============1621751542==--