staging: wilc1000: added #define for setting radiotap header
authorAjay Singh <ajay.kathat@microchip.com>
Fri, 11 May 2018 08:13:26 +0000 (13:43 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 14 May 2018 12:04:49 +0000 (14:04 +0200)
Added new macro to resolve below checkpatch issues in linux_mon.

"Lines should not end with a '('"

Signed-off-by: Ajay Singh <ajay.kathat@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wilc1000/linux_mon.c

index c372e0f..1c7e6e1 100644 (file)
@@ -22,6 +22,9 @@ static u8 bssid[6];
 #define IEEE80211_RADIOTAP_F_TX_FAIL   0x0001  /* failed due to excessive*/
 #define GET_PKT_OFFSET(a) (((a) >> 22) & 0x1ff)
 
+#define TX_RADIOTAP_PRESENT ((1 << IEEE80211_RADIOTAP_RATE) |  \
+                            (1 << IEEE80211_RADIOTAP_TX_FLAGS))
+
 void wilc_wfi_monitor_rx(u8 *buff, u32 size)
 {
        u32 header, pkt_offset;
@@ -59,9 +62,7 @@ void wilc_wfi_monitor_rx(u8 *buff, u32 size)
 
                cb_hdr->hdr.it_len = cpu_to_le16(sizeof(*cb_hdr));
 
-               cb_hdr->hdr.it_present = cpu_to_le32(
-                               (1 << IEEE80211_RADIOTAP_RATE) |
-                               (1 << IEEE80211_RADIOTAP_TX_FLAGS));
+               cb_hdr->hdr.it_present = cpu_to_le32(TX_RADIOTAP_PRESENT);
 
                cb_hdr->rate = 5; /* txrate->bitrate / 5; */
 
@@ -177,9 +178,7 @@ static netdev_tx_t wilc_wfi_mon_xmit(struct sk_buff *skb,
 
                cb_hdr->hdr.it_len = cpu_to_le16(sizeof(*cb_hdr));
 
-               cb_hdr->hdr.it_present = cpu_to_le32(
-                               (1 << IEEE80211_RADIOTAP_RATE) |
-                               (1 << IEEE80211_RADIOTAP_TX_FLAGS));
+               cb_hdr->hdr.it_present = cpu_to_le32(TX_RADIOTAP_PRESENT);
 
                cb_hdr->rate = 5; /* txrate->bitrate / 5; */
                cb_hdr->tx_flags = 0x0004;