brcmfmac: Cleanup brcmf_fw_request_done()
authorHans de Goede <hdegoede@redhat.com>
Wed, 10 Oct 2018 11:01:03 +0000 (13:01 +0200)
committerKalle Valo <kvalo@codeaurora.org>
Tue, 6 Nov 2018 16:50:27 +0000 (18:50 +0200)
The "cur" variable is now only used for a debug print and we already
print the same info from brcmf_fw_complete_request(), so the debug print
does not provide any extra info and we can remove it.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c

index 6755b23..b38c4b4 100644 (file)
@@ -560,22 +560,16 @@ static int brcmf_fw_request_firmware(const struct firmware **fw,
 static void brcmf_fw_request_done(const struct firmware *fw, void *ctx)
 {
        struct brcmf_fw *fwctx = ctx;
-       struct brcmf_fw_item *cur;
-       int ret = 0;
-
-       cur = &fwctx->req->items[fwctx->curpos];
+       int ret;
 
        ret = brcmf_fw_complete_request(fw, fwctx);
 
        while (ret == 0 && ++fwctx->curpos < fwctx->req->n_items) {
-               cur = &fwctx->req->items[fwctx->curpos];
                brcmf_fw_request_firmware(&fw, fwctx);
                ret = brcmf_fw_complete_request(fw, ctx);
        }
 
        if (ret) {
-               brcmf_dbg(TRACE, "failed err=%d: dev=%s, fw=%s\n", ret,
-                         dev_name(fwctx->dev), cur->path);
                brcmf_fw_free_request(fwctx->req);
                fwctx->req = NULL;
        }