Re: [PATCH 2/2] drm/i915: Reject compressed Y/Yf with interlaced modes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
On Tue, Aug 28, 2018 at 7:57 PM Ville Syrjala
<ville.syrjala@xxxxxxxxxxxxxxx> wrote:
>
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
>
> Y/Yf tiling can't be used with IF-ID. We already reject uncompressed
> Y/Yf but we should also reject them when compressed.
>
> Cc: José Roberto de Souza <jose.souza@xxxxxxxxx>
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/intel_atomic_plane.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/i915/intel_atomic_plane.c b/drivers/gpu/drm/i915/intel_atomic_plane.c
> index eddcdd6e4b3b..fa7df5fe154b 100644
> --- a/drivers/gpu/drm/i915/intel_atomic_plane.c
> +++ b/drivers/gpu/drm/i915/intel_atomic_plane.c
> @@ -170,7 +170,9 @@ int intel_plane_atomic_check_with_state(const struct intel_crtc_state *old_crtc_
>         if (state->fb && INTEL_GEN(dev_priv) >= 9 && crtc_state->base.enable &&
>             adjusted_mode->flags & DRM_MODE_FLAG_INTERLACE) {
>                 if (state->fb->modifier == I915_FORMAT_MOD_Y_TILED ||
> -                   state->fb->modifier == I915_FORMAT_MOD_Yf_TILED) {
> +                   state->fb->modifier == I915_FORMAT_MOD_Yf_TILED ||
> +                   state->fb->modifier == I915_FORMAT_MOD_Y_TILED_CCS ||
> +                   state->fb->modifier == I915_FORMAT_MOD_Yf_TILED_CCS) {
LGTM...
Reviewed-by:  Mahesh Kumar <mahesh1.sh.kumar@xxxxxxxxx>

-Mahesh
>                         DRM_DEBUG_KMS("Y/Yf tiling not supported in IF-ID mode\n");
>                         return -EINVAL;
>                 }
> --
> 2.16.4
>
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> https://lists.freedesktop.org/mailman/listinfo/intel-gfx
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux