From mboxrd@z Thu Jan 1 00:00:00 1970 From: Imre Deak Subject: Re: [PATCH] drm/i915: vlv: fix DP PHY lockup due to invalid PP sequencer setup Date: Thu, 30 Jan 2014 18:48:35 +0200 Message-ID: <1391100515.26112.6.camel@intelbox> References: <1391093442-16520-1-git-send-email-imre.deak@intel.com> <20140130155224.GU17001@phenom.ffwll.local> <87eh3p8moj.fsf@intel.com> Reply-To: imre.deak@intel.com Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0838061575==" Return-path: Received: from mga02.intel.com (mga02.intel.com [134.134.136.20]) by gabe.freedesktop.org (Postfix) with ESMTP id E344E101DFC for ; Thu, 30 Jan 2014 08:51:48 -0800 (PST) In-Reply-To: <87eh3p8moj.fsf@intel.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces@lists.freedesktop.org Errors-To: intel-gfx-bounces@lists.freedesktop.org To: Jani Nikula Cc: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org --===============0838061575== Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-HiFjlKHGEGNYH7E/T487" --=-HiFjlKHGEGNYH7E/T487 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Thu, 2014-01-30 at 18:22 +0200, Jani Nikula wrote: > On Thu, 30 Jan 2014, Daniel Vetter wrote: > > On Thu, Jan 30, 2014 at 04:50:42PM +0200, Imre Deak wrote: > >> Atm we setup the HW panel power sequencer logic both for eDP and DP > >> ports. On eDP we then go on and start the power on sequence and commen= ce > >> with link training when it's ready. On DP we don't do the power on > >> sequencing but do the link training immediately. At this point the DP > >> PHY block gets stuck, since - supposedly - it is waiting for the power > >> on sequence to finish. The actual register write that seems to hold of= f > >> the PHY is PIPEX_PP_ON_DELAYS[Panel Control Port Select]. Writing here > >> a non-0 value eventually sets PIPEX_PP_STATUS[Require Asset Status] to > >> 1 and blocks the PHY until the panel power on is ready. > >>=20 > >> Fix this by not doing any PP sequencing setup for DP ports. > >>=20 > >> Thanks to Ville Syrj=C3=A4l=C3=A4, Jesse Barnes and Todd Previte for t= he help in > >> tracking this down. > >>=20 > >> Signed-off-by: Imre Deak > > > > Ah, the infamous ABCD hack we're using all over the place in intel_lvds= .c. > > On edp we didn't have a need for it thus far since the "require asset > > status" checks have all been fused of, with the PP being on the PCH and > > the edp port on the north display block. If this is really all we need = to > > appease the hardware then I'm heavily in favour of it as opposed to > > resurrect the ABCD hack for intel_dp.c. > > > > One thing though: Should we add a check for the "Required Asset Status" > > bit somewhere? I don't really have a good idea for a spot to put this > > into, hence the question. >=20 > Don't know about the asset status stuff, but I know it was me who > screwed this up in >=20 > commit bf13e81b904a37d94d83dd6c3b53a147719a3ead > Author: Jani Nikula > Date: Fri Sep 6 07:40:05 2013 +0300 >=20 > drm/i915: add support for per-pipe power sequencing on vlv >=20 > We need to make sure the PP registers are set up correctly on the pipe > being enabled, which might be different from the last time. But only for > eDP. >=20 > I'm a bit surprised this hasn't been bisected to. One thing making that more difficult is buggy VBTs. If that reports the DP port being eDP, like in my case, you won't see the problem. --Imre --=-HiFjlKHGEGNYH7E/T487 Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) iQEcBAABAgAGBQJS6oJjAAoJEORIIAnNuWDF05AIALG5v8vFJLlZL4mHXFLgpvvD uP5t4Mo5c/VVqhVnbfBihaSCXt4NIiGF6cGjJQUFdSZqeYAJE+XDj3oCoba99/r7 rQ5zDQk2dyRiRszu6JXoIBlqe3zGTdD/aWwug4eo43GgbQHlYWxntJ3p1DhtzjBh EhVH7yKOa/4PEGzGVjRgo36R+WqftZfi5UEOcGFkvgt5tFocP1ynhqHbPA2vbaj9 AqJoRXGs9XlYMmUMwLTlBZQyI4tOVKz90douV5LZiRCdjh2zaNAbBRMjT3Dfm5CW wXpoV0dlQpFuikzK5JxKfbml+r2i99QhRFJKRXY7RXJlsBMFhj5tapFAVUc2a10= =bPdL -----END PGP SIGNATURE----- --=-HiFjlKHGEGNYH7E/T487-- --===============0838061575== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx --===============0838061575==--