On 26 September 2011 15:09, Kenneth Graunke <span dir="ltr">&lt;<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On 09/25/2011 09:21 AM, Paul Berry wrote:<br>
</div><div class="im">&gt; Since the i965 driver supports 8 clipping planes now, we need 4 bits<br>
&gt; to store the number of user clipping planes, not 3.<br>
&gt;<br>
&gt; In theory this isn&#39;t strictly necessary, since brw_clip.h is only used<br>
&gt; on pre-GEN6, and pre-GEN6 only advertises support for 6 clipping<br>
&gt; planes, but it seems wise to err on the safe side.<br>
<br>
</div>Patches 1, 3, 5, 6, and 7 (the easy ones) are<br>
Reviewed-by: Kenneth Graunke &lt;<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>&gt;<br>
<br>
I still need to read 2 and 4.<br>
</blockquote></div><br>Thanks, Ken.  I&#39;ll make the change you suggested on patch 1.  I look forward to hearing your comments on patches 2 and 4--those are definitely the meaty ones from this series.<br>