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: Tim Collier <osde...@gmail.com>
---
 drivers/staging/wlan-ng/prism2mgmt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/wlan-ng/prism2mgmt.c 
b/drivers/staging/wlan-ng/prism2mgmt.c
index 28e4029d46f6..ebfe69b138c7 100644
--- a/drivers/staging/wlan-ng/prism2mgmt.c
+++ b/drivers/staging/wlan-ng/prism2mgmt.c
@@ -439,7 +439,7 @@ int prism2mgmt_scan_results(struct wlandevice *wlandev, 
void *msgp)
 
 #define REQBASICRATE(N) \
        do { \
-               if ((count >= N) && DOT11_RATE5_ISBASIC_GET( \
+               if ((count >= (N)) && DOT11_RATE5_ISBASIC_GET(  \
                        item->supprates[(N) - 1])) { \
                        req->basicrate ## N .data = item->supprates[(N) - 1]; \
                        req->basicrate ## N .status = \
@@ -458,7 +458,7 @@ int prism2mgmt_scan_results(struct wlandevice *wlandev, 
void *msgp)
 
 #define REQSUPPRATE(N) \
        do { \
-               if (count >= N) { \
+               if (count >= (N)) {                                     \
                        req->supprate ## N .data = item->supprates[(N) - 1]; \
                        req->supprate ## N .status = \
                                P80211ENUM_msgitem_status_data_ok; \
-- 
2.11.0

_______________________________________________
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel

Reply via email to