static void ath10k_htt_rx_ring_refill_retry(unsigned long arg)
 {
        struct ath10k_htt *htt = (struct ath10k_htt *)arg;
+
        ath10k_htt_rx_msdu_buff_replenish(htt);
 }
 
 {
        struct ath10k *ar = htt->ar;
        int msdu_len, msdu_chaining = 0;
-       struct sk_buff *msdu;
+       struct sk_buff *msdu, *next;
        struct htt_rx_desc *rx_desc;
 
        lockdep_assert_held(&htt->rx_ring.lock);
                        msdu->next = NULL;
                        break;
                } else {
-                       struct sk_buff *next = ath10k_htt_rx_netbuf_pop(htt);
+                       next = ath10k_htt_rx_netbuf_pop(htt);
                        msdu->next = next;
                        msdu = next;
                }
 static void ath10k_htt_rx_replenish_task(unsigned long ptr)
 {
        struct ath10k_htt *htt = (struct ath10k_htt *)ptr;
+
        ath10k_htt_rx_msdu_buff_replenish(htt);
 }
 
 /* This function only applies for first msdu in an msdu chain */
 static bool ath10k_htt_rx_hdr_is_amsdu(struct ieee80211_hdr *hdr)
 {
+       u8 *qc;
+
        if (ieee80211_is_data_qos(hdr->frame_control)) {
-               u8 *qc = ieee80211_get_qos_ctl(hdr);
+               qc = ieee80211_get_qos_ctl(hdr);
                if (qc[0] & 0x80)
                        return true;
        }
 
 {
        const struct ieee80211_sta_ht_cap *ht_cap = &sta->ht_cap;
        int i, n;
+       u32 stbc;
 
        lockdep_assert_held(&ar->conf_mutex);
 
        }
 
        if (ht_cap->cap & IEEE80211_HT_CAP_RX_STBC) {
-               u32 stbc;
                stbc = ht_cap->cap & IEEE80211_HT_CAP_RX_STBC;
                stbc = stbc >> IEEE80211_HT_CAP_RX_STBC_SHIFT;
                stbc = stbc << WMI_RC_RX_STBC_FLAG_S;
        struct ath10k *ar = hw->priv;
        struct ath10k_vif *arvif = ath10k_vif_to_arvif(vif);
        int ret = 0;
-       u32 vdev_param, pdev_param;
+       u32 vdev_param, pdev_param, slottime, preamble;
 
        mutex_lock(&ar->conf_mutex);
 
        }
 
        if (changed & BSS_CHANGED_ERP_SLOT) {
-               u32 slottime;
                if (info->use_short_slot)
                        slottime = WMI_VDEV_SLOT_TIME_SHORT; /* 9us */
 
        }
 
        if (changed & BSS_CHANGED_ERP_PREAMBLE) {
-               u32 preamble;
                if (info->use_short_preamble)
                        preamble = WMI_VDEV_PREAMBLE_SHORT;
                else
 
 int ath10k_wmi_wait_for_service_ready(struct ath10k *ar)
 {
        int ret;
+
        ret = wait_for_completion_timeout(&ar->wmi.service_ready,
                                          WMI_SERVICE_READY_TIMEOUT_HZ);
        return ret;
 int ath10k_wmi_wait_for_unified_ready(struct ath10k *ar)
 {
        int ret;
+
        ret = wait_for_completion_timeout(&ar->wmi.unified_ready,
                                          WMI_UNIFIED_READY_TIMEOUT_HZ);
        return ret;
        struct ieee80211_tim_ie *tim;
        u8 *ies, *ie;
        u8 ie_len, pvm_len;
+       __le32 t;
+       u32 v;
 
        /* if next SWBA has no tim_changed the tim_bitmap is garbage.
         * we must copy the bitmap upon change and reuse it later */
                             sizeof(bcn_info->tim_info.tim_bitmap));
 
                for (i = 0; i < sizeof(arvif->u.ap.tim_bitmap); i++) {
-                       __le32 t = bcn_info->tim_info.tim_bitmap[i / 4];
-                       u32 v = __le32_to_cpu(t);
+                       t = bcn_info->tim_info.tim_bitmap[i / 4];
+                       v = __le32_to_cpu(t);
                        arvif->u.ap.tim_bitmap[i] = (v >> ((i % 4) * 8)) & 0xFF;
                }