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
