staging: r8188eu: drop redundant if check in IS_MCAST
authorVihas Makwana <makvihas@gmail.com>
Tue, 5 Apr 2022 12:42:38 +0000 (18:12 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 5 Apr 2022 14:13:38 +0000 (16:13 +0200)
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>
drivers/staging/r8188eu/include/wifi.h

index a252a416d3e5509d06044cc00477eecc53be55df..b26770e91fc0ba84ae8055976f26e2bf3360bc16 100644 (file)
@@ -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)