On Fri, 2011-01-28 at 18:00 -0800, Bhanu Gollapudi wrote:
> This patch contains header files for bnx2fc driver.

Do we have to have this litter of

#ifdef (__BIG_ENDIAN)
...
#else
...
#endif

throughout all the structures ... it really makes quite a mess, and is
very fragile (you have two entries to change and test each time, not
just one).  You can use the cpu neutral format for all the headers and
then use the cpu_to... accessors to make the conversions.

To see how to do all of this, here's an ancient driver that used to look
like yours and got converted:

http://git.kernel.org/?p=linux/kernel/git/tglx/history.git;a=commit;h=0353305208ae64b5d4a27957bb3e25c6045b3f68

James


_______________________________________________
devel mailing list
[email protected]
https://lists.open-fcoe.org/mailman/listinfo/devel

Reply via email to