[Intel-gfx] [PATCH] drm/i915: Remove unecessary check for unsupported modifiers for NV12

[Maarten Lankhorst]

Op 28-06-18 om 08:18 schreef Dhinakaran Pandiyan:
> There is already a check to allow only RGB8888 formats with CCS
> modifiers.
>
> Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan at intel.com>
> ---
>  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",

Fair enough, probably rebase noise. :)

Reviewed-by: Maarten Lankhorst <maarten.lankhorst at linux.intel.com>