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 <inki.dae@xxxxxxxxxxx> 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 <inki.dae@xxxxxxxxxxx> > Acked-by: Daniel Stone <daniels@xxxxxxxxxxxxx> > Reviewed-by: Alex Deucher <alexander.deucher@xxxxxxx> > Signed-off-by: Daniel Vetter <daniel.vetter@xxxxxxxxx> > --- > 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; > } > _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
