Re: [PATCH 4/5] drm/i915: Use only blitter forcewake

["Kamble, Sagar A" <sagar.a.kamble@xxxxxxxxx>]

Thanks Chris.

Domain refcount tracking is not common single function.
its present in __force_wake_get, __intel_uncore_forcewake_get and likewise for put functions.

Either we will have to bring them together in fw_domains_get and then add change you have suggested
Or
Prepare WA version of __force_wake_get and __intel_uncore_forcewake_get
Or
approach in the current patch where we only init blitter domain.
This will work from __intel_uncore_forcewake_get as well as it filters domains based on dev_priv->uncore.fw_domains.

Let me know which approach looks good.

On 8/23/2015 6:00 PM, Chris Wilson wrote:

On Sun, Aug 23, 2015 at 05:52:50PM +0530, Sagar Arun Kamble wrote:

Coarse power gating is disabled prior to BXT B0 and till SKL E0,
hence even for render and media well registers blitter forcewake request
need to be used.

Just insert a custom force_wake_get/put for the w/a.

Something like

gen9_force_wake_get_wa()
{
	return fw_domains_get(BLITTER);
}

The fw counting will make it work just fine.
-Chris

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx