Commit 39b2ef70 authored by Tim Collier's avatar Tim Collier Committed by Greg Kroah-Hartman

staging: wlan-ng: add parentheses to macro argument usage in prism2mgmt.c

Fix two "CHECK: Macro argument 'N' may be better as '(N)' to avoid
precedence issue" messages, reported by checkpatch, by adding
parentheses around the offending macro argument references.
Signed-off-by: default avatarTim Collier <osdevtc@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent b2679009
...@@ -439,7 +439,7 @@ int prism2mgmt_scan_results(struct wlandevice *wlandev, void *msgp) ...@@ -439,7 +439,7 @@ int prism2mgmt_scan_results(struct wlandevice *wlandev, void *msgp)
#define REQBASICRATE(N) \ #define REQBASICRATE(N) \
do { \ do { \
if ((count >= N) && DOT11_RATE5_ISBASIC_GET( \ if ((count >= (N)) && DOT11_RATE5_ISBASIC_GET( \
item->supprates[(N) - 1])) { \ item->supprates[(N) - 1])) { \
req->basicrate ## N .data = item->supprates[(N) - 1]; \ req->basicrate ## N .data = item->supprates[(N) - 1]; \
req->basicrate ## N .status = \ req->basicrate ## N .status = \
...@@ -458,7 +458,7 @@ int prism2mgmt_scan_results(struct wlandevice *wlandev, void *msgp) ...@@ -458,7 +458,7 @@ int prism2mgmt_scan_results(struct wlandevice *wlandev, void *msgp)
#define REQSUPPRATE(N) \ #define REQSUPPRATE(N) \
do { \ do { \
if (count >= N) { \ if (count >= (N)) { \
req->supprate ## N .data = item->supprates[(N) - 1]; \ req->supprate ## N .data = item->supprates[(N) - 1]; \
req->supprate ## N .status = \ req->supprate ## N .status = \
P80211ENUM_msgitem_status_data_ok; \ P80211ENUM_msgitem_status_data_ok; \
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment