On Mon, May 19, 2014 at 12:09:55PM +0400, Anton Saraev wrote: > #if SKEIN_UNROLL_256 == 0 > -#define R256(p0, p1, p2, p3, ROT, rNum) /* fully unrolled */ \ > +#define R256(p0, p1, p2, p3, ROT, r_num) /* fully unrolled */ \ > do { \ > - Round256(p0, p1, p2, p3, ROT, rNum) \ > - Skein_Show_R_Ptr(BLK_BITS, &ctx->h, rNum, Xptr); \ > + Round256(p0, p1, p2, p3, ROT, r_num); \ > + Skein_Show_R_Ptr(BLK_BITS, &ctx->h, r_num, X_ptr); \ Adding a semi-colon here looks suspiciously like a compile fix. It's great that you're fixing bugs, but do it in a separate patch with a changelog. regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel