On Fri, Apr 06, 2018 at 10:35:00PM +0300, Ville Syrjälä wrote: > On Fri, Apr 06, 2018 at 07:14:51PM +0000, Deepak Singh Rawat wrote: > > This makes sense once we got rid of plane->fb > > > > Will this go to drm-next? > > The plan is to push to drm-misc-next once we get all > the ducks in a row. > > > Could you please CC > > me so that I can do some testing myself. Thanks. > > Here's a branch if you want a head start: > git://github.com/vsyrjala/linux.git plane_fb_crtc_nuke_2 > > I'd definitely appreciate some testing of this stuff. Wouldn't > want to break you stuff accidentally.
Did we get anywhere with testing this? I'd like to land the remaining bits, but I'd feel much safer doing that if it was tested. > > > > > Reviewed-by: Deepak Rawat <[email protected]> > > > > > > > > > > From: Ville Syrjälä <[email protected]> > > > > > > We want to get rid of plane->fb on atomic drivers. Stop setting it. > > > > > > Cc: Thomas Hellstrom <[email protected]> > > > Cc: Sinclair Yeh <[email protected]> > > > Cc: VMware Graphics <[email protected]> > > > Cc: Daniel Vetter <[email protected]> > > > Signed-off-by: Ville Syrjälä <[email protected]> > > > --- > > > drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c | 2 -- > > > drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c | 2 -- > > > 2 files changed, 4 deletions(-) > > > > > > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c > > > b/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c > > > index 648f8127f65a..bbd3f19b1a0b 100644 > > > --- a/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c > > > +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_scrn.c > > > @@ -525,8 +525,6 @@ vmw_sou_primary_plane_atomic_update(struct > > > drm_plane *plane, > > > */ > > > if (ret != 0) > > > DRM_ERROR("Failed to update screen.\n"); > > > - > > > - crtc->primary->fb = plane->state->fb; > > > } else { > > > /* > > > * When disabling a plane, CRTC and FB should always be > > > NULL > > > diff --git a/drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c > > > b/drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c > > > index 67331f01ef32..90445bc590cb 100644 > > > --- a/drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c > > > +++ b/drivers/gpu/drm/vmwgfx/vmwgfx_stdu.c > > > @@ -1285,8 +1285,6 @@ vmw_stdu_primary_plane_atomic_update(struct > > > drm_plane *plane, > > > 1, 1, NULL, crtc); > > > if (ret) > > > DRM_ERROR("Failed to update STDU.\n"); > > > - > > > - crtc->primary->fb = plane->state->fb; > > > } else { > > > crtc = old_state->crtc; > > > stdu = vmw_crtc_to_stdu(crtc); > > > -- > > > 2.16.1 > > -- > Ville Syrjälä > Intel OTC > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Ville Syrjälä Intel _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
