From 1079f2771e297e3f0d4f213805cf0a05e3f589de Mon Sep 17 00:00:00 2001 From: Dustin Brown Date: Thu, 1 Sep 2016 15:58:23 -0700 Subject: [PATCH] qcacld-3.0: Cleanup HDD suspend/resume logging messages Some HDD suspend/resume logging messages are vague or include redundant status codes. Update these messages to reflect what actually happened and remove redundant status codes. Change-Id: I5c4a481a8fbbdebe88f2ecffffb1f2189922da4f CRs-Fixed: 1062544 --- core/hdd/src/wlan_hdd_driver_ops.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/core/hdd/src/wlan_hdd_driver_ops.c b/core/hdd/src/wlan_hdd_driver_ops.c index 4b6e3c9d7611..68d6e6770f80 100644 --- a/core/hdd/src/wlan_hdd_driver_ops.c +++ b/core/hdd/src/wlan_hdd_driver_ops.c @@ -603,9 +603,10 @@ int __wlan_hdd_bus_suspend_noirq(void) { hdd_context_t *hdd_ctx = cds_get_context(QDF_MODULE_ID_HDD); void *hif_ctx; - int err = wlan_hdd_validate_context(hdd_ctx); + int err; int status; + err = wlan_hdd_validate_context(hdd_ctx); if (err) { hdd_err("Invalid HDD context: %d", err); return err; @@ -637,7 +638,10 @@ resume_hif_noirq: status = hif_bus_resume_noirq(hif_ctx); QDF_BUG(!status); done: - hdd_err("suspend_noirq failed, status = %d", err); + if (err == -EAGAIN) + hdd_err("firmware not ready for suspend, try again"); + else + hdd_err("suspend_noirq failed, status = %d", err); return err; }