staging: rtl8723bs: move logical operators to end of previous line

nikolayof12 · gregkh · commit 4fa02e833e0d · 2026-03-31T10:31:18.000+02:00
Change the position of logical operators '||' and remove unnecessary curly braces around single expression to fix checkpatch.pl warnings. Signed-off-by: Nikolay Kulikov <nikolayof23@gmail.com> Link: https://patch.msgid.link/20260323150650.7168-4-nikolayof23@gmail.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
diff --git a/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c b/drivers/staging/rtl8723bs/core/rtw_ioctl_set.c
@@ -11,12 +11,10 @@ u8 rtw_validate_bssid(u8 *bssid)
 {
 	u8 ret = true;
 
-	if (is_zero_mac_addr(bssid)
-		|| is_broadcast_mac_addr(bssid)
-		|| is_multicast_mac_addr(bssid)
-	) {
+	if (is_zero_mac_addr(bssid) ||
+	    is_broadcast_mac_addr(bssid) ||
+	    is_multicast_mac_addr(bssid))
 		ret = false;
-	}
 
 	return ret;
 }

Original file line number	Diff line number	Diff line change
`@@ -11,12 +11,10 @@ u8 rtw_validate_bssid(u8 *bssid)`
`11`	`11`	`{`
`12`	`12`	`u8 ret = true;`
`13`	`13`
`14`		`- if (is_zero_mac_addr(bssid)`
`15`		`- \|\| is_broadcast_mac_addr(bssid)`
`16`		`- \|\| is_multicast_mac_addr(bssid)`
`17`		`- ) {`
	`14`	`+ if (is_zero_mac_addr(bssid) \|\|`
	`15`	`+ is_broadcast_mac_addr(bssid) \|\|`
	`16`	`+ is_multicast_mac_addr(bssid))`
`18`	`17`	`ret = false;`
`19`		`- }`
`20`	`18`
`21`	`19`	`return ret;`
`22`	`20`	`}`