From 1791848c0daf6676b79a866f0e604ea36daec911 Mon Sep 17 00:00:00 2001 From: Chaoyi Chen Date: Fri, 16 May 2025 14:25:29 +0800 Subject: [PATCH] drm/rockchip: vop: Remove redundant writeback cleanup The drm_writeback_queue_job() function takes ownership of the passed job and requires the caller to manually set the connector state writeback_job pointer to NULL. Therefore, there is no need to clear the writeback_job again. Related commit: 97eb9eaeb95b ("drm: writeback: Cleanup job ownership handling when queuing job") Change-Id: Ie56de7e3dc617daa629c4d52514ec511ecdfee53 Signed-off-by: Chaoyi Chen --- drivers/gpu/drm/rockchip/rockchip_drm_vop.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c index 0a11e344a4cc..ffa2476d7eeb 100644 --- a/drivers/gpu/drm/rockchip/rockchip_drm_vop.c +++ b/drivers/gpu/drm/rockchip/rockchip_drm_vop.c @@ -1877,7 +1877,6 @@ static void vop_wb_commit(struct drm_crtc *crtc) fb->pitches[0], &wb_state->yrgb_addr); drm_writeback_queue_job(wb_conn, conn_state); - conn_state->writeback_job = NULL; spin_lock_irqsave(&wb->job_lock, flags); wb->jobs[wb->job_index].pending = true;