Hi Jate,

Jate Sujjavanich wrote:
The following define doesn't work for me and my gcc 4.4:


diff -Nur uClinux-dist-20100315/user/e2fsprogs/lib/ext2fs/bitops.h 
uClinux-dist/user/e2fsprogs/lib/ext2fs/bitops.h
--- uClinux-dist-20100315/user/e2fsprogs/lib/ext2fs/bitops.h    2006-09-05 
21:42:09.000000000 -0400
+++ uClinux-dist/user/e2fsprogs/lib/ext2fs/bitops.h     2010-06-24 
10:32:43.986846919 -0400
@@ -291,7 +291,7 @@
 #endif /* i386 */
#if ((defined __GNUC__) && !defined(_EXT2_USE_C_VERSIONS_) && \
-     (defined(__mc68000__) && !defined(__mcf5200)))
+     (defined(__mc68000__) && !defined(__mcf5200__)))
#define _EXT2_HAVE_ASM_BITOPS_


Should __mcf5200 remain for backwards compatibility with older toolchains? If so, then 
I should add some more && !defined(*)'s.

I am not sure that __mcf5200 was ever correct. Greping around the
current source tree, there is no other use of __mcf5200. All other
uses of this are __mcf5200__, so I have a feeling that is the correct
usage, and that __mcf5200 is in fact wrong for any compiler version.

Regards
Greg


------------------------------------------------------------------------
Greg Ungerer  --  Principal Engineer        EMAIL:     g...@snapgear.com
SnapGear Group, McAfee                      PHONE:       +61 7 3435 2888
8 Gardner Close                             FAX:         +61 7 3217 5323
Milton, QLD, 4064, Australia                WEB: http://www.SnapGear.com
_______________________________________________
uClinux-dev mailing list
uClinux-dev@uclinux.org
http://mailman.uclinux.org/mailman/listinfo/uclinux-dev
This message was resent by uclinux-dev@uclinux.org
To unsubscribe see:
http://mailman.uclinux.org/mailman/options/uclinux-dev

Reply via email to