Ignatios Souvatzis wrote:

>> And are all the __asm volatile ("nop") in grf_cv.c really required?
>> I would replace them with "sync; isync" for PPC?
> 
> Yes, that's the meaning of 68k nop.
> 
> We should have include files for amigappc / amiga68k defining 
> the sync code.

I thought about that as well, but the nops only occur in grf_cv.c and
grf_cv3d.c, so I decided to solve it locally in those files:

#if defined(__m68k__)
#define cpu_sync() __asm volatile ("nop")
#elif defined(__powerpc__)
#define cpu_sync() __asm volatile ("sync; isync")
#endif


More headache is caused by all the other assembler instructions in grf_cv.c,
which are used in favour of a disabled C-block. I have enabled the old C
blocks for __powerpc__ now, but the driver doesn't really work (video signal
is out of sync).

-- 
    _  Frank Wille (frank@phoenix.owl.de)
 _ //  http://sun.hasenbraten.de/~frank/
 \X/   Phx @ #AmigaGer