Blob Blame History Raw
From 98fe05e21a6e0ca242e974650ed58b64813cb2dc Mon Sep 17 00:00:00 2001
From: Sebastian Arriola <sebdeveloper6952@gmail.com>
Date: Mon, 15 Jun 2020 09:51:31 -0600
Subject: [PATCH] staging: rtl8712: Remove unnecesary else after return statement.
Git-commit: 98fe05e21a6e0ca242e974650ed58b64813cb2dc
Patch-mainline: v5.9-rc1
References: jsc#SLE-13430

This patch fixes the checkpatch.pl warning:

Warning: else is not generally useful after a break or return
490: FILE: drivers/staging/rtl8712/rtl8712_recv.c:490:			return false;		else

Signed-off-by: Sebastian Arriola <sebdeveloper6952@gmail.com>
Link: https://lore.kernel.org/r/20200615155131.GA4563@sevic69
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Acked-by: Takashi Iwai <tiwai@suse.de>

---
 drivers/staging/rtl8712/rtl8712_recv.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/rtl8712/rtl8712_recv.c b/drivers/staging/rtl8712/rtl8712_recv.c
index fe6694f4d5e4..0bb49bcb9f1a 100644
--- a/drivers/staging/rtl8712/rtl8712_recv.c
+++ b/drivers/staging/rtl8712/rtl8712_recv.c
@@ -482,8 +482,7 @@ static int enqueue_reorder_recvframe(struct recv_reorder_ctrl *preorder_ctrl,
 			plist = plist->next;
 		else if (SN_EQUAL(pnextattrib->seq_num, pattrib->seq_num))
 			return false;
-		else
-			break;
+		break;
 	}
 	list_del_init(&(prframe->u.hdr.list));
 	list_add_tail(&(prframe->u.hdr.list), plist);
-- 
2.16.4