brcmfmac: fix system warning message during wowl suspend
authorLo-Hsiang Lo <double.lo@cypress.com>
Mon, 7 Jan 2019 08:46:16 +0000 (08:46 +0000)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 10 Jan 2019 11:40:29 +0000 (13:40 +0200)
There is a system warning message, warn_slowpath-fmt, during suspend
while using supplicant join AP and enable wowl feature by IW command.
It's caused by brcmf_pno_remove_request path can't find the reqid.
This fix will not go to remove pno request function if there is no
pno scan.

Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Lo-Hsiang Lo <double.lo@cypress.com>
Signed-off-by: Chi-Hsien Lin <chi-hsien.lin@cypress.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/pno.c

index ffa243e..37b4038 100644 (file)
@@ -496,6 +496,11 @@ int brcmf_pno_stop_sched_scan(struct brcmf_if *ifp, u64 reqid)
        brcmf_dbg(TRACE, "reqid=%llu\n", reqid);
 
        pi = ifp_to_pno(ifp);
+
+       /* No PNO request */
+       if (!pi->n_reqs)
+               return 0;
+
        err = brcmf_pno_remove_request(pi, reqid);
        if (err)
                return err;