Now that framebuffer_check() verifies that the format is properly
supported, there is no need to check it again on i915's inside
helpers.

Therefore, remove the redundant framebuffer format check from the
intel_framebuffer_init() function, letting framebuffer_check()
perform the framebuffer validation.

Signed-off-by: Maíra Canal <mca...@igalia.com>
---
 drivers/gpu/drm/i915/display/intel_fb.c | 9 ---------
 1 file changed, 9 deletions(-)

diff --git a/drivers/gpu/drm/i915/display/intel_fb.c 
b/drivers/gpu/drm/i915/display/intel_fb.c
index 63137ae5ab21..230b729e42d6 100644
--- a/drivers/gpu/drm/i915/display/intel_fb.c
+++ b/drivers/gpu/drm/i915/display/intel_fb.c
@@ -1914,15 +1914,6 @@ int intel_framebuffer_init(struct intel_framebuffer 
*intel_fb,
                }
        }
 
-       if (!drm_any_plane_has_format(&dev_priv->drm,
-                                     mode_cmd->pixel_format,
-                                     mode_cmd->modifier[0])) {
-               drm_dbg_kms(&dev_priv->drm,
-                           "unsupported pixel format %p4cc / modifier 
0x%llx\n",
-                           &mode_cmd->pixel_format, mode_cmd->modifier[0]);
-               goto err;
-       }
-
        /*
         * gen2/3 display engine uses the fence if present,
         * so the tiling mode must match the fb modifier exactly.
-- 
2.39.0

Reply via email to