From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chris Wilson Subject: Re: [PATCH] drm/i915/crt: Do not rely upon the HPD presence pin Date: Fri, 08 Jun 2012 23:23:10 +0100 Message-ID: <1339194247_596313@CP5-2952> References: <1338466133-18386-1-git-send-email-chris@chris-wilson.co.uk> <20120608222212.GI5761@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from fireflyinternet.com (smtp.fireflyinternet.com [109.228.6.236]) by gabe.freedesktop.org (Postfix) with ESMTP id F0918A0C19 for ; Fri, 8 Jun 2012 15:24:09 -0700 (PDT) In-Reply-To: <20120608222212.GI5761@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Daniel Vetter Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Sat, 9 Jun 2012 00:22:12 +0200, Daniel Vetter wrote: > On Thu, May 31, 2012 at 01:08:53PM +0100, Chris Wilson wrote: > > Whilst most monitors do wire up the HPD presence pin, it seems quite a > > few KVM do not. Therefore if we simply rely on the HPD pin being > > asserted to indicate a connected monitor we fail miserable, so fall back > > to performing a DCC query for the EDID. > > > > Reported-and-tested-by: Matthieu LAVIE > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=50501 > > Signed-off-by: Chris Wilson > > Ok, this blew up ... Can you please resend, with Dave's suggestion for a > rectified commit message & comment and with a check added such that we > don't try to do load_detect on HAS_HOTPLUG machines - I guess it doesn't > work too well. I disagree, if we cannot trust the hw autodetection, then we know that there are monitors/kvm that do not report an EDID and so we need to do the whole shebang. Which will continue to annoy Linus since his machine is behaving as expected given the circumstances. -Chris -- Chris Wilson, Intel Open Source Technology Centre