<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jan 23, 2017 at 4:59 AM, Ilia Mirkin <span dir="ltr"><<a href="mailto:imirkin@alum.mit.edu" target="_blank">imirkin@alum.mit.edu</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 Mon, Jan 23, 2017 at 1:42 AM, Kenneth Graunke <<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>> wrote:<br>
> + *ymin = MIN2(ndc_gb_ymin, ndc_gb_ymax);<br>
<br>
</span>Should this be limited on the negative end of the range?<br>
<span class=""><br>
> + *ymax = MIN2(MAX2(ndc_gb_ymin, ndc_gb_ymax), 16383);<br>
<br>
</span>And should this be a different value for gen6? Perhaps based on gb_size?<br></blockquote><div><br></div><div>I don't think so. It's a limit on some hardware for unknown reasons. That said, Ken, it would be really nice to get an actual PRM citation because I can't find it anywhere. <br></div></div><br></div></div>