The DRM core implements a format check when setting a framebuffer
for a plane. [1] Remove the unnecessary test from cirrus_fb_create().

Signed-off-by: Thomas Zimmermann <tzimmerm...@suse.de>
Link: 
https://elixir.bootlin.com/linux/v6.1/source/drivers/gpu/drm/drm_atomic.c#L629 
# [1]
---
 drivers/gpu/drm/tiny/cirrus.c | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/drivers/gpu/drm/tiny/cirrus.c b/drivers/gpu/drm/tiny/cirrus.c
index a483abc2e6ba..c1ffbbe1d545 100644
--- a/drivers/gpu/drm/tiny/cirrus.c
+++ b/drivers/gpu/drm/tiny/cirrus.c
@@ -559,10 +559,6 @@ static struct drm_framebuffer*
 cirrus_fb_create(struct drm_device *dev, struct drm_file *file_priv,
                 const struct drm_mode_fb_cmd2 *mode_cmd)
 {
-       if (mode_cmd->pixel_format != DRM_FORMAT_RGB565 &&
-           mode_cmd->pixel_format != DRM_FORMAT_RGB888 &&
-           mode_cmd->pixel_format != DRM_FORMAT_XRGB8888)
-               return ERR_PTR(-EINVAL);
        if (cirrus_check_size(mode_cmd->width, mode_cmd->height, NULL) < 0)
                return ERR_PTR(-EINVAL);
        return drm_gem_fb_create_with_dirty(dev, file_priv, mode_cmd);
-- 
2.39.1

Reply via email to