diff options
author | Maxime Ripard <maxime@cerno.tech> | 2022-03-08 15:18:01 +0100 |
---|---|---|
committer | Maxime Ripard <maxime@cerno.tech> | 2022-06-28 15:56:04 +0200 |
commit | 9e9469cee564428d21ac2a72e81e4244ebca4d53 (patch) | |
tree | 93a0b657f763c72d20d52ba5edb478eccac6ee15 | |
parent | 4736e0fa308208c7407f9cc8e2fce07562258912 (diff) |
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 <pekka.paalanen@collabora.com>
Signed-off-by: Maxime Ripard <maxime@cerno.tech>
-rw-r--r-- | lib/igt_core.h | 9 | ||||
-rw-r--r-- | tests/kms_writeback.c | 2 |
2 files changed, 10 insertions, 1 deletions
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 <assert.h> +#include <byteswap.h> #include <setjmp.h> #include <stdbool.h> #include <stdint.h> @@ -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); } |