lib/igt_fb: Pass the modifier to igt_fb_convert helpers
The modifier is part of how a frame is represented, so add it as an
extra argument so that it can be specified when converting framebuffers.
For now, only a linear modifier is supported.
Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Reviewed-by: Maxime Ripard <maxime.ripard@bootlin.com>
Reviewed-by: Lyude Paul <lyude@redhat.com>
diff --git a/lib/igt_fb.c b/lib/igt_fb.c
index 0b59ff4..25a07b0 100644
--- a/lib/igt_fb.c
+++ b/lib/igt_fb.c
@@ -2557,6 +2557,7 @@
* @dst: pointer to the #igt_fb structure that will store the conversion result
* @src: pointer to the #igt_fb structure that stores the frame we convert
* @dst_fourcc: DRM format specifier to convert to
+ * @dst_modifier: DRM format modifier to convert to
* @dst_stride: Stride for the resulting framebuffer (0 for automatic stride)
*
* This will convert a given @src content to the @dst_fourcc format,
@@ -2571,6 +2572,7 @@
*/
unsigned int igt_fb_convert_with_stride(struct igt_fb *dst, struct igt_fb *src,
uint32_t dst_fourcc,
+ uint64_t dst_modifier,
unsigned int dst_stride)
{
struct fb_convert cvt = { };
@@ -2579,8 +2581,8 @@
fb_id = igt_create_fb_with_bo_size(src->fd, src->width, src->height,
dst_fourcc,
- LOCAL_DRM_FORMAT_MOD_NONE,
- dst, 0, dst_stride);
+ LOCAL_DRM_FORMAT_MOD_NONE, dst, 0,
+ dst_stride);
igt_assert(fb_id > 0);
src_ptr = igt_fb_map_buffer(src->fd, src);
@@ -2598,6 +2600,8 @@
igt_fb_unmap_buffer(dst, dst_ptr);
igt_fb_unmap_buffer(src, src_ptr);
+ igt_assert(dst_modifier == LOCAL_DRM_FORMAT_MOD_NONE);
+
return fb_id;
}
@@ -2606,6 +2610,7 @@
* @dst: pointer to the #igt_fb structure that will store the conversion result
* @src: pointer to the #igt_fb structure that stores the frame we convert
* @dst_fourcc: DRM format specifier to convert to
+ * @dst_modifier: DRM format modifier to convert to
*
* This will convert a given @src content to the @dst_fourcc format,
* storing the result in the @dst fb, allocating the @dst fb
@@ -2618,9 +2623,10 @@
* The kms id of the created framebuffer.
*/
unsigned int igt_fb_convert(struct igt_fb *dst, struct igt_fb *src,
- uint32_t dst_fourcc)
+ uint32_t dst_fourcc, uint64_t dst_modifier)
{
- return igt_fb_convert_with_stride(dst, src, dst_fourcc, 0);
+ return igt_fb_convert_with_stride(dst, src, dst_fourcc, dst_modifier,
+ 0);
}
/**
diff --git a/lib/igt_fb.h b/lib/igt_fb.h
index 2c0a24d..e1d885e 100644
--- a/lib/igt_fb.h
+++ b/lib/igt_fb.h
@@ -138,9 +138,10 @@
uint32_t format, uint64_t tiling);
unsigned int igt_fb_convert_with_stride(struct igt_fb *dst, struct igt_fb *src,
uint32_t dst_fourcc,
+ uint64_t dst_modifier,
unsigned int stride);
unsigned int igt_fb_convert(struct igt_fb *dst, struct igt_fb *src,
- uint32_t dst_fourcc);
+ uint32_t dst_fourcc, uint64_t dst_modifier);
void igt_remove_fb(int fd, struct igt_fb *fb);
int igt_dirty_fb(int fd, struct igt_fb *fb);
void *igt_fb_map_buffer(int fd, struct igt_fb *fb);
diff --git a/tests/kms_chamelium.c b/tests/kms_chamelium.c
index 64f87d3..fe58aea 100644
--- a/tests/kms_chamelium.c
+++ b/tests/kms_chamelium.c
@@ -546,7 +546,8 @@
DRM_FORMAT_XRGB8888, 64, &fb);
igt_assert(fb_id > 0);
- frame_id = igt_fb_convert(&frame_fb, &fb, fourcc);
+ frame_id = igt_fb_convert(&frame_fb, &fb, fourcc,
+ LOCAL_DRM_FORMAT_MOD_NONE);
igt_assert(frame_id > 0);
if (check == CHAMELIUM_CHECK_CRC)