On 2010.09.19 08:38:07 +0100, Chris Wilson wrote: > > This should be cc'ed for Adam Jackson's attention as well. yep, I think I cc-ed ajax in git-send-mail command line...I'll amend the commit. > > +bool drm_detect_monitor_audio(struct edid *edid) > > drm_edid_has_monitor_audio()? That is a little more self-descriptive. > (I also think drm_detect_hdmi_monitor is poorly named.) yeah, that looks better. thanks. > The last time Adam had a chance to comment on this EDID check, he > suggested that we cannot rely on has_audio being a reliable indicator of > the sink's properties. If the EDID has no audio support, then return > early, otherwise perform the secondary check that extension block contains > audio data. That's also the problem we stuck for quite some time, and until recently we got reply on this issue. Adam is right, 'basic audio' bit might not be reliable, so we try to search audio block too, if there's any existence, we will accept 'basic audio' support as well. Thanks for the review! -- Open Source Technology Center, Intel ltd. $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827
Attachment:
signature.asc
Description: Digital signature
_______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
