drm/msm/dpu: Drop extraneous return in dpu_crtc_reassign_planes()

Jessica Zhang · Abhinav Kumar · commit 5e192eefebaa · 2025-02-15T11:46:42.000-08:00
Drop extra return at the end of dpu_crtc_reassign_planes() Fixes: 774bcfb ("drm/msm/dpu: add support for virtual planes") Signed-off-by: Jessica Zhang <quic_jesszhan@quicinc.com> Reviewed-by: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Patchwork: https://patchwork.freedesktop.org/patch/631565/ Link: https://lore.kernel.org/r/20250108-virtual-planes-fixes-v1-2-420cb36df94a@quicinc.com Signed-off-by: Abhinav Kumar <quic_abhinavk@quicinc.com>
diff --git a/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c b/drivers/gpu/drm/msm/disp/dpu1/dpu_crtc.c
@@ -1228,8 +1228,6 @@ static int dpu_crtc_reassign_planes(struct drm_crtc *crtc, struct drm_crtc_state
 done:
 	kfree(states);
 	return ret;
-
-	return 0;
 }
 
 static int dpu_crtc_atomic_check(struct drm_crtc *crtc,

Original file line number	Diff line number	Diff line change
`@@ -1228,8 +1228,6 @@ static int dpu_crtc_reassign_planes(struct drm_crtc *crtc, struct drm_crtc_state`
`1228`	`1228`	`done:`
`1229`	`1229`	`kfree(states);`
`1230`	`1230`	`return ret;`
`1231`		`-`
`1232`		`- return 0;`
`1233`	`1231`	`}`
`1234`	`1232`
`1235`	`1233`	`static int dpu_crtc_atomic_check(struct drm_crtc *crtc,`