On 17 February 2017 at 05:32, Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> wrote: > There's a lot of duplication for what's essentially N copies of > the same loop, so factor it. The upcoming AST2500 POST code adds > more of this. > > Signed-off-by: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx> Here is where things go fun: Original code does the following data flexing only for the mmc_*test2 code, while this patch adds it to both test and test2. data = ast_mindwm(...); data = (data | (data >> 16)) & 0xffff; // ast_moutdwm(...); return data; Afaict you have addressed that in the github branch/patch so things are back to normal. Tests are still a bit magical but not much you can/should do about it. Regards, Emil _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel