netdev
[Top] [All Lists]

Re: FW: Submission for S2io 10GbE driver

To: Leonid Grossman <leonid.grossman@xxxxxxxx>
Subject: Re: FW: Submission for S2io 10GbE driver
From: Jeff Garzik <jgarzik@xxxxxxxxx>
Date: Thu, 05 Feb 2004 04:29:57 -0500
Cc: "'Anton Blanchard'" <anton@xxxxxxxxx>, "'Andi Kleen'" <ak@xxxxxxx>, netdev@xxxxxxxxxxx, raghava.vatsavayi@xxxxxxxx, iod00d@xxxxxx
In-reply-to: <005701c3eb92$55dc7650$7310100a@xxxxxxxxxxxx>
References: <005701c3eb92$55dc7650$7310100a@xxxxxxxxxxxx>
Sender: netdev-bounce@xxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030703
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




<Prev in Thread] Current Thread [Next in Thread>