include: sbi: Support byteorder macros in assembly
authorVivian Wang <dramforever@live.com>
Fri, 15 Mar 2024 17:36:39 +0000 (01:36 +0800)
committerAnup Patel <anup@brainfault.org>
Fri, 5 Apr 2024 11:53:51 +0000 (17:23 +0530)
Avoid using C types and casts if sbi/sbi_byteorder.h is included in
assembly code

Signed-off-by: Vivian Wang <dramforever@live.com>
Reviewed-by: Anup Patel <anup@brainfault.org>
include/sbi/sbi_byteorder.h

index db6eb2b967ef7fcaa6adefd7ffaede56a8917a67..2b4981e028918d4ac00e9f6d3ec667ab7f6cd062 100644 (file)
@@ -7,7 +7,12 @@
 #ifndef __SBI_BYTEORDER_H__
 #define __SBI_BYTEORDER_H__
 
-#include <sbi/sbi_types.h>
+#ifdef __ASSEMBLER__
+# define _conv_cast(type, val) (val)
+#else
+# include <sbi/sbi_types.h>
+# define _conv_cast(type, val) ((type)(val))
+#endif
 
 #define BSWAP16(x)     ((((x) & 0x00ff) << 8) | \
                         (((x) & 0xff00) >> 8))
                         (((x) & 0xff00000000000000ULL) >> 56))
 
 #if __BYTE_ORDER__ == __ORDER_LITTLE_ENDIAN__  /* CPU(little-endian) */
-#define cpu_to_be16(x)         ((uint16_t)BSWAP16(x))
-#define cpu_to_be32(x)         ((uint32_t)BSWAP32(x))
-#define cpu_to_be64(x)         ((uint64_t)BSWAP64(x))
+#define cpu_to_be16(x)         _conv_cast(uint16_t, BSWAP16(x))
+#define cpu_to_be32(x)         _conv_cast(uint32_t, BSWAP32(x))
+#define cpu_to_be64(x)         _conv_cast(uint64_t, BSWAP64(x))
 
-#define be16_to_cpu(x)         ((uint16_t)BSWAP16(x))
-#define be32_to_cpu(x)         ((uint32_t)BSWAP32(x))
-#define be64_to_cpu(x)         ((uint64_t)BSWAP64(x))
+#define be16_to_cpu(x)         _conv_cast(uint16_t, BSWAP16(x))
+#define be32_to_cpu(x)         _conv_cast(uint32_t, BSWAP32(x))
+#define be64_to_cpu(x)         _conv_cast(uint64_t, BSWAP64(x))
 
-#define cpu_to_le16(x)         ((uint16_t)(x))
-#define cpu_to_le32(x)         ((uint32_t)(x))
-#define cpu_to_le64(x)         ((uint64_t)(x))
+#define cpu_to_le16(x)         _conv_cast(uint16_t, (x))
+#define cpu_to_le32(x)         _conv_cast(uint32_t, (x))
+#define cpu_to_le64(x)         _conv_cast(uint64_t, (x))
 
-#define le16_to_cpu(x)         ((uint16_t)(x))
-#define le32_to_cpu(x)         ((uint32_t)(x))
-#define le64_to_cpu(x)         ((uint64_t)(x))
+#define le16_to_cpu(x)         _conv_cast(uint16_t, (x))
+#define le32_to_cpu(x)         _conv_cast(uint32_t, (x))
+#define le64_to_cpu(x)         _conv_cast(uint64_t, (x))
 #else /* CPU(big-endian) */
-#define cpu_to_be16(x)         ((uint16_t)(x))
-#define cpu_to_be32(x)         ((uint32_t)(x))
-#define cpu_to_be64(x)         ((uint64_t)(x))
+#define cpu_to_be16(x)         _conv_cast(uint16_t, (x))
+#define cpu_to_be32(x)         _conv_cast(uint32_t, (x))
+#define cpu_to_be64(x)         _conv_cast(uint64_t, (x))
 
-#define be16_to_cpu(x)         ((uint16_t)(x))
-#define be32_to_cpu(x)         ((uint32_t)(x))
-#define be64_to_cpu(x)         ((uint64_t)(x))
+#define be16_to_cpu(x)         _conv_cast(uint16_t, (x))
+#define be32_to_cpu(x)         _conv_cast(uint32_t, (x))
+#define be64_to_cpu(x)         _conv_cast(uint64_t, (x))
 
-#define cpu_to_le16(x)         ((uint16_t)BSWAP16(x))
-#define cpu_to_le32(x)         ((uint32_t)BSWAP32(x))
-#define cpu_to_le64(x)         ((uint64_t)BSWAP64(x))
+#define cpu_to_le16(x)         _conv_cast(uint16_t, BSWAP16(x))
+#define cpu_to_le32(x)         _conv_cast(uint32_t, BSWAP32(x))
+#define cpu_to_le64(x)         _conv_cast(uint64_t, BSWAP64(x))
 
-#define le16_to_cpu(x)         ((uint16_t)BSWAP16(x))
-#define le32_to_cpu(x)         ((uint32_t)BSWAP32(x))
-#define le64_to_cpu(x)         ((uint64_t)BSWAP64(x))
+#define le16_to_cpu(x)         _conv_cast(uint16_t, BSWAP16(x))
+#define le32_to_cpu(x)         _conv_cast(uint32_t, BSWAP32(x))
+#define le64_to_cpu(x)         _conv_cast(uint64_t, BSWAP64(x))
 #endif
 
 #if __riscv_xlen == 64