[Mesa-dev] [PATCH 05/11] genxml: Remove brackets from kernel start pointer names

Rafael Antognolli rafael.antognolli at intel.com
Tue May 2 22:04:13 UTC 2017


Patch is
Reviewed-by: Rafael Antognolli <rafael.antognolli at intel.com>

On Mon, May 01, 2017 at 01:54:49PM -0700, Matt Turner wrote:
> Newer Gens' names don't have the brackets. Having common names will make
> some later patches simpler.
> ---
>  src/intel/genxml/gen4.xml  | 2 +-
>  src/intel/genxml/gen45.xml | 2 +-
>  src/intel/genxml/gen6.xml  | 6 +++---
>  3 files changed, 5 insertions(+), 5 deletions(-)
> 
> diff --git a/src/intel/genxml/gen4.xml b/src/intel/genxml/gen4.xml
> index 5ea15e7..22755bf 100644
> --- a/src/intel/genxml/gen4.xml
> +++ b/src/intel/genxml/gen4.xml
> @@ -474,7 +474,7 @@
>  
>    <struct name="WM_STATE" length="7">
>      <!-- DWord 0 -->
> -    <field name="Kernel Start Pointer[0]" start="6" end="31" type="address"/>
> +    <field name="Kernel Start Pointer 0" start="6" end="31" type="address"/>
>      <field name="GRF Register Count" start="1" end="3" type="uint"/>
>      <!-- DWord 1 -->
>      <field name="Single Program Flow" start="63" end="63" type="bool"/>
> diff --git a/src/intel/genxml/gen45.xml b/src/intel/genxml/gen45.xml
> index 4582beb..9c43aef 100644
> --- a/src/intel/genxml/gen45.xml
> +++ b/src/intel/genxml/gen45.xml
> @@ -466,7 +466,7 @@
>  
>    <struct name="WM_STATE" length="7">
>      <!-- DWord 0 -->
> -    <field name="Kernel Start Pointer[0]" start="6" end="31" type="address"/>
> +    <field name="Kernel Start Pointer 0" start="6" end="31" type="address"/>
>      <field name="GRF Register Count" start="1" end="3" type="uint"/>
>      <!-- DWord 1 -->
>      <field name="Single Program Flow" start="63" end="63" type="bool"/>
> diff --git a/src/intel/genxml/gen6.xml b/src/intel/genxml/gen6.xml
> index 3059bfc..a5e39d3 100644
> --- a/src/intel/genxml/gen6.xml
> +++ b/src/intel/genxml/gen6.xml
> @@ -1392,7 +1392,7 @@
>      <field name="3D Command Opcode" start="24" end="26" type="uint" default="0"/>
>      <field name="3D Command Sub Opcode" start="16" end="23" type="uint" default="20"/>
>      <field name="DWord Length" start="0" end="7" type="uint" default="7"/>
> -    <field name="Kernel Start Pointer[0]" start="38" end="63" type="offset"/>
> +    <field name="Kernel Start Pointer 0" start="38" end="63" type="offset"/>
>      <field name="Single Program Flow" start="95" end="95" type="bool"/>
>      <field name="Vector Mask Enable" start="94" end="94" type="bool"/>
>      <field name="Sampler Count" start="91" end="93" type="uint"/>
> @@ -1475,8 +1475,8 @@
>        <value name="MSDISPMODE_PERSAMPLE" value="0"/>
>        <value name="MSDISPMODE_PERPIXEL" value="1"/>
>      </field>
> -    <field name="Kernel Start Pointer[1]" start="230" end="255" type="offset"/>
> -    <field name="Kernel Start Pointer[2]" start="262" end="287" type="offset"/>
> +    <field name="Kernel Start Pointer 1" start="230" end="255" type="offset"/>
> +    <field name="Kernel Start Pointer 2" start="262" end="287" type="offset"/>
>    </instruction>
>  
>    <instruction name="MEDIA_CURBE_LOAD" bias="2" length="4">
> -- 
> 2.10.2
> 
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/mesa-dev


More information about the mesa-dev mailing list