Thread (4 messages) 4 messages, 2 authors, 2014-04-14

Re: [PATCH] video: Check EDID for HDMI connection

From: Tomi Valkeinen <hidden>
Date: 2014-04-14 12:53:36

On 09/04/14 00:43, Christopher Freeman wrote:

On 4/8/14 2:19 AM, "Tomi Valkeinen" [off-list ref] wrote:
quoted
* PGP Signed by an unknown key

Hi,

On 27/03/14 00:35, cfreeman@nvidia.com wrote:
quoted
From: David Ung <redacted>

Check EDID Vendor Specific Data Block bytes to see if the connection
is HDMI and set FB_MISC_HDMI.

Signed-off-by: David Ung <redacted>
Signed-off-by: Christopher Freeman<redacted>
---
Patch looks fine, but how do you use the flag? Shouldn't things like
detecting if the monitor supports audio be done with some more detailed
bits in the EDID data (not that I remember much from the HDMI spec).
The Tegra HDMI driver uses FB_MISC_HDMI to select the type of connection
it has hdmi/dvi and setup the avi info frames.
agree that we could use the EDID data to determine the audio format
support and sampling rates, but currently that's not being used.
I think determining the audio format and sample rate support could be
added in a separate and future patch.
Yep. I don't see any harm with the patch, even if it's not perfect =).
So I'm queuing it for 3.16.

 Tomi

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help