On Wed, Nov 30, 2011 at 02:09:20PM +0100, Segher Boessenkool wrote: > >+#define mfdcrx(rn) \ > >+ ({ \ > >+ unsigned long rval; \ > >+ asm volatile("mfdcrx %0,%1" : "=r"(rval) : "g"(rn)); \ > >+ rval; \ > >+ }) > > "g" is never correct on PowerPC, you want "r" here. You can write > this as a static inline btw, you only need the #define stuff when > there is an "i" constraint involved.
Okay I'll change it to "i", mostly I used a #define to match the style of m[tf]dcr. To be honnest I didn't know about the issue with "i" constraints and static inlines. Yours Tony
pgpS0mxIQ2IzI.pgp
Description: PGP signature
_______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev