From mboxrd@z Thu Jan 1 00:00:00 1970 From: Daniel Vetter Subject: Re: [PATCH] drm/i915/crt: Do not rely upon the HPD presence pin Date: Mon, 11 Jun 2012 21:00:44 +0200 Message-ID: <20120611190044.GH4695@phenom.ffwll.local> References: <1339407996_5899@CP5-2952> <1339425520-12637-1-git-send-email-daniel.vetter@ffwll.ch> <1339425754_7427@CP5-2952> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wg0-f43.google.com (mail-wg0-f43.google.com [74.125.82.43]) by gabe.freedesktop.org (Postfix) with ESMTP id C6D569E88D for ; Mon, 11 Jun 2012 11:59:14 -0700 (PDT) Received: by wgbdr1 with SMTP id dr1so2800519wgb.12 for ; Mon, 11 Jun 2012 11:59:14 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1339425754_7427@CP5-2952> 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: Chris Wilson Cc: Daniel Vetter , Intel Graphics Development List-Id: intel-gfx@lists.freedesktop.org On Mon, Jun 11, 2012 at 03:42:32PM +0100, Chris Wilson wrote: > On Mon, 11 Jun 2012 16:38:40 +0200, Daniel Vetter wrote: > > From: Chris Wilson > > > > VGA hotplug detection "works" by measuring the resistance across > > certain pins. A lot of kvm switches fumble this and wire up cheap > > resistors with the wrong resistance or don't bother at all. > > > > To accomodate these, also try to detect a connected monitor by trying > > to grab the edid. Contrary to !HAS_HOTPLUG platforms we don't bother > > with an actual load-detection cycle when the output is life - that > > would be actual work to implement because things moved around. This is > > the big difference to Chris Wilson's original approach: > > > > commit 9e612a008fa7fe493a473454def56aa321479495 > > Author: Chris Wilson > > Date: Thu May 31 13:08:53 2012 +0100 > > > > drm/i915/crt: Do not rely upon the HPD presence pin > > > > This blew up on Linus' machine because it errornously detected a vga > > screen (without and edid and hence only the default modes), leading to > > it's prompt removal: > > > > commit 8f53369b753f5f4c7684c2eb0b592152abb1dd00 > > Author: Linus Torvalds > > Date: Fri Jun 8 14:53:06 2012 -0700 > > > > Revert "drm/i915/crt: Do not rely upon the HPD presence pin" > > > > Some digging around in Bspec shows the reason why load detect doesn't work on > > newer chips - the legacy VGA load detect bit isn't wired up any longer: > > > > Public Snb Bspec, Vol3 Part1, 1.1.1 ST00 Input Status 0, bit4: > > > > "RGB Comparator / Sense. This bit is here for compatibility and will > > always return one. Monitor detection must be done be done through the > > programming of registers in the MMIO space. > > 0 = Below threshold > > 1 = Above threshold" > > > > v2: Add a comment in the code that load detect on hotplug capable > > machines is broken and pimp the commit message with a quote of Bspec > > to show why. > > > > Reported-and-tested-by: Matthieu LAVIE > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=50501 > > Signed-off-by: Chris Wilson > > Just missing your s-o-b, and you can add my reviewed-by to your > amendment. :) Fixed and applied to -fixes, thanks for the review (and kickstarting this with your original patch). -Daniel -- Daniel Vetter Mail: daniel@ffwll.ch Mobile: +41 (0)79 365 57 48