Re: [PATCH v1] staging: rtl8723bs: Remove boolean comparisons

[Dan Carpenter]

On Sat, Apr 18, 2026 at 10:49:16PM +0800, Yuma Ochi wrote:
> @@ -1950,8 +1950,8 @@ static int recv_indicatepkts_in_order(struct adapter *padapter, struct recv_reor
>  
>  			/* indicate this recv_frame */
>  			if (!pattrib->amsdu) {
> -				if ((padapter->bDriverStopped == false) &&
> -				    (padapter->bSurpriseRemoved == false))
> +				if ((!padapter->bDriverStopped) &&
> +				    (!padapter->bSurpriseRemoved))

Originally the parentheses were not a problem, but now there are too many.
Same thing in other places as well.

>  					rtw_recv_indicatepkt(padapter, prframe);/* indicate this recv_frame */
>  
>  			} else if (pattrib->amsdu == 1) {

regards,
dan carpenter