From f08aeded81b56d0fdd23b1c102ed5a1871f3bc06 Mon Sep 17 00:00:00 2001 From: AleX Pelosi Date: Thu, 15 Feb 2024 18:42:11 +0000 Subject: [PATCH] Revert "healthd: BatteryMonitor: doLogValues: reduce log level to debug" This reverts commit aeccbb8f3a4f2a7256a6308b0216dcb88b4bff8c. Reason for revert: needed for debug and investigations of battery and charging issues. The tooling depends on these lines (and have been for years). You should have these lines only on SOC changes and in the presence of a serious problem. Please do not remove logging without consulting with the BMS SW team first. Change-Id: Ife0079a7e0c1499b22e843429832b019139c0d89 --- healthd/BatteryMonitor.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/healthd/BatteryMonitor.cpp b/healthd/BatteryMonitor.cpp index fbdf5fed1..b8bb58682 100644 --- a/healthd/BatteryMonitor.cpp +++ b/healthd/BatteryMonitor.cpp @@ -530,7 +530,7 @@ static void doLogValues(const HealthInfo& props, const struct healthd_config& he props.chargerAcOnline ? "a" : "", props.chargerUsbOnline ? "u" : "", props.chargerWirelessOnline ? "w" : "", props.chargerDockOnline ? "d" : ""); - KLOG_DEBUG(LOG_TAG, "%s\n", dmesgline); + KLOG_WARNING(LOG_TAG, "%s\n", dmesgline); } void BatteryMonitor::logValues(const HealthInfo_2_1& health_info,