On Wed, Nov 27, 2019 at 06:31:59PM +0200, Jani Nikula wrote: > Deferred IO now preserves the fb_ops. > > Cc: Steve Glendinning <[email protected]> > Cc: [email protected] > Signed-off-by: Jani Nikula <[email protected]> > --- > drivers/video/fbdev/smscufx.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/drivers/video/fbdev/smscufx.c b/drivers/video/fbdev/smscufx.c > index 0e0f5bbfc5ef..e362d7da87fc 100644 > --- a/drivers/video/fbdev/smscufx.c > +++ b/drivers/video/fbdev/smscufx.c > @@ -1170,7 +1170,6 @@ static int ufx_ops_release(struct fb_info *info, int > user) > fb_deferred_io_cleanup(info); > kfree(info->fbdefio); > info->fbdefio = NULL; > - info->fbops->fb_mmap = ufx_ops_mmap;
Also totally pointless to restore this here, since if you have indeed loaded the driver twice shit has hit the fan already. I guess that was for the module option .... wtf. Reviewed-by: Daniel Vetter <[email protected]> > } > > pr_debug("released /dev/fb%d user=%d count=%d", > -- > 2.20.1 > > _______________________________________________ > Intel-gfx mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/intel-gfx -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch _______________________________________________ dri-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/dri-devel
