On 12/20/22 13:11, Thomas Zimmermann wrote: > Change the source-buffer type of le32buf_to_cpu() to __le32* to > reflect endianness. Result buffers are converted to local endianness, > so instantiate them from regular u8 or u32 types. > > Signed-off-by: Thomas Zimmermann <tzimmermann@xxxxxxx> Reviewed-by: Maíra Canal <mcanal@xxxxxxxxxx> Best Regards, - Maíra Canal > --- > drivers/gpu/drm/tests/drm_format_helper_test.c | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/drivers/gpu/drm/tests/drm_format_helper_test.c b/drivers/gpu/drm/tests/drm_format_helper_test.c > index 04fe373c9d97..c2411ec808a1 100644 > --- a/drivers/gpu/drm/tests/drm_format_helper_test.c > +++ b/drivers/gpu/drm/tests/drm_format_helper_test.c > @@ -279,7 +279,7 @@ static u16 *le16buf_to_cpu(struct kunit *test, const __le16 *buf, size_t buf_siz > return dst; > } > > -static u32 *le32buf_to_cpu(struct kunit *test, const u32 *buf, size_t buf_size) > +static u32 *le32buf_to_cpu(struct kunit *test, const __le32 *buf, size_t buf_size) > { > u32 *dst = NULL; > int n; > @@ -323,7 +323,7 @@ static void drm_test_fb_xrgb8888_to_gray8(struct kunit *test) > const struct convert_xrgb8888_case *params = test->param_value; > const struct convert_to_gray8_result *result = ¶ms->gray8_result; > size_t dst_size; > - __u8 *buf = NULL; > + u8 *buf = NULL; > __le32 *xrgb8888 = NULL; > struct iosys_map dst, src; > > @@ -353,7 +353,7 @@ static void drm_test_fb_xrgb8888_to_rgb332(struct kunit *test) > const struct convert_xrgb8888_case *params = test->param_value; > const struct convert_to_rgb332_result *result = ¶ms->rgb332_result; > size_t dst_size; > - __u8 *buf = NULL; > + u8 *buf = NULL; > __le32 *xrgb8888 = NULL; > struct iosys_map dst, src; > > @@ -419,7 +419,7 @@ static void drm_test_fb_xrgb8888_to_rgb888(struct kunit *test) > const struct convert_xrgb8888_case *params = test->param_value; > const struct convert_to_rgb888_result *result = ¶ms->rgb888_result; > size_t dst_size; > - __u8 *buf = NULL; > + u8 *buf = NULL; > __le32 *xrgb8888 = NULL; > struct iosys_map dst, src; > > @@ -453,7 +453,7 @@ static void drm_test_fb_xrgb8888_to_xrgb2101010(struct kunit *test) > const struct convert_xrgb8888_case *params = test->param_value; > const struct convert_to_xrgb2101010_result *result = ¶ms->xrgb2101010_result; > size_t dst_size; > - __u32 *buf = NULL; > + u32 *buf = NULL; > __le32 *xrgb8888 = NULL; > struct iosys_map dst, src; > > @@ -475,7 +475,7 @@ static void drm_test_fb_xrgb8888_to_xrgb2101010(struct kunit *test) > iosys_map_set_vaddr(&src, xrgb8888); > > drm_fb_xrgb8888_to_xrgb2101010(&dst, &result->dst_pitch, &src, &fb, ¶ms->clip); > - buf = le32buf_to_cpu(test, buf, dst_size / sizeof(u32)); > + buf = le32buf_to_cpu(test, (__force const __le32 *)buf, dst_size / sizeof(u32)); > KUNIT_EXPECT_EQ(test, memcmp(buf, result->expected, dst_size), 0); > } >