On Tue, Aug 29, 2017 at 04:22:29PM -0700, Rodrigo Vivi wrote: > Let's get a proper HDMI DDI entry level for vswing programming > sequences on CNL. > > Spec doesn't specify any default for HDMI tables, > so let's pick the last entry as the default for now. > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@xxxxxxxxx> > --- > drivers/gpu/drm/i915/intel_ddi.c | 5 ++++- > 1 file changed, 4 insertions(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/i915/intel_ddi.c b/drivers/gpu/drm/i915/intel_ddi.c > index 3ce02cbd4483..f1757a8e481a 100644 > --- a/drivers/gpu/drm/i915/intel_ddi.c > +++ b/drivers/gpu/drm/i915/intel_ddi.c > @@ -660,7 +660,10 @@ static int intel_ddi_hdmi_level(struct drm_i915_private *dev_priv, enum port por > if (IS_GEN9_LP(dev_priv)) > return hdmi_level; > > - if (IS_GEN9_BC(dev_priv)) { > + if (IS_CANNONLAKE(dev_priv)) { > + cnl_get_buf_trans_hdmi(dev_priv, &n_hdmi_entries); > + hdmi_default_entry = n_hdmi_entries - 1; Hmm. I guess we might try to do the same thing for BXT, for extra consistency. But that's a separate issue. Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> > + } else if (IS_GEN9_BC(dev_priv)) { > skl_get_buf_trans_hdmi(dev_priv, &n_hdmi_entries); > hdmi_default_entry = 8; > } else if (IS_BROADWELL(dev_priv)) { > -- > 2.13.2 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel OTC _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx