Re: [PATCH] drm/i915: Fix the state checker for ICL Y planes

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

 



On Mon, Mar 04, 2019 at 03:12:17PM +0200, Ville Syrjala wrote:
> From: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> 
> The plane used to scan out NV12 luma on ICL is logically
> off but actually on. Fix the state checker to account for this.
> 
> Cc: Imre Deak <imre.deak@xxxxxxxxx>
> Cc: Maarten Lankhorst <maarten.lankhorst@xxxxxxxxxxxxxxx>
> Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=109457
> Signed-off-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx>
> ---
>  drivers/gpu/drm/i915/intel_display.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/gpu/drm/i915/intel_display.c b/drivers/gpu/drm/i915/intel_display.c
> index 7c5e84ef5171..b49e789b747e 100644
> --- a/drivers/gpu/drm/i915/intel_display.c
> +++ b/drivers/gpu/drm/i915/intel_display.c
> @@ -12589,7 +12589,8 @@ intel_verify_planes(struct intel_atomic_state *state)
>  
>  	for_each_new_intel_plane_in_state(state, plane,
>  					  plane_state, i)
> -		assert_plane(plane, plane_state->base.visible);
> +		assert_plane(plane, plane_state->slave ||
> +			     plane_state->base.visible);

Looks like we don't have proper readout for such linked planes either,
but that's a separate issue. The change looks ok:

Reviewed-by: Imre Deak <imre.deak@xxxxxxxxx>

>  }
>  
>  static void
> -- 
> 2.19.2
> 
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




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

  Powered by Linux