[PATCH] staging: rtl8723bs: simplify return checks in validate_recv_data_frame()
From: Giacomo Di Clerico
Date: Tue Mar 17 2026 - 06:13:44 EST
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@xxxxxxxxx>
---
drivers/staging/rtl8723bs/core/rtw_recv.c | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/staging/rtl8723bs/core/rtw_recv.c b/drivers/staging/rtl8723bs/core/rtw_recv.c
index 1a52d3e1d285..a52884d2129f 100644
--- a/drivers/staging/rtl8723bs/core/rtw_recv.c
+++ b/drivers/staging/rtl8723bs/core/rtw_recv.c
@@ -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) {
--
2.53.0