| 1234567891011121314151617181920212223242526272829303132333435363738394041 |
- From 55e491edbf14b2da5419c2a319ea3b1d6368d9a2 Mon Sep 17 00:00:00 2001
- From: Hans de Goede <[email protected]>
- Date: Wed, 10 Oct 2018 13:01:03 +0200
- Subject: [PATCH] brcmfmac: Cleanup brcmf_fw_request_done()
- 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 <[email protected]>
- Signed-off-by: Kalle Valo <[email protected]>
- ---
- drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c | 8 +-------
- 1 file changed, 1 insertion(+), 7 deletions(-)
- --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c
- +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/firmware.c
- @@ -560,22 +560,16 @@ static int brcmf_fw_request_firmware(con
- 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;
- }
|
