diff options
author | Arend Van Spriel <arend.vanspriel@broadcom.com> | 2018-03-22 16:28:20 -0400 |
---|---|---|
committer | Kalle Valo <kvalo@codeaurora.org> | 2018-03-27 05:04:20 -0400 |
commit | 1170f6d1be6a39e1a115a2c0f50923eb4ce2a7ec (patch) | |
tree | 52f09331f5b7391979287bac751af756318f5fbc | |
parent | e153292ae4d1e0c5b27677db9356725f5b614134 (diff) |
brcmfmac: do not convert linux error to firmware error string
In case of a linux error brcmf_fil_cmd_data() blurts an error message
in which the error code is translated to an error string. However, it
maps it to a firmware error string which should not happen. Simply
print only the numeric error code and be done with it.
Reviewed-by: Hante Meuleman <hante.meuleman@broadcom.com>
Reviewed-by: Pieter-Paul Giesberts <pieter-paul.giesberts@broadcom.com>
Reviewed-by: Franky Lin <franky.lin@broadcom.com>
Signed-off-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
-rw-r--r-- | drivers/net/wireless/broadcom/brcm80211/brcmfmac/fwil.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/fwil.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/fwil.c index fc5751116d99..802d7cb73b80 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/fwil.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/fwil.c | |||
@@ -124,8 +124,7 @@ brcmf_fil_cmd_data(struct brcmf_if *ifp, u32 cmd, void *data, u32 len, bool set) | |||
124 | data, len, &fwerr); | 124 | data, len, &fwerr); |
125 | 125 | ||
126 | if (err) { | 126 | if (err) { |
127 | brcmf_dbg(FIL, "Failed: %s (%d)\n", | 127 | brcmf_dbg(FIL, "Failed: error=%d\n", err); |
128 | brcmf_fil_get_errstr((u32)(-err)), err); | ||
129 | } else if (fwerr < 0) { | 128 | } else if (fwerr < 0) { |
130 | brcmf_dbg(FIL, "Firmware error: %s (%d)\n", | 129 | brcmf_dbg(FIL, "Firmware error: %s (%d)\n", |
131 | brcmf_fil_get_errstr((u32)(-fwerr)), fwerr); | 130 | brcmf_fil_get_errstr((u32)(-fwerr)), fwerr); |