brcmfmac: be more verbose when PSM's watchdog fires
authorRafał Miłecki <rafal@milecki.pl>
Mon, 30 Jan 2017 15:09:52 +0000 (16:09 +0100)
committerKalle Valo <kvalo@codeaurora.org>
Thu, 2 Feb 2017 06:14:26 +0000 (08:14 +0200)
It's important to inform user so he knows things went wrong. He may also
want to get memory dump for further debugging purposes.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
Acked-by: Arend van Spriel <arend.vanspriel@broadcom.com>
Signed-off-by: Kalle Valo <kvalo@codeaurora.org>
drivers/net/wireless/broadcom/brcm80211/brcmfmac/debug.c

index 6f8a4b074c312155bd3873804dd91eeea828cfd2..f4644cf371c7e058401fcba2797fb9bd11f63993 100644 (file)
@@ -58,10 +58,18 @@ static int brcmf_debug_psm_watchdog_notify(struct brcmf_if *ifp,
                                           const struct brcmf_event_msg *evtmsg,
                                           void *data)
 {
+       int err;
+
        brcmf_dbg(TRACE, "enter: bsscfgidx=%d\n", ifp->bsscfgidx);
 
-       return brcmf_debug_create_memdump(ifp->drvr->bus_if, data,
-                                         evtmsg->datalen);
+       brcmf_err("PSM's watchdog has fired!\n");
+
+       err = brcmf_debug_create_memdump(ifp->drvr->bus_if, data,
+                                        evtmsg->datalen);
+       if (err)
+               brcmf_err("Failed to get memory dump, %d\n", err);
+
+       return err;
 }
 
 void brcmf_debugfs_init(void)