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

staging: wlan-ng: replace macro with inline function in prism2mgmt.c

checkpatch gives the following message for the p80211rate_to_p2bit
macro:

CHECK: Macro argument reuse 'n' - possible side-effects?

To fix the message, replace the macro with an equivalent inline
function.
Signed-off-by: default avatarTim Collier <osdevtc@gmail.com>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 9e5ffffb
...@@ -85,10 +85,21 @@ ...@@ -85,10 +85,21 @@
#include "prism2mgmt.h" #include "prism2mgmt.h"
/* Converts 802.11 format rate specifications to prism2 */ /* Converts 802.11 format rate specifications to prism2 */
#define p80211rate_to_p2bit(n) ((((n) & ~BIT(7)) == 2) ? BIT(0) : \ static inline u16 p80211rate_to_p2bit(u32 rate)
(((n) & ~BIT(7)) == 4) ? BIT(1) : \ {
(((n) & ~BIT(7)) == 11) ? BIT(2) : \ switch (rate & ~BIT(7)) {
(((n) & ~BIT(7)) == 22) ? BIT(3) : 0) case 2:
return BIT(0);
case 4:
return BIT(1);
case 11:
return BIT(2);
case 22:
return BIT(3);
default:
return 0;
}
}
/*---------------------------------------------------------------- /*----------------------------------------------------------------
* prism2mgmt_scan * prism2mgmt_scan
......
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