opensbi.git/include/sbi/sbi_byteorder.h, branch master

include: sbi: fix swap errors with newer gcc -Werror=sequence-point

2025-07-20T15:45:10Z

The BSWAPxx() macros are now throwing the following warnings with newer gcc versions. This is due to throwing an argument in that may be evaluated more than one (I think) and therefore things like the example below should be avoided. Fix by making a set of BSWAPxx() wrappers which specifically only evaluate 'x' once. In file included lib/sbi/sbi_mpxy.c:21: lib/sbi/sbi_mpxy.c: In function ‘sbi_mpxy_write_attrs’: ib/sbi/sbi_mpxy.c:632:63: error: operation on ‘mem_idx’ may be undefined [-Werror=sequence-point] 632 | attr_val = le32_to_cpu(mem_ptr[mem_idx++]); | ~~~~~~~^~ Signed-off-by: Ben Dooks Reviewed-by: Rahul Pathak Reviewed-by: Xiang W Link: https://lore.kernel.org/r/20250704122938.897832-1-ben.dooks@codethink.co.uk Signed-off-by: Anup Patel

include: sbi: Support byteorder macros in assembly

2024-04-05T11:53:51Z

Avoid using C types and casts if sbi/sbi_byteorder.h is included in assembly code Signed-off-by: Vivian Wang Reviewed-by: Anup Patel

include: sbi: Introduce common endianess conversion macro

2024-01-10T04:13:28Z

Introduce cpu_to_lle and lle_to_cpu macros which invoke correct word length cpu_to_le<64/32> conversion based on __riscv_xlen. Signed-off-by: Himanshu Chauhan Reviewed-by: Anup Patel

include: sbi: Fix BSWAPx() macros for big-endian host

2023-02-09T04:01:10Z

The BSWAPx() macros won't do any swapping for big-endian host because the EXTRACT_BYTE() macro will pickup bytes in reverse order. Also, the EXTRACT_BYTE() will generate compile error for constants. To fix this, we get remove the EXTRACT_BYTE() macro and re-write BSWAPx() using simple mask and shift operations. Fixes: 09b34d8cca51 ("include: Add support for byteorder/endianness conversion") Reported-by: Samuel Holland Signed-off-by: Anup Patel

include: Add support for byteorder/endianness conversion

2023-02-08T12:40:39Z

Define macros general byteorder conversion Define functions for endianness conversion from general byteorder conversion macros Signed-off-by: Rahul Pathak Reviewed-by: Xiang W Reviewed-by: Sergey Matyukevich Reviewed-by: Anup Patel