Hi Daniel, It seems your patch is exactly same as below my one I posted before, http://www.spinics.net/lists/dri-devel/msg97922.html
Anyway, it's ok if this patch can go to mainline. Acked-by: Inki Dae <[email protected]> 2016년 01월 12일 06:41에 Daniel Vetter 이(가) 쓴 글: > The core takes care of this now. And since kfree(NULL) is ok we can > simplify the function even further now. > > Cc: Inki Dae <[email protected]> > Acked-by: Daniel Stone <[email protected]> > Reviewed-by: Alex Deucher <[email protected]> > Signed-off-by: Daniel Vetter <[email protected]> > --- > drivers/gpu/drm/exynos/exynos_drm_drv.c | 14 -------------- > 1 file changed, 14 deletions(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_drv.c > b/drivers/gpu/drm/exynos/exynos_drm_drv.c > index 9756797a15a5..868ab9f54f17 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_drv.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_drv.c > @@ -335,20 +335,6 @@ static void exynos_drm_preclose(struct drm_device *dev, > > static void exynos_drm_postclose(struct drm_device *dev, struct drm_file > *file) > { > - struct drm_pending_event *e, *et; > - unsigned long flags; > - > - if (!file->driver_priv) > - return; > - > - spin_lock_irqsave(&dev->event_lock, flags); > - /* Release all events handled by page flip handler but not freed. */ > - list_for_each_entry_safe(e, et, &file->event_list, link) { > - list_del(&e->link); > - e->destroy(e); > - } > - spin_unlock_irqrestore(&dev->event_lock, flags); > - > kfree(file->driver_priv); > file->driver_priv = NULL; > } > _______________________________________________ Intel-gfx mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/intel-gfx
