Re: [Mesa-dev] [PATCH] genxml: Fix 3DSTATE_DEPTH_BUFFER length on gen5.

2017-05-03 Thread Kenneth Graunke
On Wednesday, May 3, 2017 5:53:35 PM PDT Rafael Antognolli wrote:
> The hardware docs are wrong, but the length used in the xml is also
> wrong.
> 
> Signed-off-by: Rafael Antognolli 
> ---
>  src/intel/genxml/gen5.xml | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
> 
> diff --git a/src/intel/genxml/gen5.xml b/src/intel/genxml/gen5.xml
> index 0b84650..447499c 100644
> --- a/src/intel/genxml/gen5.xml
> +++ b/src/intel/genxml/gen5.xml
> @@ -649,14 +649,14 @@
>  
>
>  
> -  
> -
> +  
> +
>  
>  
>   default="3"/>
>   default="1"/>
>   default="5"/>
> -
> +
>  
>  
>
> 

Yep, the Ironlake docs are awful.  "DWord 6" claims to contain Render
Target View Extent - but it does not.  It does not exist on Gen5.
It contains MOCS on Gen6+.  It contains RTVE on Gen7+.

Reviewed-by: Kenneth Graunke 


signature.asc
Description: This is a digitally signed message part.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [PATCH] genxml: Fix 3DSTATE_DEPTH_BUFFER length on gen5.

2017-05-03 Thread Rafael Antognolli
The hardware docs are wrong, but the length used in the xml is also
wrong.

Signed-off-by: Rafael Antognolli 
---
 src/intel/genxml/gen5.xml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/intel/genxml/gen5.xml b/src/intel/genxml/gen5.xml
index 0b84650..447499c 100644
--- a/src/intel/genxml/gen5.xml
+++ b/src/intel/genxml/gen5.xml
@@ -649,14 +649,14 @@
 
   
 
-  
-
+  
+
 
 
 
 
 
-
+
 
 
   
-- 
2.9.3

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev