Re: [RFT PATCH v3 3/5] staging: rtl8723bs: simplify boolean return in IsFrameTypeCtrl()

[Luka Gejak]

On April 5, 2026 1:41:30 PM GMT+02:00, Prithvi Tambewagh <activprithvi@xxxxxxxxx> wrote:
>Replace the simple if-else statement which checked value of
>GetFrameType(pframe), if equal to WIFI_CTRL_TYPE, function
>IsFrameTypeCtrl() returned true else false, with a single return
>statement returning true only if GetFrameType(pframe) == WIFI_CTRL_TYPE
>otherwise returns false.
>
>Signed-off-by: Prithvi Tambewagh <activprithvi@xxxxxxxxx>
>---
> drivers/staging/rtl8723bs/include/wifi.h | 5 +----
> 1 file changed, 1 insertion(+), 4 deletions(-)
>
>diff --git a/drivers/staging/rtl8723bs/include/wifi.h b/drivers/staging/rtl8723bs/include/wifi.h
>index 230b2c4ffd3b..2876f15f13d1 100644
>--- a/drivers/staging/rtl8723bs/include/wifi.h
>+++ b/drivers/staging/rtl8723bs/include/wifi.h
>@@ -275,10 +275,7 @@ static inline unsigned char *get_hdr_bssid(unsigned char *pframe)
> 
> static inline int IsFrameTypeCtrl(unsigned char *pframe)
> {
>-	if (WIFI_CTRL_TYPE == GetFrameType(pframe))
>-		return true;
>-	else
>-		return false;
>+	return GetFrameType(pframe) == WIFI_CTRL_TYPE;
> }
> /*-----------------------------------------------------------------------------
> 			Below is for the security related definition

LGTM,
Reviewed-by: Luka Gejak <luka.gejak@xxxxxxxxx>