From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 4/4] drm/i915: don't try to retrain a DP link on an inactive CRTC Date: Tue, 12 Aug 2014 15:18:33 +0300 Message-ID: <20140812121833.GX4193@intel.com> References: <1407783891-9406-1-git-send-email-imre.deak@intel.com> <1407783891-9406-2-git-send-email-imre.deak@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by gabe.freedesktop.org (Postfix) with ESMTP id D033B6E059 for ; Tue, 12 Aug 2014 05:18:36 -0700 (PDT) Content-Disposition: inline In-Reply-To: <1407783891-9406-2-git-send-email-imre.deak@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Imre Deak Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Mon, Aug 11, 2014 at 10:04:51PM +0300, Imre Deak wrote: > Atm we may retrain the DP link even if the CRTC is inactive through > HPD work->intel_dp_check_link_status(). This in turn can lock up the PHY > (at least on BYT), since the DP port is disabled. > = > Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=3D81948 > Signed-off-by: Imre Deak > --- > drivers/gpu/drm/i915/intel_dp.c | 3 +++ > 1 file changed, 3 insertions(+) > = > diff --git a/drivers/gpu/drm/i915/intel_dp.c b/drivers/gpu/drm/i915/intel= _dp.c > index d7f5d0a..49de9be 100644 > --- a/drivers/gpu/drm/i915/intel_dp.c > +++ b/drivers/gpu/drm/i915/intel_dp.c > @@ -3551,6 +3551,9 @@ intel_dp_check_link_status(struct intel_dp *intel_d= p) > if (WARN_ON(!intel_encoder->base.crtc)) > return; > = > + if (!to_intel_crtc(intel_encoder->base.crtc)->active) > + return; Yeah the connectors_active check isn't enough since we don't clear that when we disable the crtcs for system suspend. Maybe we should do that instead? Anyway this check seems fine to me regardless of how we deal with connectors_active during system suspend, so: Reviewed-by: Ville Syrj=E4l=E4 > + > /* Try to read receiver status if the link appears to be up */ > if (!intel_dp_get_link_status(intel_dp, link_status)) { > return; > -- = > 1.8.4 > = > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@lists.freedesktop.org > http://lists.freedesktop.org/mailman/listinfo/intel-gfx -- = Ville Syrj=E4l=E4 Intel OTC