From: mifi <mifi@b42882b7-edfa-0310-969c-e2dbd0fdcd60>
Date: Tue, 19 Feb 2008 21:44:19 +0000 (+0000)
Subject: - added patch which fixed big/small endian problems on non-x86 host CPUs.
X-Git-Tag: v0.1.0~976
X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=commitdiff_plain;h=6057fc8ab8a0e4395b1675250d03e199976186eb

- added patch which fixed big/small endian problems on non-x86 host CPUs.
https://lists.berlios.de/pipermail/openocd-development/2008-February/000846.html
(thanks to Øyvind for the patch)

git-svn-id: svn://svn.berlios.de/openocd/trunk@313 b42882b7-edfa-0310-969c-e2dbd0fdcd60
---

diff --git a/src/helper/types.h b/src/helper/types.h
index c06a7ececa..50d0df7eee 100644
--- a/src/helper/types.h
+++ b/src/helper/types.h
@@ -19,10 +19,10 @@
  ***************************************************************************/
 #ifndef TYPES_H
 #define TYPES_H
-
+
 #ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif
+#endif
 
 #ifndef u8
 typedef unsigned char u8;
@@ -40,53 +40,40 @@ typedef unsigned int u32;
 typedef unsigned long long u64;
 #endif
 
-#ifdef WORDS_BIGENDIAN /* big endian host */
-
-#define le_to_h_u32(x) (u32)((x)[0] | (x)[1] << 8 | (x)[2] << 16 | (x)[3] << 24)
-#define le_to_h_u16(x) (u16)((x)[0] | (x)[1] << 8)
-#define be_to_h_u32(x) (*(u32*)(x))
-#define be_to_h_u16(x) (*(u16*)(x))
-
-#define h_u32_to_le(buf, val) \
-	do { \
-		(buf)[3] = ((val) & 0xff000000) >> 24; \
-		(buf)[2] = ((val) & 0x00ff0000) >> 16; \
-		(buf)[1] = ((val) & 0x0000ff00) >> 8; \
-		(buf)[0] = ((val) & 0x000000ff); \
-	} while (0)
 
-#define h_u16_to_le(buf, val)  \
-	do { \
-		(buf)[1] = ((val) & 0xff00) >> 8; \
-		(buf)[0] = ((val) & 0x00ff); \
-	} while (0)
 
-#define h_u32_to_be(buf, val) do { *(u32*)(buf) = (val); } while (0)
-#define h_u16_to_be(buf, val) do { *(u16*)(buf) = (val); } while (0)
+/* DANGER!!!! here be dragons! Note that the pointer in 
+ * memory might be unaligned. On some CPU's, i.e. ARM7,
+ * the 2 lsb are ignored for 32 bit access, on others
+ * it will cause an exception and on e.g. x86, it works
+ * the same as if aligned.
+ */
+#define le_to_h_u32(x) ((u32)((x)[0] | (x)[1] << 8 | (x)[2] << 16 | (x)[3] << 24))
+#define le_to_h_u16(x) ((u16)((x)[0] | (x)[1] << 8))
+#define be_to_h_u32(x) ((u32)((x)[3] | (x)[2] << 8 | (x)[1] << 16 | (x)[0] << 24))
+#define be_to_h_u16(x) ((u16)((x)[1] | (x)[0] << 8))
 
-#else /* little endian host */
+#define h_u32_to_le(buf, val) do {\
+(buf)[3] = ((val) & 0xff000000) >> 24;\
+(buf)[2] = ((val) & 0x00ff0000) >> 16;\
+(buf)[1] = ((val) & 0x0000ff00) >> 8;\
+(buf)[0] = ((val) & 0x000000ff);\
+} while (0)
+#define h_u32_to_be(buf, val) do {\
+(buf)[0] = ((val) & 0xff000000) >> 24;\
+(buf)[1] = ((val) & 0x00ff0000) >> 16;\
+(buf)[2] = ((val) & 0x0000ff00) >> 8;\
+(buf)[3] = ((val) & 0x000000ff);\
+} while (0)
 
-#define le_to_h_u32(x) (*(u32*)(x))
-#define le_to_h_u16(x) (*(u16*)(x))
-#define be_to_h_u32(x) (u32)((x)[3] | (x)[2] << 8 | (x)[1] << 16 | (x)[0] << 24)
-#define be_to_h_u16(x) (u16)((x)[1] | (x)[0] << 8)
+#define h_u16_to_le(buf, val) do {\
+(buf)[1] = ((val) & 0xff00) >> 8;\
+(buf)[0] = ((val) & 0x00ff) >> 0;\
+} while (0)
+#define h_u16_to_be(buf, val) do {\
+(buf)[0] = ((val) & 0xff00) >> 8;\
+(buf)[1] = ((val) & 0x00ff) >> 0;\
+} while (0)
 
-#define h_u32_to_le(buf, val) do { *(u32*)(buf) = (val); } while (0)
-#define h_u16_to_le(buf, val) do { *(u16*)(buf) = (val); } while (0)
-
-#define h_u32_to_be(buf, val) \
-	do { \
-		(buf)[0] = ((val) & 0xff000000) >> 24; \
-		(buf)[1] = ((val) & 0x00ff0000) >> 16; \
-		(buf)[2] = ((val) & 0x0000ff00) >> 8; \
-		(buf)[3] = ((val) & 0x000000ff); \
-	} while (0)
-
-#define h_u16_to_be(buf, val) \
-	do { \
-		(buf)[0] = ((val) & 0xff00) >> 8; \
-		(buf)[1] = ((val) & 0x00ff); \
-	} while (0)
-#endif
 
 #endif /* TYPES_H */