On Thu, Nov 17, 2016 at 09:02:43AM +0000, Tvrtko Ursulin wrote:
> From: Tvrtko Ursulin <[email protected]>
> 
> Commit 0dd356bb6ff5 ("drm/i915: Eliminate Gen9 special case")
> accidentaly dropped a MMIO range between 0xc000 to 0xcfff out
> of the blitter forcewake domain. Fix it.
> 
> Signed-off-by: Tvrtko Ursulin <[email protected]>
> Fixes: 0dd356bb6ff5 ("drm/i915: Eliminate Gen9 special case")
> Reported-by: Daniele Ceraolo Spurio <[email protected]>
> Cc: Daniele Ceraolo Spurio <[email protected]>
> Cc: Chris Wilson <[email protected]>
> Cc: Tvrtko Ursulin <[email protected]>
> ---
>  drivers/gpu/drm/i915/intel_uncore.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_uncore.c 
> b/drivers/gpu/drm/i915/intel_uncore.c
> index e953303bff54..a0944dde7c41 100644
> --- a/drivers/gpu/drm/i915/intel_uncore.c
> +++ b/drivers/gpu/drm/i915/intel_uncore.c
> @@ -789,7 +789,7 @@ static const struct intel_forcewake_range 
> __gen9_fw_ranges[] = {
>       GEN_FW_RANGE(0x9400, 0x97ff, FORCEWAKE_RENDER | FORCEWAKE_MEDIA),
>       GEN_FW_RANGE(0x9800, 0xafff, FORCEWAKE_BLITTER),
>       GEN_FW_RANGE(0xb000, 0xb47f, FORCEWAKE_RENDER),
> -     GEN_FW_RANGE(0xb480, 0xbfff, FORCEWAKE_BLITTER),
> +     GEN_FW_RANGE(0xb480, 0xcfff, FORCEWAKE_BLITTER),
>       GEN_FW_RANGE(0xd000, 0xd7ff, FORCEWAKE_MEDIA),

The table should be watertight...
Reviewed-by: Chris Wilson <[email protected]>

So, possible to test in intel_fw_table_check()?
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to