Reviewed-by: Lyude Paul <lyude@xxxxxxxxxx> I assume you'll handle pushing this yourself? (JFYI - using drm-misc for pushing this is fine by me) On Mon, 2022-02-21 at 10:59 +0100, Maxime Ripard wrote: > The nouveau KMS driver will call drm_plane_create_zpos_property() with > an init value depending on the plane purpose. > > Since the initial value wasn't carried over in the state, the driver had > to set it again in nv50_wndw_reset(). However, the helpers have been > adjusted to set it properly at reset, so this is not needed anymore. > > Cc: nouveau@xxxxxxxxxxxxxxxxxxxxx > Cc: Ben Skeggs <bskeggs@xxxxxxxxxx> > Cc: Karol Herbst <kherbst@xxxxxxxxxx> > Cc: Lyude Paul <lyude@xxxxxxxxxx> > Signed-off-by: Maxime Ripard <maxime@xxxxxxxxxx> > --- > drivers/gpu/drm/nouveau/dispnv50/wndw.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/gpu/drm/nouveau/dispnv50/wndw.c > b/drivers/gpu/drm/nouveau/dispnv50/wndw.c > index 133c8736426a..0c1a2ea0ed04 100644 > --- a/drivers/gpu/drm/nouveau/dispnv50/wndw.c > +++ b/drivers/gpu/drm/nouveau/dispnv50/wndw.c > @@ -635,8 +635,6 @@ nv50_wndw_reset(struct drm_plane *plane) > plane->funcs->atomic_destroy_state(plane, plane->state); > > __drm_atomic_helper_plane_reset(plane, &asyw->state); > - plane->state->zpos = nv50_wndw_zpos_default(plane); > - plane->state->normalized_zpos = nv50_wndw_zpos_default(plane); > } > > static void -- Cheers, Lyude Paul (she/her) Software Engineer at Red Hat