From: Vihas Makwana <makvihas@gmail.com>
Date: Tue, 5 Apr 2022 12:42:38 +0000 (+0530)
Subject: staging: r8188eu: drop redundant if check in IS_MCAST
X-Git-Tag: v6.6.17~7348^2~220
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=a461466e60d2a009ce83ac7a25d931c1ec5e17ad;p=platform%2Fkernel%2Flinux-rpi.git

staging: r8188eu: drop redundant if check in IS_MCAST

The if check is redundant. Drop it and simplify the funciton.

Signed-off-by: Vihas Makwana <makvihas@gmail.com>
Link: https://lore.kernel.org/r/20220405124239.3372-2-makvihas@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/staging/r8188eu/include/wifi.h b/drivers/staging/r8188eu/include/wifi.h
index a252a416d3e5..b26770e91fc0 100644
--- a/drivers/staging/r8188eu/include/wifi.h
+++ b/drivers/staging/r8188eu/include/wifi.h
@@ -240,10 +240,7 @@ enum WIFI_REG_DOMAIN {
 
 static inline bool IS_MCAST(unsigned char *da)
 {
-	if ((*da) & 0x01)
-		return true;
-	else
-		return false;
+	return (*da) & 0x01;
 }
 
 static inline unsigned char *get_da(unsigned char *pframe)