]> err.no Git - linux-2.6/commitdiff
avr32: Work around byteswap bug in gcc < 4.2
authorHaavard Skinnemoen <hskinnemoen@atmel.com>
Mon, 10 Mar 2008 11:56:12 +0000 (12:56 +0100)
committerHaavard Skinnemoen <hskinnemoen@atmel.com>
Mon, 10 Mar 2008 12:32:33 +0000 (13:32 +0100)
gcc versions earlier than 4.2 sign-extends the result of le16_to_cpu()
and friends when we implement __arch__swabX() using
__builtin_bswap_X(). Disable our arch-specific optimizations when those
gcc versions are being used.

Signed-off-by: Haavard Skinnemoen <hskinnemoen@atmel.com>
include/asm-avr32/byteorder.h

index 402ff4125cdc25d570bf9f769045f6d17aa62b61..d77b48ba73387ba57ac932e5a6676d43b1c6a584 100644 (file)
@@ -12,8 +12,14 @@ extern unsigned long __builtin_bswap_32(unsigned long x);
 extern unsigned short __builtin_bswap_16(unsigned short x);
 #endif
 
+/*
+ * avr32-linux-gcc versions earlier than 4.2 improperly sign-extends
+ * the result.
+ */
+#if !(__GNUC__ == 4 && __GNUC_MINOR__ < 2)
 #define __arch__swab32(x) __builtin_bswap_32(x)
 #define __arch__swab16(x) __builtin_bswap_16(x)
+#endif
 
 #if !defined(__STRICT_ANSI__) || defined(__KERNEL__)
 # define __BYTEORDER_HAS_U64__