drm: vkms: Replace hardcoded value of `vkms_composer.map` to DRM_FORMAT_MAX_PLANES
authorIgor Torrente <igormtorrente@gmail.com>
Mon, 5 Sep 2022 19:08:03 +0000 (16:08 -0300)
committerMelissa Wen <melissa.srw@gmail.com>
Mon, 5 Sep 2022 21:12:14 +0000 (20:12 -0100)
The `map` vector at `vkms_composer` uses a hardcoded value to define its
size.

If someday the maximum number of planes increases, this hardcoded value
can be a problem.

This value is being replaced with the DRM_FORMAT_MAX_PLANES macro.

Acked-by: Thomas Zimmermann <tzimmermann@suse.de>
Reviewed-by: Melissa Wen <mwen@igalia.com>
Signed-off-by: Igor Torrente <igormtorrente@gmail.com>
Signed-off-by: Melissa Wen <melissa.srw@gmail.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20220905190811.25024-2-igormtorrente@gmail.com
drivers/gpu/drm/vkms/vkms_drv.h

index 1d60654b553b282348f7ea70a87ea2e8153d3447..ae6c5a3d356c7a74e88884c2e813df1d08e1e315 100644 (file)
@@ -31,7 +31,7 @@ struct vkms_writeback_job {
 struct vkms_composer {
        struct drm_framebuffer fb;
        struct drm_rect src, dst;
-       struct iosys_map map[4];
+       struct iosys_map map[DRM_FORMAT_MAX_PLANES];
        unsigned int offset;
        unsigned int pitch;
        unsigned int cpp;