Fw: [PATCH] Remove powerpc specific parts of 3c509 driver

Paul Mackerras paulus at samba.org
Wed Sep 20 09:27:11 EST 2006


Linas Vepstas writes:

> The rest of this patch might indeed be correct, but the above comment 
> bothers me. The "ns" versions of routines are supposed to be
> non-byte-swapped versions of the insl/outsl routines (which would
> byte-swap on big-endian archs such as powerpc.)

If it were true that in/outsw and in/outsl were actually used to
transfer arrays of 16-bit data items or 32-bit data items to/from an
I/O device, I would agree with you, but they aren't.  They are
universally used to transfer arrays of bytes, with the optimization of
doing so 2 or 4 bytes at a time.  That is why in/outsw and in/outsl
don't (and shouldn't) do byte swapping.

Paul.



More information about the Linuxppc-dev mailing list