[igt-dev] [PATCH i-g-t 2/2] lib/igt_kms: reset plane alpha and blend in igt_plane_reset()

[Ville Syrjälä]

On Mon, Mar 18, 2019 at 02:42:11PM -0700, Dhinakaran Pandiyan wrote:
> On Mon, 2019-03-18 at 16:32 +0200, Ville Syrjälä wrote:
> > On Mon, Mar 18, 2019 at 01:15:10PM +0200, Juha-Pekka Heikkila wrote:
> > > plane alpha and blend would affect other tests if left
> > > in unexpected states
> > > 
> > > Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila at gmail.com>
> > > ---
> > >  lib/igt_kms.c | 7 +++++++
> > >  1 file changed, 7 insertions(+)
> > > 
> > > diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> > > index e1eacc1..cdb5aba 100644
> > > --- a/lib/igt_kms.c
> > > +++ b/lib/igt_kms.c
> > > @@ -1781,6 +1781,13 @@ static void igt_plane_reset(igt_plane_t
> > > *plane)
> > >  	if (igt_plane_has_prop(plane, IGT_PLANE_ROTATION))
> > >  		igt_plane_set_prop_value(plane, IGT_PLANE_ROTATION,
> > > IGT_ROTATION_0);
> > >  
> > > +	if (igt_plane_has_prop(plane, IGT_PLANE_PIXEL_BLEND_MODE))
> > > +		igt_plane_set_prop_enum(plane,
> > > IGT_PLANE_PIXEL_BLEND_MODE, "None");
> > 
> > "Pre-multiplied" is the default value for this prop.
> 
> You're right, drm_plane_create_blend_mode_property() documents "Pre-
> multiplied" as the default value and reset_alpha() in the alpha
> blending test does the same.
> 
> On a related note, I see that skl_plane_ctl_alpha() has
>         default:
>                 MISSING_CASE(plane_state->base.pixel_blend_mode);
>                 return PLANE_CTL_ALPHA_DISABLE;
> 
> Shouldn't the fallback here be PLANE_CTL_ALPHA_SW_PREMULTIPLY?

MISSING_CASE so doens't matter.

> 
> > 
> > > +
> > > +	if (igt_plane_has_prop(plane, IGT_PLANE_ALPHA))
> > > +		igt_plane_set_prop_value(plane, IGT_PLANE_ALPHA,
> > > 0xffff);
> > > +
> > > +
> > >  	igt_plane_clear_prop_changed(plane, IGT_PLANE_IN_FENCE_FD);
> > >  	plane->values[IGT_PLANE_IN_FENCE_FD] = ~0ULL;
> > >  	plane->gem_handle = 0;
> > > -- 
> > > 2.7.4
> > > 
> > > _______________________________________________
> > > igt-dev mailing list
> > > igt-dev at lists.freedesktop.org
> > > https://lists.freedesktop.org/mailman/listinfo/igt-dev
> > 
> > 

-- 
Ville Syrjälä
Intel