From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Anholt Subject: Re: [PATCH 1/7] drm/vc4: hdmi: Check that the monitor supports HDMI audio Date: Mon, 04 Mar 2019 13:09:19 -0800 Message-ID: <877edeuz28.fsf@anholt.net> References: <4f17bf31-c5ba-286d-f08e-97b557fb5b43@i2se.com> <87sgw21ok9.fsf@anholt.net> <1185446515.301320.1551729991062@email.ionos.de> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0103018991==" Return-path: Received: from anholt.net (anholt.net [50.246.234.109]) by gabe.freedesktop.org (Postfix) with ESMTP id DB97189D87 for ; Mon, 4 Mar 2019 21:09:22 +0000 (UTC) In-Reply-To: <1185446515.301320.1551729991062@email.ionos.de> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: Stefan Wahren , Maxime Ripard , Maarten Lankhorst , Sean Paul , Daniel Vetter , David Airlie Cc: Paul Kocialkowski , linux-arm-kernel@lists.infradead.org, Thomas Petazzoni , dri-devel@lists.freedesktop.org, eben@raspberrypi.org List-Id: dri-devel@lists.freedesktop.org --===============0103018991== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" --=-=-= Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Stefan Wahren writes: >> Eric Anholt hat am 4. M=C3=A4rz 2019 um 19:28 geschrie= ben: >>=20 >>=20 >> Stefan Wahren writes: >>=20 >> > Hi Maxime, >> > >> > Am 04.03.2019 um 15:52 schrieb Maxime Ripard: >> >> The current code assumes as soon as the device is an HDMI one that it >> >> supports an audio sink. However, strictly speaking, this is exposed a= s a >> >> separate part of EDID. >> >> >> >> This can be checked through the drm_detect_monitor_audio function, so= let's >> >> use it and make sure that we can use the HDMI monitor as an output be= fore >> >> sending sound. >> > >> > does the audio output work in the following setup after applying this = patch? >> > >> > VC4 --- HDMI Audio extractor --- Non audio capable monitor >>=20 >> A 1-minute google of audio extractors says they manage the EDID. Do you >> have some reason to think this wouldn't work? > > My only concern is the existence of some audio extractors which doesn't c= are about EDID. I that case we need to provide some kind of force switch. I'm having a hard time imagining such a product existing, given that I don't think Windows will have "ignore the EDID, force audio anyway" switches in their UIs. --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE/JuuFDWp9/ZkuCBXtdYpNtH8nugFAlx9k/8ACgkQtdYpNtH8 nuijew//a/cS5fBNvG/2+9Qr+cUdZ597e3Ar1ZLbR3zEIRhLpIVDmu6OgSGrKpQw 6onkAu4AiL7xNOpE3svjRb2oVIcXUr28klXde1lRGWSX52dZ7xjwLR6qbQc8a84t 6SGtaeFi6OmU5qPBiiADbUhPA1qAMil7MMTFHWSLmFbf2zwpnLvLmN6tulw6sc7H DDpqKJi0hZDNupSxdEqAB1hvVz0RaarfMlBI6Tz66+YJilt+8oQL8NIf31rVw5YQ Rv2EYdm7V0pNOb3j2H9ZtSFWzG76JL5AbEufGh3+v4I861EphOsnG0nzb1SxvDQg k2mgC+RF+6lG+/MZTn5d/rekuioCwpnKElsoQcfEKdwA7Iq6B87WiV90P6H2F0YP gaw7kQJUYEUphqvF3pAOoe81BFHm01ew/m4MFlWL7/YrBK/KA0yrvA/BkQDRn/So 4sOQjUQkgUox/GMRaJpQBOU4vfhQaPRsr7uwZVd0RGpGt947uIrCp9i4+vcZ4aWz /K9fCoJ+jHNIGprwMiuQytv9Z0nb2bniI0WJtNgGb1HL4r09pUWF1iMPf3i508iy tmVzqqrYl3AuomfzcTHnL2Me4X2mxooRynYcijpCiVjvNx4DYbZMZ1bHwXD7Jpgz PNKYzGWf6zbLwnNwxR4fcsIhFlJ/7aS+bXQJ0quPzGhbgxSwyxw= =K6go -----END PGP SIGNATURE----- --=-=-=-- --===============0103018991== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs --===============0103018991==--