summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Ferguson <ferguson.david@gmail.com>2013-11-17 18:55:37 +0400
committerRicardo Cerqueira <cyanogenmod@cerqueira.org>2014-02-11 17:18:26 +0000
commitad3d2a2ae7409faa1a885363b6d97bc9cd705e77 (patch)
tree44bd5dcec973c656a260ff000e7f82618763cad3
parentc2bbe95d98f2eaf8d4bec28c1423ca53675c70f6 (diff)
downloaddevice_samsung_smdk4412-common-ad3d2a2ae7409faa1a885363b6d97bc9cd705e77.tar.gz
device_samsung_smdk4412-common-ad3d2a2ae7409faa1a885363b6d97bc9cd705e77.tar.bz2
device_samsung_smdk4412-common-ad3d2a2ae7409faa1a885363b6d97bc9cd705e77.zip
Proper battery paths
* 4.4 introduces a new daemon, healthd, that handles battery status - resides in: /system/core/healthd - BatteryMonitor.cpp iterates through all the *directories* under /sys/class/power_supply looking for a file called 'type' - if the type == Battery, it assumes this is the location for the battery stats for the device. Only the first directory that matches this criteria is used. - on msm8660/d2/jf/s2 kernels, this search results in: /sys/class/power_supply/fuelgauge. - To determine capacity, healthd looks at the contents of the 'capacity' file under this discovered base path. This results in /sys/class/power_supply/fuelgauge/capacity on msm8660/d2/jf/s2 kernels. - Unfortunately, this is the wrong path. The capacity file at this location returns a large negative number. - Other paths like status, voltage_now, and present are also incorrectly determined. * For comparison, in 4.3 battery statistics gathering was handled by frameworks/base/services/jni/com_android_server_BatteryService.cpp - BatteryService does a similar search for a file called 'type' with the contents of 'Battery' - But the search logic is different here: it takes the *last* path that matches the criteria. - This results in the correct /sys/class/power_supply/battery * This patch overrides the discovery mechanism for battery status completely. Change-Id: I4579c92bba5596eef4dac67b8c0687be50a1d2f5
-rw-r--r--BoardCommonConfig.mk3
-rw-r--r--libhealthd/Android.mk22
-rw-r--r--libhealthd/healthd_board_exynos4.cpp39
3 files changed, 64 insertions, 0 deletions
diff --git a/BoardCommonConfig.mk b/BoardCommonConfig.mk
index d45fe93..1b7ca17 100644
--- a/BoardCommonConfig.mk
+++ b/BoardCommonConfig.mk
@@ -155,5 +155,8 @@ BOARD_HAS_NO_SELECT_BUTTON := true
BOARD_CHARGING_MODE_BOOTING_LPM := /sys/class/power_supply/battery/batt_lp_charging
BOARD_BATTERY_DEVICE_NAME := "battery"
+# Override healthd HAL
+BOARD_HAL_STATIC_LIBRARIES := libhealthd.exynos4
+
# inherit from the proprietary version
-include vendor/samsung/smdk4412-common/BoardConfigVendor.mk
diff --git a/libhealthd/Android.mk b/libhealthd/Android.mk
new file mode 100644
index 0000000..f1b83d6
--- /dev/null
+++ b/libhealthd/Android.mk
@@ -0,0 +1,22 @@
+# Copyright (C) 2013 The Android Open Source Project
+# Copyright (C) 2013 The CyanogenMod Project
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+LOCAL_PATH := $(call my-dir)
+
+include $(CLEAR_VARS)
+LOCAL_SRC_FILES := healthd_board_exynos4.cpp
+LOCAL_MODULE := libhealthd.exynos4
+LOCAL_C_INCLUDES := system/core/healthd
+include $(BUILD_STATIC_LIBRARY)
diff --git a/libhealthd/healthd_board_exynos4.cpp b/libhealthd/healthd_board_exynos4.cpp
new file mode 100644
index 0000000..5809692
--- /dev/null
+++ b/libhealthd/healthd_board_exynos4.cpp
@@ -0,0 +1,39 @@
+/*
+ * Copyright (C) 2013 The Android Open Source Project
+ * Copyright (C) 2013 The CyanogenMod Project
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <healthd.h>
+
+void
+healthd_board_init(struct healthd_config *config)
+{
+ config->batteryCapacityPath = "/sys/class/power_supply/battery/capacity";
+ config->batteryStatusPath = "/sys/class/power_supply/battery/status";
+ config->batteryVoltagePath = "/sys/class/power_supply/battery/voltage_now";
+ config->batteryCurrentNowPath = "/sys/class/power_supply/battery/batt_current_now";
+ config->batteryPresentPath = "/sys/class/power_supply/battery/present";
+ config->batteryHealthPath = "/sys/class/power_supply/battery/health";
+ config->batteryTemperaturePath = "/sys/class/power_supply/battery/temp";
+ config->batteryTechnologyPath = "/sys/class/power_supply/battery/technology";
+ // others use defaults
+}
+
+int
+healthd_board_battery_update(struct android::BatteryProperties *props)
+{
+ // return 0 to log periodic polled battery status to kernel log
+ return 0;
+}