staging: wfx: declare variables at beginning of functions
authorJérôme Pouiller <jerome.pouiller@silabs.com>
Mon, 13 Sep 2021 13:01:47 +0000 (15:01 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 14 Sep 2021 07:16:32 +0000 (09:16 +0200)
For better code, we prefer to declare all the local variables at
beginning of the functions.

Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Link: https://lore.kernel.org/r/20210913130203.1903622-17-Jerome.Pouiller@silabs.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/wfx/data_tx.c

index caeaf83..00c305f 100644 (file)
@@ -108,6 +108,7 @@ static int wfx_tx_policy_get(struct wfx_vif *wvif,
        int idx;
        struct tx_policy_cache *cache = &wvif->tx_policy_cache;
        struct tx_policy wanted;
+       struct tx_policy *entry;
 
        wfx_tx_policy_build(wvif, &wanted, rates);
 
@@ -121,11 +122,10 @@ static int wfx_tx_policy_get(struct wfx_vif *wvif,
        if (idx >= 0) {
                *renew = false;
        } else {
-               struct tx_policy *entry;
-               *renew = true;
-               /* If policy is not found create a new one
-                * using the oldest entry in "free" list
+               /* If policy is not found create a new one using the oldest
+                * entry in "free" list
                 */
+               *renew = true;
                entry = list_entry(cache->free.prev, struct tx_policy, link);
                memcpy(entry->rates, wanted.rates, sizeof(entry->rates));
                entry->uploaded = false;