Hey! work brought this issue to my attention, so I figured I'd help speed things up and review this :) Anyway-looks fine to me Reviewed-by: Lyude Paul <lyude@xxxxxxxxxx> On Thu, 2019-10-17 at 12:42 -0700, James Ausmus wrote: > Even the AML devices that behave like CFLs can be paired with an SPT > PCH. Allow this to happen without blowing up dmesg. > > BSpec: 33665 > > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=112013 > Cc: Quanxian Wang <quanxian.wang@xxxxxxxxx> > Cc: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > Signed-off-by: James Ausmus <james.ausmus@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_pch.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_pch.c > b/drivers/gpu/drm/i915/intel_pch.c > index 1035d3d46fd8..bb1cb6f12a50 100644 > --- a/drivers/gpu/drm/i915/intel_pch.c > +++ b/drivers/gpu/drm/i915/intel_pch.c > @@ -52,7 +52,8 @@ intel_pch_type(const struct drm_i915_private *dev_priv, > unsigned short id) > return PCH_SPT; > case INTEL_PCH_SPT_LP_DEVICE_ID_TYPE: > DRM_DEBUG_KMS("Found SunrisePoint LP PCH\n"); > - WARN_ON(!IS_SKYLAKE(dev_priv) && !IS_KABYLAKE(dev_priv)); > + WARN_ON(!IS_SKYLAKE(dev_priv) && !IS_KABYLAKE(dev_priv) && > + !IS_COFFEELAKE(dev_priv)); > return PCH_SPT; > case INTEL_PCH_KBP_DEVICE_ID_TYPE: > DRM_DEBUG_KMS("Found Kaby Lake PCH (KBP)\n"); -- Cheers, Lyude Paul _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx
