On Wed, Jun 27, 2018 at 11:18:54PM -0700, Dhinakaran Pandiyan wrote: > There is already a check to allow only RGB8888 formats with CCS > modifiers. > > Signed-off-by: Dhinakaran Pandiyan <[email protected]>
Reviewed-by: Daniel Vetter <[email protected]> Aside: intel_framebuffer_init is horribly long function, and it would probably make a lot of sense to extract all the framebuffer related stuff into a new intel_framebuffer.c file. intel_display.c is a bit ... big :-) -Daniel > --- > drivers/gpu/drm/i915/intel_display.c | 5 ----- > 1 file changed, 5 deletions(-) > > diff --git a/drivers/gpu/drm/i915/intel_display.c > b/drivers/gpu/drm/i915/intel_display.c > index eaa0663963a5..c5ec13e0f484 100644 > --- a/drivers/gpu/drm/i915/intel_display.c > +++ b/drivers/gpu/drm/i915/intel_display.c > @@ -14521,11 +14521,6 @@ static int intel_framebuffer_init(struct > intel_framebuffer *intel_fb, > } > break; > case DRM_FORMAT_NV12: > - if (mode_cmd->modifier[0] == I915_FORMAT_MOD_Y_TILED_CCS || > - mode_cmd->modifier[0] == I915_FORMAT_MOD_Yf_TILED_CCS) { > - DRM_DEBUG_KMS("RC not to be enabled with NV12\n"); > - goto err; > - } > if (INTEL_GEN(dev_priv) < 9 || IS_SKYLAKE(dev_priv) || > IS_BROXTON(dev_priv)) { > DRM_DEBUG_KMS("unsupported pixel format: %s\n", > -- > 2.14.1 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ Intel-gfx mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/intel-gfx
