- added patch which fixed big/small endian problems on non-x86 host CPUs.
authormifi <mifi@b42882b7-edfa-0310-969c-e2dbd0fdcd60>
Tue, 19 Feb 2008 21:44:19 +0000 (21:44 +0000)
committermifi <mifi@b42882b7-edfa-0310-969c-e2dbd0fdcd60>
Tue, 19 Feb 2008 21:44:19 +0000 (21:44 +0000)
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

src/helper/types.h

index c06a7ececa6880ccdd8095b9c7bce1769fbd6aff..50d0df7eeed32dfcf47e4934c125636c5fb0d155 100644 (file)
  ***************************************************************************/
 #ifndef TYPES_H
 #define TYPES_H
-\r
+
 #ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif\r
+#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 */

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)