On Tue, Jan 24, 2012 at 09:04, Daniel Vetter <dan...@ffwll.ch> wrote:
> On Tue, Jan 24, 2012 at 10:35:04AM +0000, Chris Wilson wrote: > > On Tue, 24 Jan 2012 09:44:28 +0100, Daniel Vetter < > daniel.vet...@ffwll.ch> wrote: > > > We need this to correctly access registers in the gt power well from > > > userspace. > > > > How about a INTEL_INFO(dev)->has_forcewake or check for the > > ->forcewake_get? That would be more self-documenting, and we are less > > likely to miss one in the future, than adding more random generation > > checks. > > I'll jot down a todo to create a cleanup patch for -next that introduce > some feature flags like has_forcewake has_rc6 and such to avoid such > gaffles (hopefully). > I actually wrote about the very same idea in my previous email :). So +1 to that! -- Eugeni Dodonov <http://eugeni.dodonov.net/>
_______________________________________________ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/intel-gfx
