<div dir="ltr"><div>Hi,<br><div class="gmail_extra"><br></div><div class="gmail_extra">I think this patch could this still benefit from being broken into smaller bits -- it looks to me like it is still both adding new code and moving old code around at the same time. <br>
<br>I'd suggest breaking it into at least two separate patches where the first one adds an empty MIPSr2 implementation that doesn't do anytning, and the second one then moves the various fast paths from the current dspr2 implementation into the mips2r2 one. A third one can then add any new MIPSr2 patches. <br>
<br><br><br><br><br></div><div class="gmail_extra">This way, it's easy to see what code needs careful review, and which code can be assumed to already work well.<br><br></div><div class="gmail_extra">I also have some specific comments on the changes to the pixman-mips.c file, but I'd rather wait with those until those changes are available in a smaller patch.<br>
</div><br><br></div>Søren<br><br></div>