From: Geert U. <ge...@li...> - 2005-08-01 05:58:13
|
On Sun, 31 Jul 2005, John W. Linville wrote: > Fix-up some basic compile failures related to APUS. > > Signed-off-by: John W. Linville <lin...@tu...> > --- > The "asm volatile" fixes in zorro.h were actually warning fix-ups, but > easier to just include all the fixes for that file in one patch... > > arch/ppc/kernel/head.S | 4 ++-- > include/asm-ppc/zorro.h | 37 +++++++++++++++++-------------------- > 2 files changed, 19 insertions(+), 22 deletions(-) > > --- linux-apus/include/asm-ppc/zorro.h.orig 2005-07-31 14:18:57.000000000 -0400 > +++ linux-apus/include/asm-ppc/zorro.h 2005-07-31 14:36:47.000000000 -0400 > @@ -7,8 +7,8 @@ static inline unsigned int z_readb(unsig > { > unsigned int ret; > > - asm volatile ("lbz%U1%X1 %0,%1; eieio" > - : "=r" (ret) : "m" (*(unsigned char *)addr)); > + __asm__ __volatile__ ("lbz%U1%X1 %0,%1; eieio" > + : "=r" (ret) : "m" (*(unsigned char *)addr)); What warning do you get with the `asm volatile'? Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@li... In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds |