On 09/04/14 00:43, Christopher Freeman wrote: > > > On 4/8/14 2:19 AM, "Tomi Valkeinen" <tomi.valkeinen@xxxxxx> wrote: > >> * PGP Signed by an unknown key >> >> Hi, >> >> On 27/03/14 00:35, cfreeman@xxxxxxxxxx wrote: >>> From: David Ung <davidu@xxxxxxxxxx> >>> >>> Check EDID Vendor Specific Data Block bytes to see if the connection >>> is HDMI and set FB_MISC_HDMI. >>> >>> Signed-off-by: David Ung <davidu@xxxxxxxxxx> >>> Signed-off-by: Christopher Freeman<cfreeman@xxxxxxxxxx> >>> --- >> >> 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
Attachment:
signature.asc
Description: OpenPGP digital signature
