Re: FW: Submission for S2io 10GbE driver

[Jeff Garzik <jgarzik@xxxxxxxxx>]

Leonid Grossman wrote:
> > Thats how all big endian platforms work. in* and out*, read* 
> > and write* byteswap. 
>
>
> So, we should make the code big endian specific rather than PPC64
> specific, right?


{read,write}[bwlq] should work the same regardless of whether its big 
endian or little endian.  The rule is "PCI is defined to be little 
endian".  On little endian platforms, no byte swapping occurs.  On big 
endian platforms, the platform will byteswap.  Thus, the driver should 
not have big-endian-specific or PPC64-specific code...

(you still have to do your own byteswapping for DMA)

        Jeff