diff options
author | Dennis Jeon <dennis.jeon@broadcom.corp-partner.google.com> | 2022-10-18 17:25:18 +0900 |
---|---|---|
committer | Roger Wang <wangroger@google.com> | 2022-10-18 13:42:27 +0000 |
commit | 287ce4ac156e7d98e84105934d78683fb861bfd7 (patch) | |
tree | 221e6c733c073e875be4b0beb687e0a963bc91f7 | |
parent | f7eff96ffea6fe94a7b04410dae8e1fdd847530b (diff) | |
download | bcm4389-287ce4ac156e7d98e84105934d78683fb861bfd7.tar.gz |
bcmdhd: Redirect RuntimePM logs to dbgring
changed from DHD_ERROR to DHD_RPM to push logs to driver_log ringbuffer only.
It seems better to keep the L1ss log to confirm that DHD calls Exynos API to enable/disable L1.2 mode.
It will help to identify issues in power consumption issues
Bug: 247124708
Test: logs are not seen in dmesg
Change-Id: I33db80a77f90d68c71f66e14e1db804ed3996b77
Signed-off-by: Dennis Jeon <dennis.jeon@broadcom.corp-partner.google.com>
-rw-r--r-- | dhd_custom_google.c | 2 | ||||
-rw-r--r-- | dhd_pcie_linux.c | 6 |
2 files changed, 4 insertions, 4 deletions
diff --git a/dhd_custom_google.c b/dhd_custom_google.c index 3a1c23d..4bc0057 100644 --- a/dhd_custom_google.c +++ b/dhd_custom_google.c @@ -934,7 +934,7 @@ dhd_wlan_deinit(void) void dhd_plat_l1ss_ctrl(bool ctrl) { #if defined(CONFIG_SOC_GOOGLE) - printk(KERN_DEBUG "%s: Control L1ss RC side %d \n", __FUNCTION__, ctrl); + DHD_CONS_ONLY(("%s: Control L1ss RC side %d \n", __FUNCTION__, ctrl)); exynos_pcie_rc_l1ss_ctrl(ctrl, PCIE_L1SS_CTRL_WIFI, 1); #endif /* CONFIG_SOC_GOOGLE */ return; diff --git a/dhd_pcie_linux.c b/dhd_pcie_linux.c index 5eaeefe..0a8ee7d 100644 --- a/dhd_pcie_linux.c +++ b/dhd_pcie_linux.c @@ -3308,7 +3308,7 @@ bool dhd_runtimepm_state(dhd_pub_t *dhd) !DHD_CHECK_CFG_IN_PROGRESS(dhd) && !dhd_os_check_wakelock_all(bus->dhd)) { #ifdef WL_CFG80211 ps_mode_off_dur = dhd_ps_mode_managed_dur(dhd); - DHD_ERROR(("%s: DHD Idle state!! - idletime :%d, wdtick :%d, " + DHD_RPM(("%s: DHD Idle state!! - idletime :%d, wdtick :%d, " "PS mode off dur: %d sec \n", __FUNCTION__, bus->idletime, dhd_runtimepm_ms, ps_mode_off_dur)); #else @@ -3415,7 +3415,7 @@ bool dhd_runtimepm_state(dhd_pub_t *dhd) smp_wmb(); wake_up(&bus->rpm_queue); - DHD_ERROR(("%s : runtime resume ended \n", __FUNCTION__)); + DHD_RPM(("%s : runtime resume ended \n", __FUNCTION__)); return TRUE; } else { DHD_GENERAL_UNLOCK(dhd, flags); @@ -3462,7 +3462,7 @@ bool dhd_runtime_bus_wake(dhd_bus_t *bus, bool wait, void *func_addr) DHD_GENERAL_UNLOCK(bus->dhd, flags); - DHD_ERROR(("Runtime Resume is called in %ps\n", func_addr)); + DHD_RPM(("Runtime Resume is called in %ps\n", func_addr)); smp_wmb(); wake_up(&bus->rpm_queue); /* No need to wake up the RPM state thread */ |