[Intel-gfx] [Resend][PATCH 4/4] drm/i915: detect hdmi monitor by reading edid

[Eric Anholt]

On Fri, 2009-07-03 at 23:29 +0800, ling.ma at intel.com wrote:
> We detect HDMI output status by setting interrupt enable bit,
> then read live status, which is effective for most cases.
> However some G4x platform will indicate incorrect detection result,
> because according to the HW team live status detection need special board
> design and won't work on all boards, even if current output is HDMI
> monitor. The patch intends to read edid, then determin connect status
> by EDID type.
> 
> It fixed freedesktop.org bug #21322.
> 
> Signed-off-by: Ma Ling <ling.ma at intel.com>

If PORT_HOTPLUG_STAT is unreliable, why continue using it?  Why not just
probe EDID and use that to base connection detection on?  This code can
only support digital monitors anyway, right?  If so, then we could rely
on EDID presence and the digital bit.

-- 
Eric Anholt
eric at anholt.net                         eric.anholt at intel.com


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
URL: <http://lists.freedesktop.org/archives/intel-gfx/attachments/20090707/9b2e4c7a/attachment.sig>