staging: r8188eu: drop redundant if check in IS_MCAST

MakwanaVihas · gregkh · commit a461466e60d2 · 2022-04-05T16:13:38.000+02:00
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
@@ -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)

Original file line number	Diff line number	Diff line change
`@@ -240,10 +240,7 @@ enum WIFI_REG_DOMAIN {`
`240`	`240`
`241`	`241`	`static inline bool IS_MCAST(unsigned char *da)`
`242`	`242`	`{`
`243`		`- if ((*da) & 0x01)`
`244`		`- return true;`
`245`		`- else`
`246`		`- return false;`
	`243`	`+ return (*da) & 0x01;`
`247`	`244`	`}`
`248`	`245`
`249`	`246`	`static inline unsigned char get_da(unsigned char pframe)`