<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, May 16, 2017 at 4:07 PM, Matt Turner <span dir="ltr"><<a href="mailto:mattst88@gmail.com" target="_blank">mattst88@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Tue, May 16, 2017 at 3:45 PM, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> Iron Lake introduced the multiple KSP thing and so you have KSP0-3.<br>
> However, the genxml didn't have an index on the first "Kernel Start<br>
> Pointer" or "GRF Register Count".  Add one to match gen6+.<br>
> ---<br>
>  src/intel/genxml/gen5.xml | 4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/src/intel/genxml/gen5.xml b/src/intel/genxml/gen5.xml<br>
> index 87377df..a521737 100644<br>
> --- a/src/intel/genxml/gen5.xml<br>
> +++ b/src/intel/genxml/gen5.xml<br>
> @@ -864,8 +864,8 @@<br>
>    </struct><br>
><br>
>    <struct name="WM_STATE" length="11"><br>
> -    <field name="Kernel Start Pointer" start="6" end="31" type="offset"/><br>
> -    <field name="GRF Register Count" start="1" end="3" type="uint"/><br>
> +    <field name="Kernel Start Pointer[0]" start="6" end="31" type="offset"/><br>
<br>
</span>I removed the brackets from the Kernel Start Pointer names in commit<br>
3443bd45a3. Please just add 0 and not [0]<br></blockquote><div><br></div><div>So you did!  I'll make it just 0.  I'll strip the brackets off the GRF register counts while I'm at it.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
With that,<br>
<br>
Reviewed-by: Matt Turner <<a href="mailto:mattst88@gmail.com">mattst88@gmail.com</a>><br>
</blockquote></div><br></div><div class="gmail_extra">Thanks!<br></div></div>