brcmfmac: don't WARN when there are no requests
authorAdrian Ratiu <adrian.ratiu@collabora.com>
Wed, 25 Sep 2019 13:44:57 +0000 (16:44 +0300)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 1 Oct 2019 09:14:20 +0000 (12:14 +0300)
When n_reqs == 0 there is nothing to do so it doesn't make sense to
search for requests and issue a warning because none is found.

Signed-off-by: Martyn Welch <martyn.welch@collabora.com>
Signed-off-by: Adrian Ratiu <adrian.ratiu@collabora.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/pno.c

index 14e5306..fabfbb0 100644 (file)
@@ -57,6 +57,10 @@ static int brcmf_pno_remove_request(struct brcmf_pno_info *pi, u64 reqid)
 
        mutex_lock(&pi->req_lock);
 
+       /* Nothing to do if we have no requests */
+       if (pi->n_reqs == 0)
+               goto done;
+
        /* find request */
        for (i = 0; i < pi->n_reqs; i++) {
                if (pi->reqs[i]->reqid == reqid)