[igt-dev] [PATCH i-g-t] i915/gem_workarounds: adding 0xe420 register to WO list

Mika Kuoppala mika.kuoppala at linux.intel.com
Thu Dec 20 10:54:33 UTC 2018


Talha Nassar <talha.nassar at intel.com> writes:

> From: talha nassar <talha.nassar at intel.com>
>
> HW team confirmed that this register is write-only.

It could be that it is that bit only. We don't need
to care about that now,

Reviewed-by: Mika Kuoppala <mika.kuoppala at linux.intel.com>

>
> References: https://bugs.freedesktop.org/show_bug.cgi?id=107338
> Cc: Chris Wilson <chris at chris-wilson.co.uk>
> Cc: Mika Kuoppala <mika.kuoppala at linux.intel.com>
> Signed-off-by: talha nassar <talha.nassar at intel.com>
> ---
>  tests/i915/gem_workarounds.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/tests/i915/gem_workarounds.c b/tests/i915/gem_workarounds.c
> index 78478ad..b7d7d9c 100644
> --- a/tests/i915/gem_workarounds.c
> +++ b/tests/i915/gem_workarounds.c
> @@ -51,7 +51,8 @@ static struct write_only_list {
>  	unsigned int gen;
>  	uint32_t addr;
>  } wo_list[] = {
> -	{ 10, 0xE5F0 } /* WaForceContextSaveRestoreNonCoherent:cnl */
> +	{ 10, 0xE5F0 }, /* WaForceContextSaveRestoreNonCoherent:cnl */
> +	{ 11, 0xE420 }  /* WaEnableFloatBlendOptimization:icl */
>  
>  	/*
>  	 * FIXME: If you are contemplating adding stuff here
> -- 
> 2.7.4


More information about the igt-dev mailing list