drm/amd/display: Remove else after return statement in 'dm_update_plane_state'
authorSrinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Mon, 19 Jun 2023 11:27:56 +0000 (16:57 +0530)
committerAlex Deucher <alexander.deucher@amd.com>
Fri, 7 Jul 2023 17:38:44 +0000 (13:38 -0400)
Else is not necessary after return statements, hence remove it.

Reported by checkpatch:

WARNING: else is not generally useful after a break or return
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c:9776:
                               return -EINVAL;
                       else

Cc: Bhawanpreet Lakha <Bhawanpreet.Lakha@amd.com>
Cc: Qingqing Zhuo <qingqing.zhuo@amd.com>
Cc: Nicholas Kazlauskas <Nicholas.Kazlauskas@amd.com>
Cc: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Cc: Aurabindo Pillai <aurabindo.pillai@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmugam@amd.com>
Reviewed-by: Christian König <christian.koenig@amd.com>
Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c

index 4bfd7c8..3beceec 100644 (file)
@@ -9758,8 +9758,8 @@ static int dm_update_plane_state(struct dc *dc,
                if (plane->type == DRM_PLANE_TYPE_OVERLAY) {
                        if (is_video_format(new_plane_state->fb->format->format) && *is_top_most_overlay)
                                return -EINVAL;
-                       else
-                               *is_top_most_overlay = false;
+
+                       *is_top_most_overlay = false;
                }
 
                DRM_DEBUG_ATOMIC("Enabling DRM plane: %d on DRM crtc %d\n",