From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [PATCH] drm/i915: fix gen4 digital port hotplug definitions Date: Fri, 26 Jul 2013 13:21:48 +0300 Message-ID: <87txjh62pv.fsf@intel.com> References: <1374830869-4891-1-git-send-email-daniel.vetter@ffwll.ch> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id 49AFCE63FF for ; Fri, 26 Jul 2013 03:20:19 -0700 (PDT) In-Reply-To: <1374830869-4891-1-git-send-email-daniel.vetter@ffwll.ch> 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: Intel Graphics Development Cc: Daniel Vetter , Jan Niggemann , Egbert Eich List-Id: intel-gfx@lists.freedesktop.org On Fri, 26 Jul 2013, Daniel Vetter wrote: > Apparently Bspec is wrong in this case here even for gm45. Note that > Bspec is horribly misguided on i965g/gm, so we don't have any other > data points besides that it seems to make machines work better. > > With this changes all the bits in PORT_HOTPLUG_STAT for the digital > ports are ordered the same way. This seems to agree with what register > dumps from the hpd storm handling code shows, where the LIVE bit and > the short/long pulse STATUS bits light up at the same time with this > enumeration (but no with the one from Bspec). Would a comment about this near the #defines be in order? To avoid the "these values are all wrong per bspec" patches. Cheers, Jani. > Also tested on my gm45 which has two DP+ ports, and everything seems > to still work as expected. > > References: http://www.mail-archive.com/intel-gfx@lists.freedesktop.org/msg23054.html > Cc: Egbert Eich > Cc: Jan Niggemann > Tested-by: Jan Niggemann > Signed-off-by: Daniel Vetter > --- > drivers/gpu/drm/i915/i915_reg.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/gpu/drm/i915/i915_reg.h b/drivers/gpu/drm/i915/i915_reg.h > index 6caa748..2d4c884 100644 > --- a/drivers/gpu/drm/i915/i915_reg.h > +++ b/drivers/gpu/drm/i915/i915_reg.h > @@ -1925,9 +1925,9 @@ > > #define PORT_HOTPLUG_STAT (dev_priv->info->display_mmio_offset + 0x61114) > /* HDMI/DP bits are gen4+ */ > -#define PORTB_HOTPLUG_LIVE_STATUS (1 << 29) > +#define PORTD_HOTPLUG_LIVE_STATUS (1 << 29) > #define PORTC_HOTPLUG_LIVE_STATUS (1 << 28) > -#define PORTD_HOTPLUG_LIVE_STATUS (1 << 27) > +#define PORTB_HOTPLUG_LIVE_STATUS (1 << 27) > #define PORTD_HOTPLUG_INT_STATUS (3 << 21) > #define PORTC_HOTPLUG_INT_STATUS (3 << 19) > #define PORTB_HOTPLUG_INT_STATUS (3 << 17) > -- > 1.8.3.2 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Jani Nikula, Intel Open Source Technology Center