Commit e88231fe authored by Fabio Aiuto's avatar Fabio Aiuto Committed by Greg Kroah-Hartman
Browse files

staging: rtl8723bs: put parentheses on macros with complex values in include/wifi.h 



fix the following checkpatch warnings:

ERROR: Macros with complex values should be enclosed in parentheses
114: FILE: drivers/staging/rtl8723bs/include/wifi.h:114:
+#define SetToDs(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
122: FILE: drivers/staging/rtl8723bs/include/wifi.h:122:
+#define SetFrDs(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
132: FILE: drivers/staging/rtl8723bs/include/wifi.h:132:
+#define SetMFrag(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
137: FILE: drivers/staging/rtl8723bs/include/wifi.h:137:
+#define ClearMFrag(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
145: FILE: drivers/staging/rtl8723bs/include/wifi.h:145:
+#define ClearRetry(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
148: FILE: drivers/staging/rtl8723bs/include/wifi.h:148:
+#define SetPwrMgt(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
153: FILE: drivers/staging/rtl8723bs/include/wifi.h:153:
+#define ClearPwrMgt(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
156: FILE: drivers/staging/rtl8723bs/include/wifi.h:156:
+#define SetMData(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
161: FILE: drivers/staging/rtl8723bs/include/wifi.h:161:
+#define ClearMData(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
164: FILE: drivers/staging/rtl8723bs/include/wifi.h:164:
+#define SetPrivacy(pbuf)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
220: FILE: drivers/staging/rtl8723bs/include/wifi.h:220:
+#define SetDuration(pbuf, dur) \
--
ERROR: Macros with complex values should be enclosed in parentheses
224: FILE: drivers/staging/rtl8723bs/include/wifi.h:224:
+#define SetPriority(pbuf, tid)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
229: FILE: drivers/staging/rtl8723bs/include/wifi.h:229:
+#define SetEOSP(pbuf, eosp)	\
--
ERROR: Macros with complex values should be enclosed in parentheses
232: FILE: drivers/staging/rtl8723bs/include/wifi.h:232:
+#define SetAckpolicy(pbuf, ack)	\

Signed-off-by: default avatarFabio Aiuto <fabioaiuto83@gmail.com>
Link: https://lore.kernel.org/r/99d3e605501c1792035e7403d6da86243e6b48cb.1616854134.git.fabioaiuto83@gmail.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 78a62638

drivers/staging/rtl8723bs/include/wifi.h

+14 −14
Original line number Diff line number Diff line
@@ -98,48 +98,48 @@ enum { 
#define _ORDER_			BIT(15)



#define SetToDs(pbuf)	\

	*(__le16 *)(pbuf) |= cpu_to_le16(_TO_DS_)

	(*(__le16 *)(pbuf) |= cpu_to_le16(_TO_DS_))



#define GetToDs(pbuf)	(((*(__le16 *)(pbuf)) & cpu_to_le16(_TO_DS_)) != 0)



#define SetFrDs(pbuf)	\

	*(__le16 *)(pbuf) |= cpu_to_le16(_FROM_DS_)

	(*(__le16 *)(pbuf) |= cpu_to_le16(_FROM_DS_))



#define GetFrDs(pbuf)	(((*(__le16 *)(pbuf)) & cpu_to_le16(_FROM_DS_)) != 0)



#define get_tofr_ds(pframe)	((GetToDs(pframe) << 1) | GetFrDs(pframe))



#define SetMFrag(pbuf)	\

	*(__le16 *)(pbuf) |= cpu_to_le16(_MORE_FRAG_)

	(*(__le16 *)(pbuf) |= cpu_to_le16(_MORE_FRAG_))



#define GetMFrag(pbuf)	(((*(__le16 *)(pbuf)) & cpu_to_le16(_MORE_FRAG_)) != 0)



#define ClearMFrag(pbuf)	\

	*(__le16 *)(pbuf) &= (~cpu_to_le16(_MORE_FRAG_))

	(*(__le16 *)(pbuf) &= (~cpu_to_le16(_MORE_FRAG_)))



#define GetRetry(pbuf)	(((*(__le16 *)(pbuf)) & cpu_to_le16(_RETRY_)) != 0)



#define ClearRetry(pbuf)	\

	*(__le16 *)(pbuf) &= (~cpu_to_le16(_RETRY_))

	(*(__le16 *)(pbuf) &= (~cpu_to_le16(_RETRY_)))



#define SetPwrMgt(pbuf)	\

	*(__le16 *)(pbuf) |= cpu_to_le16(_PWRMGT_)

	(*(__le16 *)(pbuf) |= cpu_to_le16(_PWRMGT_))



#define GetPwrMgt(pbuf)	(((*(__le16 *)(pbuf)) & cpu_to_le16(_PWRMGT_)) != 0)



#define ClearPwrMgt(pbuf)	\

	*(__le16 *)(pbuf) &= (~cpu_to_le16(_PWRMGT_))

	(*(__le16 *)(pbuf) &= (~cpu_to_le16(_PWRMGT_)))



#define SetMData(pbuf)	\

	*(__le16 *)(pbuf) |= cpu_to_le16(_MORE_DATA_)

	(*(__le16 *)(pbuf) |= cpu_to_le16(_MORE_DATA_))



#define GetMData(pbuf)	(((*(__le16 *)(pbuf)) & cpu_to_le16(_MORE_DATA_)) != 0)



#define ClearMData(pbuf)	\

	*(__le16 *)(pbuf) &= (~cpu_to_le16(_MORE_DATA_))

	(*(__le16 *)(pbuf) &= (~cpu_to_le16(_MORE_DATA_)))



#define SetPrivacy(pbuf)	\

	*(__le16 *)(pbuf) |= cpu_to_le16(_PRIVACY_)

	(*(__le16 *)(pbuf) |= cpu_to_le16(_PRIVACY_))



#define GetPrivacy(pbuf)					\

	(((*(__le16 *)(pbuf)) & cpu_to_le16(_PRIVACY_)) != 0)
@@ -191,19 +191,19 @@ enum { 
	} while (0)



#define SetDuration(pbuf, dur) \

	*(__le16 *)((size_t)(pbuf) + 2) = cpu_to_le16(0xffff & (dur))

	(*(__le16 *)((size_t)(pbuf) + 2) = cpu_to_le16(0xffff & (dur)))





#define SetPriority(pbuf, tid)	\

	*(__le16 *)(pbuf) |= cpu_to_le16(tid & 0xf)

	(*(__le16 *)(pbuf) |= cpu_to_le16(tid & 0xf))



#define GetPriority(pbuf)	((le16_to_cpu(*(__le16 *)(pbuf))) & 0xf)



#define SetEOSP(pbuf, eosp)	\

		*(__le16 *)(pbuf) |= cpu_to_le16((eosp & 1) << 4)

		(*(__le16 *)(pbuf) |= cpu_to_le16((eosp & 1) << 4))



#define SetAckpolicy(pbuf, ack)	\

	*(__le16 *)(pbuf) |= cpu_to_le16((ack & 3) << 5)

	(*(__le16 *)(pbuf) |= cpu_to_le16((ack & 3) << 5))



#define GetAckpolicy(pbuf) (((le16_to_cpu(*(__le16 *)pbuf)) >> 5) & 0x3)