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@samsung.com
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@samsung.com Acked-by: Daniel Stone daniels@collabora.com Reviewed-by: Alex Deucher alexander.deucher@amd.com Signed-off-by: Daniel Vetter daniel.vetter@intel.com
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;
}