From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keith Packard Subject: Re: Regression: Borked display on second output with Intel G45 in 3.0-rc2 Date: Mon, 06 Jun 2011 13:21:53 -0700 Message-ID: References: <20110606171253.GA2925@elliptictech.com> <201106061920.07144@rk-nord.at> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============1434177522==" Return-path: Received: from keithp.com (home.keithp.com [63.227.221.253]) by gabe.freedesktop.org (Postfix) with ESMTP id 808EA9E79D for ; Mon, 6 Jun 2011 13:21:59 -0700 (PDT) In-Reply-To: <201106061920.07144@rk-nord.at> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org Errors-To: dri-devel-bounces+sf-dri-devel=m.gmane.org@lists.freedesktop.org To: Melchior FRANZ , linux-kernel@vger.kernel.org Cc: Nick Bowler , Linus Torvalds , dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============1434177522== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha1; protocol="application/pgp-signature" --=-=-= Content-Transfer-Encoding: quoted-printable On Mon, 6 Jun 2011 19:20:06 +0200, Melchior FRANZ wrote: > That's apparently the bug that I've submitted a patch for on 2011/5/31: > https://lkml.org/lkml/2011/5/31/393 > I assume/hope it's still in someone's queue. Yeah, we "shouldn't" need to call intel_enable_plane from i9xx_crtc_mode_set as it is called immediately afterwards from i9xx_crtc_enable. In any case, there's a bogus call in ironlake_crtc_mode_set which clearly belongs in i9xx_crtc_mode_set: diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/in= tel_display.c index 81a9059..aa43e7b 100644 =2D-- a/drivers/gpu/drm/i915/intel_display.c +++ b/drivers/gpu/drm/i915/intel_display.c @@ -4687,6 +4687,7 @@ static int i9xx_crtc_mode_set(struct drm_crtc *crtc, =20 I915_WRITE(DSPCNTR(plane), dspcntr); POSTING_READ(DSPCNTR(plane)); + intel_enable_plane(dev_priv, plane, pipe); =20 ret =3D intel_pipe_set_base(crtc, x, y, old_fb); =20 @@ -5217,8 +5218,6 @@ static int ironlake_crtc_mode_set(struct drm_crtc *cr= tc, =20 I915_WRITE(DSPCNTR(plane), dspcntr); POSTING_READ(DSPCNTR(plane)); =2D if (!HAS_PCH_SPLIT(dev)) =2D intel_enable_plane(dev_priv, plane, pipe); =20 ret =3D intel_pipe_set_base(crtc, x, y, old_fb); =20 We need to figure out why this call (in i9xx_crtc_mode_set) is required, but that will require finding hardware that reproduces the bug and fixing it there. =2D-=20 keith.packard@intel.com --=-=-= Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iD8DBQFN7TbhQp8BWwlsTdMRAiqQAJ40Tr4L6/tmo4ZuU3LVsgXKNUIbtACdGugo jz46IpDmn9W4nes7VP/09hc= =bZY2 -----END PGP SIGNATURE----- --=-=-=-- --===============1434177522== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/dri-devel --===============1434177522==--