From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ville =?iso-8859-1?Q?Syrj=E4l=E4?= Subject: Re: [PATCH 3/3] drm/i915: Try harder to disable trickle feed on VLV Date: Tue, 4 Jun 2013 18:55:16 +0300 Message-ID: <20130604155516.GE5004@intel.com> References: <1369139314-3003-1-git-send-email-ville.syrjala@linux.intel.com> <1369139314-3003-4-git-send-email-ville.syrjala@linux.intel.com> <20130604141912.GD2052@cantiga.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: Received: from mga14.intel.com (mga14.intel.com [143.182.124.37]) by gabe.freedesktop.org (Postfix) with ESMTP id D2422E5C2F for ; Tue, 4 Jun 2013 08:58:06 -0700 (PDT) Content-Disposition: inline In-Reply-To: <20130604141912.GD2052@cantiga.alporthouse.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org Errors-To: intel-gfx-bounces+gcfxdi-intel-gfx=m.gmane.org@lists.freedesktop.org To: Chris Wilson , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org On Tue, Jun 04, 2013 at 03:19:12PM +0100, Chris Wilson wrote: > On Tue, May 21, 2013 at 03:28:34PM +0300, ville.syrjala@linux.intel.com w= rote: > > From: Ville Syrj=E4l=E4 > > = > > The specs are a bit unclear whether the per-plane trickle feed disable > > control exists on VLV. There is another trickle feed disable control > > in the MI_ARB register. > > = > > Based on some quick register dumps from Jani's VLV machine, the per-pla= ne > > bits don't actually seem to exist, so I'm guessing we should use the > > MI_ARB method instead. > = > I think neither. I'm guessing this behaviour is covered by 'Note: On > mobile products this bit will be ignored such that Trickle Feed is > always disabled.' in the DSP.CNTR spec. And I don't think MI_ARB_STATE > is applicable as it wasn't used for the display block VLV was based > upon. > = > Being VPN-less I can't actually dig through the VLV bspec to confirm. The register spec has it, and it's even highlighted so it was apparetnly changed at some point. Also note that the register seems to differ from MI_ARB_STATE by being the non masked type, so clearly someone felt the need to edit it for some reason. The display cluster doc also says that trickle feed is configurable, but doesn't go into any further details on the subject. I guess we should confirm on real hardware whether the bit sticks. I don't remember if I poked Jani to do that since he was a bit busy with other stuff when I made the patch. Anyways, that could give us a hint whether the bit really does something. -- = Ville Syrj=E4l=E4 Intel OTC