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.la...@amd.com>
Cc: Qingqing Zhuo <qingqing.z...@amd.com>
Cc: Nicholas Kazlauskas <nicholas.kazlaus...@amd.com>
Cc: Rodrigo Siqueira <rodrigo.sique...@amd.com>
Cc: Aurabindo Pillai <aurabindo.pil...@amd.com>
Signed-off-by: Srinivasan Shanmugam <srinivasan.shanmu...@amd.com>
---
 drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c 
b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
index 2446529c329a..e657214e0104 100644
--- a/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
+++ b/drivers/gpu/drm/amd/display/amdgpu_dm/amdgpu_dm.c
@@ -9777,8 +9777,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",
-- 
2.25.1

Reply via email to