Re: [PATCH 2/2] drm/tests: Fix drm_test_fb_xrgb8888_to_xrgb2101010() on big-endian

[Thomas Zimmermann]

Hi

Am 11.08.25 um 12:24 schrieb José Expósito:

Hi Thomas,

On Mon, Jun 30, 2025 at 01:37:58PM +0200, Thomas Zimmermann wrote:

Am 30.06.25 um 11:00 schrieb José Expósito:

Fix failures on big-endian architectures on tests cases
single_pixel_source_buffer, single_pixel_clip_rectangle,
well_known_colors and destination_pitch.

Fixes: 15bda1f8de5d ("drm/tests: Add calls to drm_fb_blit() on supported format 
conversion tests")
Signed-off-by: José Expósito <jose.exposit...@gmail.com>

Reviewed-by: Thomas Zimmermann <tzimmerm...@suse.de>

Just a friendly reminder about this patches. Can I merge them?
They have been in the ML for a while and it doesn't look like
there are concerns.

Merged into drm-misc-fixes now. Sorry for the delay.

Best regards
Thomas

Best wishes,
Jose

---
   drivers/gpu/drm/tests/drm_format_helper_test.c | 1 +
   1 file changed, 1 insertion(+)

diff --git a/drivers/gpu/drm/tests/drm_format_helper_test.c 
b/drivers/gpu/drm/tests/drm_format_helper_test.c
index 86829e1cb7f0..981dada8f3a8 100644
--- a/drivers/gpu/drm/tests/drm_format_helper_test.c
+++ b/drivers/gpu/drm/tests/drm_format_helper_test.c
@@ -1040,6 +1040,7 @@ static void drm_test_fb_xrgb8888_to_xrgb2101010(struct 
kunit *test)
        memset(buf, 0, dst_size);
        drm_fb_xrgb8888_to_xrgb2101010(&dst, dst_pitch, &src, &fb, &params->clip, 
&fmtcnv_state);
+       buf = le32buf_to_cpu(test, (__force const __le32 *)buf, dst_size / 
sizeof(u32));
        KUNIT_EXPECT_MEMEQ(test, buf, result->expected, dst_size);
   }

--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)

--
--
Thomas Zimmermann
Graphics Driver Developer
SUSE Software Solutions Germany GmbH
Frankenstrasse 146, 90461 Nuernberg, Germany
GF: Ivo Totev, Andrew Myers, Andrew McDonald, Boudien Moerman
HRB 36809 (AG Nuernberg)