* [PATCH] staging: rtl8723bs: simplify return checks in validate_recv_data_frame()
@ 2026-03-17 10:07 Giacomo Di Clerico
0 siblings, 0 replies; only message in thread
From: Giacomo Di Clerico @ 2026-03-17 10:07 UTC (permalink / raw)
To: gregkh
Cc: hansg, straube.linux, andriy.shevchenko, khushalchitturi,
s9430939, 02khasar, ethantidmore06, linux-staging, linux-kernel,
Giacomo Di Clerico
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>
---
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
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2026-03-17 10:07 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-03-17 10:07 [PATCH] staging: rtl8723bs: simplify return checks in validate_recv_data_frame() Giacomo Di Clerico
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®