From 9e9469cee564428d21ac2a72e81e4244ebca4d53 Mon Sep 17 00:00:00 2001 From: Maxime Ripard Date: Tue, 8 Mar 2022 15:18:01 +0100 Subject: tests/kms_writeback: Use endianness accessor to fill pixels The fill_fb() function in the kms_writeback test suite will fill an XRGB8888 buffer using a pattern passed an an argument. However, the pattern is native endian, while XRGB8888 is little-endian. Add an accessor and use it to fill the framebuffer with our pattern. Acked-by: Pekka Paalanen Signed-off-by: Maxime Ripard --- lib/igt_core.h | 9 +++++++++ tests/kms_writeback.c | 2 +- 2 files changed, 10 insertions(+), 1 deletion(-) diff --git a/lib/igt_core.h b/lib/igt_core.h index 526282fa..aa98e8ed 100644 --- a/lib/igt_core.h +++ b/lib/igt_core.h @@ -31,6 +31,7 @@ #define IGT_CORE_H #include +#include #include #include #include @@ -1447,6 +1448,14 @@ void igt_kmsg(const char *format, ...); #define READ_ONCE(x) (*(volatile typeof(x) *)(&(x))) #define WRITE_ONCE(x, v) do *(volatile typeof(x) *)(&(x)) = (v); while (0) +#if __BYTE_ORDER__ == __ORDER_BIG_ENDIAN__ +#define cpu_to_le32(x) bswap_32(x) +#define le32_to_cpu(x) bswap_32(x) +#else +#define cpu_to_le32(x) (x) +#define le32_to_cpu(x) (x) +#endif + #define MSEC_PER_SEC (1000) #define USEC_PER_SEC (1000*MSEC_PER_SEC) #define NSEC_PER_SEC (1000*USEC_PER_SEC) diff --git a/tests/kms_writeback.c b/tests/kms_writeback.c index 6efc72df..cf70a28e 100644 --- a/tests/kms_writeback.c +++ b/tests/kms_writeback.c @@ -244,7 +244,7 @@ static void fill_fb(igt_fb_t *fb, uint32_t pixel) pixel_count = fb->strides[0] * fb->height / sizeof(uint32_t); for (i = 0; i < pixel_count; i++) - ptr[i] = pixel; + ptr[i] = cpu_to_le32(pixel); igt_fb_unmap_buffer(fb, ptr); } -- cgit v1.2.3