summaryrefslogtreecommitdiffstats
path: root/drivers/net/wireless/brcm80211/brcmfmac/fweh.c
diff options
context:
space:
mode:
authorArend van Spriel <arend@broadcom.com>2012-12-07 10:49:57 +0100
committerJohn W. Linville <linville@tuxdriver.com>2012-12-10 15:33:06 -0500
commit5e8149f5036afe2d94b5fafc9ff752283804a752 (patch)
treeb21016be1f39e649b4b79a4abfb75e93fecfe7b9 /drivers/net/wireless/brcm80211/brcmfmac/fweh.c
parentdc5cbdfdf7936178361c386de84aaf64e8842789 (diff)
downloadtalos-op-linux-5e8149f5036afe2d94b5fafc9ff752283804a752.tar.gz
talos-op-linux-5e8149f5036afe2d94b5fafc9ff752283804a752.zip
brcmfmac: error messages should not be suppressed
The call to brcmf_dbg(ERROR, ...) only resulted in a log message when compiled with -DDEBUG. Error messages are valuable for resolving issues so this patch replaces it with brcmf_err(...) so they always end up in the log. Reviewed-by: Hante Meuleman <meuleman@broadcom.com> Reviewed-by: Pieter-Paul Giesberts <pieterpg@broadcom.com> Signed-off-by: Arend van Spriel <arend@broadcom.com> Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/brcm80211/brcmfmac/fweh.c')
-rw-r--r--drivers/net/wireless/brcm80211/brcmfmac/fweh.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/drivers/net/wireless/brcm80211/brcmfmac/fweh.c b/drivers/net/wireless/brcm80211/brcmfmac/fweh.c
index c26493fb01b6..ba0b22512f12 100644
--- a/drivers/net/wireless/brcm80211/brcmfmac/fweh.c
+++ b/drivers/net/wireless/brcm80211/brcmfmac/fweh.c
@@ -157,9 +157,9 @@ static int brcmf_fweh_call_event_handler(struct brcmf_if *ifp,
if (ifp->ndev && fweh->evt_handler[code])
err = fweh->evt_handler[code](ifp, emsg, data);
else
- brcmf_dbg(ERROR, "unhandled event %d ignored\n", code);
+ brcmf_err("unhandled event %d ignored\n", code);
} else {
- brcmf_dbg(ERROR, "no interface object\n");
+ brcmf_err("no interface object\n");
}
return err;
}
@@ -184,7 +184,7 @@ static void brcmf_fweh_handle_if_event(struct brcmf_pub *drvr,
ifevent->bssidx, ifevent->flags);
if (ifevent->ifidx >= BRCMF_MAX_IFS) {
- brcmf_dbg(ERROR, "invalid interface index: %u\n",
+ brcmf_err("invalid interface index: %u\n",
ifevent->ifidx);
return;
}
@@ -286,7 +286,7 @@ static void brcmf_fweh_event_worker(struct work_struct *work)
err = brcmf_fweh_call_event_handler(ifp, event->code, &emsg,
event->data);
if (err) {
- brcmf_dbg(ERROR, "event handler failed (%d)\n",
+ brcmf_err("event handler failed (%d)\n",
event->code);
err = 0;
}
@@ -343,7 +343,7 @@ int brcmf_fweh_register(struct brcmf_pub *drvr, enum brcmf_fweh_event_code code,
brcmf_fweh_handler_t handler)
{
if (drvr->fweh.evt_handler[code]) {
- brcmf_dbg(ERROR, "event code %d already registered\n", code);
+ brcmf_err("event code %d already registered\n", code);
return -ENOSPC;
}
drvr->fweh.evt_handler[code] = handler;
@@ -391,7 +391,7 @@ int brcmf_fweh_activate_events(struct brcmf_if *ifp)
err = brcmf_fil_iovar_data_set(ifp, "event_msgs",
eventmask, BRCMF_EVENTING_MASK_LEN);
if (err)
- brcmf_dbg(ERROR, "Set event_msgs error (%d)\n", err);
+ brcmf_err("Set event_msgs error (%d)\n", err);
return err;
}
OpenPOWER on IntegriCloud