From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jani Nikula Subject: Re: [Intel-gfx] [PATCH 1/2] drm/i915: Disallow interlaced modes on g4x DP outputs Date: Thu, 14 Jun 2018 15:26:09 +0300 Message-ID: <87k1r1zh9a.fsf@intel.com> References: <20180613160553.11664-1-ville.syrjala@linux.intel.com> <87o9gdzha5.fsf@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8BIT Return-path: In-Reply-To: <87o9gdzha5.fsf@intel.com> Sender: stable-owner@vger.kernel.org To: Ville Syrjala , intel-gfx@lists.freedesktop.org Cc: stable@vger.kernel.org List-Id: intel-gfx@lists.freedesktop.org On Thu, 14 Jun 2018, Jani Nikula wrote: > On Wed, 13 Jun 2018, Ville Syrjala wrote: >> From: Ville Syrjälä >> >> Looks like interlaced DP output doesn't work on g4x either. Not all >> that surprising considering we already established that interlaced >> DP output is busted on VLV/CHV. >> >> Cc: stable@vger.kernel.org >> Signed-off-by: Ville Syrjälä > > Reviewed-by: Jani Nikula Oh, did you have a bug reference? > > >> --- >> drivers/gpu/drm/i915/intel_dp.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel_dp.c >> index 40ffd9163175..6068986fd985 100644 >> --- a/drivers/gpu/drm/i915/intel_dp.c >> +++ b/drivers/gpu/drm/i915/intel_dp.c >> @@ -1869,7 +1869,7 @@ intel_dp_compute_config(struct intel_encoder *encoder, >> conn_state->scaling_mode); >> } >> >> - if ((IS_VALLEYVIEW(dev_priv) || IS_CHERRYVIEW(dev_priv)) && >> + if (HAS_GMCH_DISPLAY(dev_priv) && >> adjusted_mode->flags & DRM_MODE_FLAG_INTERLACE) >> return false; >> >> @@ -6351,7 +6351,7 @@ intel_dp_init_connector(struct intel_digital_port *intel_dig_port, >> drm_connector_init(dev, connector, &intel_dp_connector_funcs, type); >> drm_connector_helper_add(connector, &intel_dp_connector_helper_funcs); >> >> - if (!IS_VALLEYVIEW(dev_priv) && !IS_CHERRYVIEW(dev_priv)) >> + if (!HAS_GMCH_DISPLAY(dev_priv)) >> connector->interlace_allowed = true; >> connector->doublescan_allowed = 0; -- Jani Nikula, Intel Open Source Graphics Center