diff options
author | Dmitry Tarnyagin <dmitry.tarnyagin@stericsson.com> | 2011-10-10 12:17:09 +0200 |
---|---|---|
committer | Philippe LANGLAIS <philippe.langlais@stericsson.com> | 2011-10-13 10:24:06 +0200 |
commit | 76d54824478f3d14eadde3e83f12ced24483739f (patch) | |
tree | 20f4ac68d18d38d74553de77a01aa84a8feb5c39 /drivers/staging/cw1200/txrx.c | |
parent | 847817a7ac38983d030d87d4c1b38e379a26fe8e (diff) |
cw1200: Fix checkpatch warnings.
Change-Id: Ibda6002f9ce429dabcc098bb2a88a4ca7e3cf5a4
Signed-off-by: Dmitry Tarnyagin <dmitry.tarnyagin@stericsson.com>
Reviewed-on: http://gerrit.lud.stericsson.com/gerrit/33543
Reviewed-by: Bartosz MARKOWSKI <bartosz.markowski@tieto.com>
Reviewed-by: Philippe LANGLAIS <philippe.langlais@stericsson.com>
Diffstat (limited to 'drivers/staging/cw1200/txrx.c')
-rw-r--r-- | drivers/staging/cw1200/txrx.c | 18 |
1 files changed, 10 insertions, 8 deletions
diff --git a/drivers/staging/cw1200/txrx.c b/drivers/staging/cw1200/txrx.c index 72f5acdccdc..109ea2a0286 100644 --- a/drivers/staging/cw1200/txrx.c +++ b/drivers/staging/cw1200/txrx.c @@ -468,9 +468,8 @@ cw1200_tx_h_crypt(struct cw1200_common *priv, iv_len = t->tx_info->control.hw_key->iv_len; icv_len = t->tx_info->control.hw_key->icv_len; - if (t->tx_info->control.hw_key->cipher == WLAN_CIPHER_SUITE_TKIP) { + if (t->tx_info->control.hw_key->cipher == WLAN_CIPHER_SUITE_TKIP) icv_len += 8; /* MIC */ - } if ((skb_headroom(t->skb) + skb_tailroom(t->skb) < iv_len + icv_len + WSM_TX_EXTRA_HEADROOM) || @@ -587,8 +586,7 @@ cw1200_tx_h_bt(struct cw1200_common *priv, u16 *ethertype = (u16 *) &payload[6]; if (unlikely(*ethertype == __be16_to_cpu(ETH_P_PAE))) priority = WSM_EPTA_PRIORITY_EAPOL; - } - else if (unlikely(ieee80211_is_assoc_req(t->hdr->frame_control) || + } else if (unlikely(ieee80211_is_assoc_req(t->hdr->frame_control) || ieee80211_is_reassoc_req(t->hdr->frame_control))) { struct ieee80211_mgmt *mgt_frame = (struct ieee80211_mgmt *)t->hdr; @@ -708,7 +706,8 @@ void cw1200_tx(struct ieee80211_hw *dev, struct sk_buff *skb) if (WARN_ON(t.queue >= 4 || !t.rate)) goto drop; - if ((ret = cw1200_tx_h_calc_link_ids(priv, &t))) + ret = cw1200_tx_h_calc_link_ids(priv, &t); + if (ret) goto drop; txrx_printk(KERN_DEBUG "[TX] TX %d bytes " @@ -718,11 +717,14 @@ void cw1200_tx(struct ieee80211_hw *dev, struct sk_buff *skb) cw1200_tx_h_pm(priv, &t); cw1200_tx_h_calc_tid(priv, &t); - if ((ret = cw1200_tx_h_crypt(priv, &t))) + ret = cw1200_tx_h_crypt(priv, &t); + if (ret) goto drop; - if ((ret = cw1200_tx_h_align(priv, &t))) + ret = cw1200_tx_h_align(priv, &t); + if (ret) goto drop; - if ((ret = cw1200_tx_h_action(priv, &t))) + ret = cw1200_tx_h_action(priv, &t); + if (ret) goto drop; wsm = cw1200_tx_h_wsm(priv, &t); if (!wsm) { |