MLK-21579 brcmfmac: bcmsdh: move the func->num check early in .brcmf_ops_sdio_suspend()
authorFugang Duan <fugang.duan@nxp.com>
Wed, 24 Apr 2019 08:22:03 +0000 (16:22 +0800)
committerFugang Duan <fugang.duan@nxp.com>
Mon, 29 Apr 2019 01:35:18 +0000 (09:35 +0800)
Move the func->num check early in .brcmf_ops_sdio_suspend() to
save suspend time.

Reviewed-by: Richard Zhu <hongxing.zhu@nxp.com>
Signed-off-by: Fugang Duan <fugang.duan@nxp.com>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/bcmsdh.c

index b656901..c59ec99 100644 (file)
@@ -1133,6 +1133,9 @@ static int brcmf_ops_sdio_suspend(struct device *dev)
 
        brcmf_dbg(SDIO, "Enter: F%d\n", func->num);
 
+       if (func->num != 1)
+               return 0;
+
        while (retry &&
               config->pm_state == BRCMF_CFG80211_PM_STATE_SUSPENDING) {
                usleep_range(10000, 20000);
@@ -1141,9 +1144,6 @@ static int brcmf_ops_sdio_suspend(struct device *dev)
        if (!retry && config->pm_state == BRCMF_CFG80211_PM_STATE_SUSPENDING)
                brcmf_err("timed out wait for cfg80211 suspended\n");
 
-       if (func->num != 1)
-               return 0;
-
        sdiodev = bus_if->bus_priv.sdio;
 
        brcmf_sdiod_freezer_on(sdiodev);