On Tue, 03 Feb 2015, gregkh@xxxxxxxxxxxxxxxxxxx wrote: > The patch below does not apply to the 3.18-stable tree. > If someone wants it applied there, or to any other stable or longterm > tree, then please email the backport, including the original git commit > id to <stable@xxxxxxxxxxxxxxx>. Rodrigo, can you do a backport for 3.18 please? BR, Jani. > > thanks, > > greg k-h > > ------------------ original commit in Linus's tree ------------------ > > From a35cc9d0c0118fb18f7c5dd7a44adb454868a679 Mon Sep 17 00:00:00 2001 > From: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > Date: Wed, 21 Jan 2015 10:33:53 -0800 > Subject: [PATCH] drm/i915: Fix and clean BDW PCH identification > > It seems in the past we have BDW with PCH not been propperly identified > and we force it to be LPT and we were warning !IS_HASWELL on propper identification. > > Now that products are out there we are receiveing logs with this incorrect WARN. > And also according to local tests on all production BDW here ULT or HALO we don't > need this force anymore. So let's clean this block for real. > > v2: Fix LPT_LP WARNs to avoid wrong warns on BDW_ULT (By Jani). > > Reference: https://bugs.freedesktop.org/attachment.cgi?id=110972 > > Cc: Jani Nikula <jani.nikula@xxxxxxxxx> > Cc: Paulo Zanoni <paulo.r.zanoni@xxxxxxxxx> > Cc: Xion Zhang <xiong.y.zhang@xxxxxxxxx> > Cc: stable@xxxxxxxxxxxxxxx > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > Signed-off-by: Jani Nikula <jani.nikula@xxxxxxxxx> > > diff --git a/drivers/gpu/drm/i915/i915_drv.c b/drivers/gpu/drm/i915/i915_drv.c > index 574057cd1d09..7643300828c3 100644 > --- a/drivers/gpu/drm/i915/i915_drv.c > +++ b/drivers/gpu/drm/i915/i915_drv.c > @@ -462,19 +462,13 @@ void intel_detect_pch(struct drm_device *dev) > } else if (id == INTEL_PCH_LPT_DEVICE_ID_TYPE) { > dev_priv->pch_type = PCH_LPT; > DRM_DEBUG_KMS("Found LynxPoint PCH\n"); > - WARN_ON(!IS_HASWELL(dev)); > - WARN_ON(IS_HSW_ULT(dev)); > - } else if (IS_BROADWELL(dev)) { > - dev_priv->pch_type = PCH_LPT; > - dev_priv->pch_id = > - INTEL_PCH_LPT_LP_DEVICE_ID_TYPE; > - DRM_DEBUG_KMS("This is Broadwell, assuming " > - "LynxPoint LP PCH\n"); > + WARN_ON(!IS_HASWELL(dev) && !IS_BROADWELL(dev)); > + WARN_ON(IS_HSW_ULT(dev) || IS_BDW_ULT(dev)); > } else if (id == INTEL_PCH_LPT_LP_DEVICE_ID_TYPE) { > dev_priv->pch_type = PCH_LPT; > DRM_DEBUG_KMS("Found LynxPoint LP PCH\n"); > - WARN_ON(!IS_HASWELL(dev)); > - WARN_ON(!IS_HSW_ULT(dev)); > + WARN_ON(!IS_HASWELL(dev) && !IS_BROADWELL(dev)); > + WARN_ON(!IS_HSW_ULT(dev) && !IS_BDW_ULT(dev)); > } else if (id == INTEL_PCH_SPT_DEVICE_ID_TYPE) { > dev_priv->pch_type = PCH_SPT; > DRM_DEBUG_KMS("Found SunrisePoint PCH\n"); > -- Jani Nikula, Intel Open Source Technology Center -- To unsubscribe from this list: send the line "unsubscribe stable" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html