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 at linux.intel.com wrote: > > From: Ville Syrj?l? <ville.syrjala at linux.intel.com> > > > > 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-plane > > 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?l? Intel OTC
