mt76: fix some checkpatch warnings
authorRyder Lee <ryder.lee@mediatek.com>
Wed, 21 Aug 2019 03:11:15 +0000 (11:11 +0800)
committerFelix Fietkau <nbd@nbd.name>
Thu, 5 Sep 2019 15:42:32 +0000 (17:42 +0200)
This fixes the following checkpatch warnings:
CHECK: Alignment should match open parenthesis
CHECK: No space is necessary after a cast

Signed-off-by: Ryder Lee <ryder.lee@mediatek.com>
Signed-off-by: Felix Fietkau <nbd@nbd.name>
drivers/net/wireless/mediatek/mt76/agg-rx.c
drivers/net/wireless/mediatek/mt76/dma.c
drivers/net/wireless/mediatek/mt76/mac80211.c
drivers/net/wireless/mediatek/mt76/mt76.h
drivers/net/wireless/mediatek/mt76/mt7615/mac.c
drivers/net/wireless/mediatek/mt76/trace.h
drivers/net/wireless/mediatek/mt76/tx.c
drivers/net/wireless/mediatek/mt76/usb.c
drivers/net/wireless/mediatek/mt76/usb_trace.h
drivers/net/wireless/mediatek/mt76/util.h

index 2ba157f73bf10b842abc16e983e8919c993690af..8f3d36a15e1740e32c95bdff0763cb2ab6219c09 100644 (file)
@@ -23,8 +23,9 @@ mt76_aggr_release(struct mt76_rx_tid *tid, struct sk_buff_head *frames, int idx)
 }
 
 static void
-mt76_rx_aggr_release_frames(struct mt76_rx_tid *tid, struct sk_buff_head *frames,
-                        u16 head)
+mt76_rx_aggr_release_frames(struct mt76_rx_tid *tid,
+                           struct sk_buff_head *frames,
+                           u16 head)
 {
        int idx;
 
@@ -63,15 +64,14 @@ mt76_rx_aggr_check_release(struct mt76_rx_tid *tid, struct sk_buff_head *frames)
        for (idx = (tid->head + 1) % tid->size;
             idx != start && nframes;
             idx = (idx + 1) % tid->size) {
-
                skb = tid->reorder_buf[idx];
                if (!skb)
                        continue;
 
                nframes--;
-               status = (struct mt76_rx_status *) skb->cb;
-               if (!time_after(jiffies, status->reorder_time +
-                                        REORDER_TIMEOUT))
+               status = (struct mt76_rx_status *)skb->cb;
+               if (!time_after(jiffies,
+                               status->reorder_time + REORDER_TIMEOUT))
                        continue;
 
                mt76_rx_aggr_release_frames(tid, frames, status->seqno);
@@ -111,8 +111,8 @@ mt76_rx_aggr_reorder_work(struct work_struct *work)
 static void
 mt76_rx_aggr_check_ctl(struct sk_buff *skb, struct sk_buff_head *frames)
 {
-       struct mt76_rx_status *status = (struct mt76_rx_status *) skb->cb;
-       struct ieee80211_bar *bar = (struct ieee80211_bar *) skb->data;
+       struct mt76_rx_status *status = (struct mt76_rx_status *)skb->cb;
+       struct ieee80211_bar *bar = (struct ieee80211_bar *)skb->data;
        struct mt76_wcid *wcid = status->wcid;
        struct mt76_rx_tid *tid;
        u16 seqno;
@@ -137,8 +137,8 @@ mt76_rx_aggr_check_ctl(struct sk_buff *skb, struct sk_buff_head *frames)
 
 void mt76_rx_aggr_reorder(struct sk_buff *skb, struct sk_buff_head *frames)
 {
-       struct mt76_rx_status *status = (struct mt76_rx_status *) skb->cb;
-       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;
+       struct mt76_rx_status *status = (struct mt76_rx_status *)skb->cb;
+       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *)skb->data;
        struct mt76_wcid *wcid = status->wcid;
        struct ieee80211_sta *sta;
        struct mt76_rx_tid *tid;
@@ -222,7 +222,8 @@ void mt76_rx_aggr_reorder(struct sk_buff *skb, struct sk_buff_head *frames)
        tid->nframes++;
        mt76_rx_aggr_release_head(tid, frames);
 
-       ieee80211_queue_delayed_work(tid->dev->hw, &tid->reorder_work, REORDER_TIMEOUT);
+       ieee80211_queue_delayed_work(tid->dev->hw, &tid->reorder_work,
+                                    REORDER_TIMEOUT);
 
 out:
        spin_unlock_bh(&tid->lock);
index dbfd15e861e91980465d692aa6f5afc3eddf83f5..46f5223b4d8964d48d90612b1331818832f7b2e0 100644 (file)
@@ -493,7 +493,7 @@ mt76_dma_rx_process(struct mt76_dev *dev, struct mt76_queue *q, int budget)
                skb_reserve(skb, q->buf_offset);
 
                if (q == &dev->q_rx[MT_RXQ_MCU]) {
-                       u32 *rxfce = (u32 *) skb->cb;
+                       u32 *rxfce = (u32 *)skb->cb;
                        *rxfce = info;
                }
 
index ba67b0a5539092f9fb8355758468b22df47ad566..20bb4b23d27fe1f1ec27aa93db63612f91ed28b0 100644 (file)
@@ -494,7 +494,7 @@ struct ieee80211_sta *mt76_rx_convert(struct sk_buff *skb)
        struct ieee80211_rx_status *status = IEEE80211_SKB_RXCB(skb);
        struct mt76_rx_status mstat;
 
-       mstat = *((struct mt76_rx_status *) skb->cb);
+       mstat = *((struct mt76_rx_status *)skb->cb);
        memset(status, 0, sizeof(*status));
 
        status->flag = mstat.flag;
@@ -509,8 +509,10 @@ struct ieee80211_sta *mt76_rx_convert(struct sk_buff *skb)
        status->chains = mstat.chains;
 
        BUILD_BUG_ON(sizeof(mstat) > sizeof(skb->cb));
-       BUILD_BUG_ON(sizeof(status->chain_signal) != sizeof(mstat.chain_signal));
-       memcpy(status->chain_signal, mstat.chain_signal, sizeof(mstat.chain_signal));
+       BUILD_BUG_ON(sizeof(status->chain_signal) !=
+                    sizeof(mstat.chain_signal));
+       memcpy(status->chain_signal, mstat.chain_signal,
+              sizeof(mstat.chain_signal));
 
        return wcid_to_sta(mstat.wcid);
 }
@@ -519,7 +521,7 @@ EXPORT_SYMBOL(mt76_rx_convert);
 static int
 mt76_check_ccmp_pn(struct sk_buff *skb)
 {
-       struct mt76_rx_status *status = (struct mt76_rx_status *) skb->cb;
+       struct mt76_rx_status *status = (struct mt76_rx_status *)skb->cb;
        struct mt76_wcid *wcid = status->wcid;
        struct ieee80211_hdr *hdr;
        int ret;
@@ -535,7 +537,7 @@ mt76_check_ccmp_pn(struct sk_buff *skb)
                 * Validate the first fragment both here and in mac80211
                 * All further fragments will be validated by mac80211 only.
                 */
-               hdr = (struct ieee80211_hdr *) skb->data;
+               hdr = (struct ieee80211_hdr *)skb->data;
                if (ieee80211_is_frag(hdr) &&
                    !ieee80211_is_first_frag(hdr->frame_control))
                        return 0;
@@ -558,8 +560,8 @@ mt76_check_ccmp_pn(struct sk_buff *skb)
 static void
 mt76_check_sta(struct mt76_dev *dev, struct sk_buff *skb)
 {
-       struct mt76_rx_status *status = (struct mt76_rx_status *) skb->cb;
-       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;
+       struct mt76_rx_status *status = (struct mt76_rx_status *)skb->cb;
+       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *)skb->data;
        struct ieee80211_sta *sta;
        struct mt76_wcid *wcid = status->wcid;
        bool ps;
@@ -568,13 +570,13 @@ mt76_check_sta(struct mt76_dev *dev, struct sk_buff *skb)
        if (ieee80211_is_pspoll(hdr->frame_control) && !wcid) {
                sta = ieee80211_find_sta_by_ifaddr(dev->hw, hdr->addr2, NULL);
                if (sta)
-                       wcid = status->wcid = (struct mt76_wcid *) sta->drv_priv;
+                       wcid = status->wcid = (struct mt76_wcid *)sta->drv_priv;
        }
 
        if (!wcid || !wcid->sta)
                return;
 
-       sta = container_of((void *) wcid, struct ieee80211_sta, drv_priv);
+       sta = container_of((void *)wcid, struct ieee80211_sta, drv_priv);
 
        if (status->signal <= 0)
                ewma_signal_add(&wcid->rssi, -status->signal);
@@ -590,8 +592,8 @@ mt76_check_sta(struct mt76_dev *dev, struct sk_buff *skb)
        }
 
        if (ieee80211_has_morefrags(hdr->frame_control) ||
-               !(ieee80211_is_mgmt(hdr->frame_control) ||
-                 ieee80211_is_data(hdr->frame_control)))
+           !(ieee80211_is_mgmt(hdr->frame_control) ||
+             ieee80211_is_data(hdr->frame_control)))
                return;
 
        ps = ieee80211_has_pm(hdr->frame_control);
@@ -620,7 +622,7 @@ mt76_check_sta(struct mt76_dev *dev, struct sk_buff *skb)
                if (!sta->txq[i])
                        continue;
 
-               mtxq = (struct mt76_txq *) sta->txq[i]->drv_priv;
+               mtxq = (struct mt76_txq *)sta->txq[i]->drv_priv;
                if (!skb_queue_empty(&mtxq->retry_q))
                        ieee80211_schedule_txq(dev->hw, sta->txq[i]);
        }
@@ -742,7 +744,7 @@ int mt76_sta_state(struct ieee80211_hw *hw, struct ieee80211_vif *vif,
                dev->drv->sta_assoc(dev, vif, sta);
 
        if (old_state == IEEE80211_STA_NONE &&
-                new_state == IEEE80211_STA_NOTEXIST)
+           new_state == IEEE80211_STA_NOTEXIST)
                mt76_sta_remove(dev, vif, sta);
 
        return 0;
@@ -782,7 +784,7 @@ static void
 __mt76_csa_finish(void *priv, u8 *mac, struct ieee80211_vif *vif)
 {
        if (vif->csa_active && ieee80211_csa_is_complete(vif))
-           ieee80211_csa_finish(vif);
+               ieee80211_csa_finish(vif);
 }
 
 void mt76_csa_finish(struct mt76_dev *dev)
index 0668ddb1ccbdd017371b35c237a808c59679b9f8..714b1dbbb624c83a22f855bd71db70f347243ae9 100644 (file)
@@ -674,7 +674,7 @@ static inline struct mt76_tx_cb *mt76_tx_skb_cb(struct sk_buff *skb)
 {
        BUILD_BUG_ON(sizeof(struct mt76_tx_cb) >
                     sizeof(IEEE80211_SKB_CB(skb)->status.status_driver_data));
-       return ((void *) IEEE80211_SKB_CB(skb)->status.status_driver_data);
+       return ((void *)IEEE80211_SKB_CB(skb)->status.status_driver_data);
 }
 
 static inline void mt76_insert_hdr_pad(struct sk_buff *skb)
index 001e402f0d33a0e03cac0c23e0c3f931205aac1d..c35c386ea6bdd22de2e162415f7ba451ed652aab 100644 (file)
@@ -337,7 +337,8 @@ int mt7615_mac_write_txwi(struct mt7615_dev *dev, __le32 *txwi,
        fc_stype = (le16_to_cpu(fc) & IEEE80211_FCTL_STYPE) >> 4;
 
        if (ieee80211_is_data(fc) || ieee80211_is_bufferable_mmpdu(fc)) {
-               q_idx = skb_get_queue_mapping(skb) + wmm_idx * MT7615_MAX_WMM_SETS;
+               q_idx = wmm_idx * MT7615_MAX_WMM_SETS +
+                       skb_get_queue_mapping(skb);
                p_fmt = MT_TX_TYPE_CT;
        } else if (ieee80211_is_beacon(fc)) {
                q_idx = MT_LMAC_BCN0;
index e432f9e845df7ed849b7bd6f46acf2dc8092f7d6..0b3e635da868d21c8eba2bb2fa5ba17fe9685e5d 100644 (file)
 #define TRACE_SYSTEM mt76
 
 #define MAXNAME                32
-#define DEV_ENTRY   __array(char, wiphy_name, 32)
-#define DEV_ASSIGN  strlcpy(__entry->wiphy_name, wiphy_name(dev->hw->wiphy), MAXNAME)
-#define DEV_PR_FMT  "%s"
-#define DEV_PR_ARG  __entry->wiphy_name
+#define DEV_ENTRY      __array(char, wiphy_name, 32)
+#define DEV_ASSIGN     strlcpy(__entry->wiphy_name,    \
+                               wiphy_name(dev->hw->wiphy), MAXNAME)
+#define DEV_PR_FMT     "%s"
+#define DEV_PR_ARG     __entry->wiphy_name
 
 #define REG_ENTRY      __field(u32, reg) __field(u32, val)
 #define REG_ASSIGN     __entry->reg = reg; __entry->val = val
index 763b16186b254ee38ac647c97b2d4272ee5314b3..d7982aa83c115b558ce86a2437aaf12d316749a7 100644 (file)
@@ -86,7 +86,7 @@ mt76_txq_get_qid(struct ieee80211_txq *txq)
 static void
 mt76_check_agg_ssn(struct mt76_txq *mtxq, struct sk_buff *skb)
 {
-       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;
+       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *)skb->data;
 
        if (!ieee80211_is_data_qos(hdr->frame_control) ||
            !ieee80211_is_data_present(hdr->frame_control))
@@ -206,8 +206,8 @@ mt76_tx_status_skb_get(struct mt76_dev *dev, struct mt76_wcid *wcid, int pktid,
                if (cb->pktid == pktid)
                        return skb;
 
-               if (pktid >= 0 &&
-                   !time_after(jiffies, cb->jiffies + MT_TX_STATUS_SKB_TIMEOUT))
+               if (pktid >= 0 && !time_after(jiffies, cb->jiffies +
+                                             MT_TX_STATUS_SKB_TIMEOUT))
                        continue;
 
                __mt76_tx_status_skb_done(dev, skb, MT_TX_CB_TXS_FAILED |
@@ -249,7 +249,7 @@ mt76_tx(struct mt76_dev *dev, struct ieee80211_sta *sta,
        struct mt76_wcid *wcid, struct sk_buff *skb)
 {
        struct ieee80211_tx_info *info = IEEE80211_SKB_CB(skb);
-       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;
+       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *)skb->data;
        struct mt76_queue *q;
        int qid = skb_get_queue_mapping(skb);
 
@@ -269,7 +269,7 @@ mt76_tx(struct mt76_dev *dev, struct ieee80211_sta *sta,
 
                tid = skb->priority & IEEE80211_QOS_CTL_TID_MASK;
                txq = sta->txq[tid];
-               mtxq = (struct mt76_txq *) txq->drv_priv;
+               mtxq = (struct mt76_txq *)txq->drv_priv;
 
                if (mtxq->aggr)
                        mt76_check_agg_ssn(mtxq, skb);
@@ -317,7 +317,7 @@ static void
 mt76_queue_ps_skb(struct mt76_dev *dev, struct ieee80211_sta *sta,
                  struct sk_buff *skb, bool last)
 {
-       struct mt76_wcid *wcid = (struct mt76_wcid *) sta->drv_priv;
+       struct mt76_wcid *wcid = (struct mt76_wcid *)sta->drv_priv;
        struct ieee80211_tx_info *info = IEEE80211_SKB_CB(skb);
 
        info->control.flags |= IEEE80211_TX_CTRL_PS_RESPONSE;
@@ -343,7 +343,7 @@ mt76_release_buffered_frames(struct ieee80211_hw *hw, struct ieee80211_sta *sta,
        spin_lock_bh(&hwq->lock);
        for (i = 0; tids && nframes; i++, tids >>= 1) {
                struct ieee80211_txq *txq = sta->txq[i];
-               struct mt76_txq *mtxq = (struct mt76_txq *) txq->drv_priv;
+               struct mt76_txq *mtxq = (struct mt76_txq *)txq->drv_priv;
                struct sk_buff *skb;
 
                if (!(tids & 1))
@@ -606,7 +606,7 @@ void mt76_txq_remove(struct mt76_dev *dev, struct ieee80211_txq *txq)
        if (!txq)
                return;
 
-       mtxq = (struct mt76_txq *) txq->drv_priv;
+       mtxq = (struct mt76_txq *)txq->drv_priv;
 
        while ((skb = skb_dequeue(&mtxq->retry_q)) != NULL)
                ieee80211_free_txskb(dev->hw, skb);
@@ -615,7 +615,7 @@ EXPORT_SYMBOL_GPL(mt76_txq_remove);
 
 void mt76_txq_init(struct mt76_dev *dev, struct ieee80211_txq *txq)
 {
-       struct mt76_txq *mtxq = (struct mt76_txq *) txq->drv_priv;
+       struct mt76_txq *mtxq = (struct mt76_txq *)txq->drv_priv;
 
        skb_queue_head_init(&mtxq->retry_q);
 
index cba19a551224f4f82ed34f38369dc7518870665a..20c6fe510e9db8725f3b6f34ba06a4adb500e8ee 100644 (file)
@@ -154,7 +154,7 @@ static void mt76u_copy(struct mt76_dev *dev, u32 offset,
 
        mutex_lock(&usb->usb_ctrl_mtx);
        for (i = 0; i < DIV_ROUND_UP(len, 4); i++) {
-               put_unaligned(val[i], (u32 *) usb->data);
+               put_unaligned(val[i], (u32 *)usb->data);
                ret = __mt76u_vendor_request(dev, MT_VEND_MULTI_WRITE,
                                             USB_DIR_OUT | USB_TYPE_VENDOR,
                                             0, offset + i * 4, usb->data,
@@ -309,14 +309,13 @@ mt76u_refill_rx(struct mt76_dev *dev, struct urb *urb, int nsgs, gfp_t gfp)
 {
        struct mt76_queue *q = &dev->q_rx[MT_RXQ_MAIN];
 
-       if (dev->usb.sg_en) {
+       if (dev->usb.sg_en)
                return mt76u_fill_rx_sg(dev, q, urb, nsgs, gfp);
-       } else {
-               urb->transfer_buffer_length = q->buf_size;
-               urb->transfer_buffer = page_frag_alloc(&q->rx_page,
-                                                      q->buf_size, gfp);
-               return urb->transfer_buffer ? 0 : -ENOMEM;
-       }
+
+       urb->transfer_buffer_length = q->buf_size;
+       urb->transfer_buffer = page_frag_alloc(&q->rx_page, q->buf_size, gfp);
+
+       return urb->transfer_buffer ? 0 : -ENOMEM;
 }
 
 static int
@@ -752,13 +751,14 @@ mt76u_tx_setup_buffers(struct mt76_dev *dev, struct sk_buff *skb,
        if (!dev->usb.sg_en) {
                urb->transfer_buffer = skb->data;
                return 0;
-       } else {
-               sg_init_table(urb->sg, MT_TX_SG_MAX_SIZE);
-               urb->num_sgs = skb_to_sgvec(skb, urb->sg, 0, skb->len);
-               if (urb->num_sgs == 0)
-                       return -ENOMEM;
-               return urb->num_sgs;
        }
+
+       sg_init_table(urb->sg, MT_TX_SG_MAX_SIZE);
+       urb->num_sgs = skb_to_sgvec(skb, urb->sg, 0, skb->len);
+       if (!urb->num_sgs)
+               return -ENOMEM;
+
+       return urb->num_sgs;
 }
 
 static int
@@ -874,7 +874,8 @@ void mt76u_stop_tx(struct mt76_dev *dev)
        struct mt76_queue *q;
        int i, j, ret;
 
-       ret = wait_event_timeout(dev->tx_wait, !mt76_has_tx_pending(dev), HZ/5);
+       ret = wait_event_timeout(dev->tx_wait, !mt76_has_tx_pending(dev),
+                                HZ / 5);
        if (!ret) {
                dev_err(dev->dev, "timed out waiting for pending tx\n");
 
index f37bd9feacf0970ca578b070792289a5ad5e33dc..f5ab3215af800e0ef7432bab17428293bb53985c 100644 (file)
 #define TRACE_SYSTEM mt76_usb
 
 #define MAXNAME                32
-#define DEV_ENTRY   __array(char, wiphy_name, 32)
-#define DEV_ASSIGN  strlcpy(__entry->wiphy_name, wiphy_name(dev->hw->wiphy), MAXNAME)
-#define DEV_PR_FMT  "%s "
-#define DEV_PR_ARG  __entry->wiphy_name
+#define DEV_ENTRY      __array(char, wiphy_name, 32)
+#define DEV_ASSIGN     strlcpy(__entry->wiphy_name,    \
+                               wiphy_name(dev->hw->wiphy), MAXNAME)
+#define DEV_PR_FMT     "%s "
+#define DEV_PR_ARG     __entry->wiphy_name
 
 #define REG_ENTRY      __field(u32, reg) __field(u32, val)
 #define REG_ASSIGN     __entry->reg = reg; __entry->val = val
@@ -54,7 +55,7 @@ DECLARE_EVENT_CLASS(urb_transfer,
        TP_PROTO(struct mt76_dev *dev, struct urb *u),
        TP_ARGS(dev, u),
        TP_STRUCT__entry(
-               DEV_ENTRY __field(unsigned, pipe) __field(u32, len)
+               DEV_ENTRY __field(unsigned int, pipe) __field(u32, len)
        ),
        TP_fast_assign(
                DEV_ASSIGN;
index 9f8d59269a9e7bb29cd590608ed410454f2afce7..fe3479c8e561d1cfc426ca06c90177ccdd945e11 100644 (file)
@@ -12,7 +12,7 @@
 #include <linux/bitfield.h>
 
 #define MT76_INCR(_var, _size) \
-       _var = (((_var) + 1) % _size)
+       (_var = (((_var) + 1) % (_size)))
 
 int mt76_wcid_alloc(unsigned long *mask, int size);
 
@@ -25,7 +25,7 @@ mt76_wcid_free(unsigned long *mask, int idx)
 static inline void
 mt76_skb_set_moredata(struct sk_buff *skb, bool enable)
 {
-       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *) skb->data;
+       struct ieee80211_hdr *hdr = (struct ieee80211_hdr *)skb->data;
 
        if (enable)
                hdr->frame_control |= cpu_to_le16(IEEE80211_FCTL_MOREDATA);