]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
staging: rtl8723bs: simplify return checks in validate_recv_data_frame()
authorGiacomo Di Clerico <giacomodiclerico@gmail.com>
Tue, 17 Mar 2026 10:07:23 +0000 (11:07 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 18 Mar 2026 15:56:58 +0000 (16:56 +0100)
Combine the return value checks for _FAIL and RTW_RX_HANDLED into a
single logical OR statement and remove unnecessary braces. This improves
code readability and resolves the following checkpatch.pl warning:
"WARNING: braces {} are not necessary for any arm of this statement"

Signed-off-by: Giacomo Di Clerico <giacomodiclerico@gmail.com>
Link: https://patch.msgid.link/20260317100723.72476-1-giacomodiclerico@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/rtl8723bs/core/rtw_recv.c

index 1a52d3e1d285d77ac0f83e09ff4435a9a7efa0bb..a52884d2129ffd4dbe59ded8d6906c97bad12456 100644 (file)
@@ -1339,11 +1339,8 @@ static signed int validate_recv_data_frame(struct adapter *adapter, union recv_f
 
        }
 
-       if (ret == _FAIL) {
+       if (ret == _FAIL || ret == RTW_RX_HANDLED)
                goto exit;
-       } else if (ret == RTW_RX_HANDLED) {
-               goto exit;
-       }
 
 
        if (!psta) {