summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorJohn Baldwin <jhb@FreeBSD.org>2015-10-08 13:15:38 -0700
committerJohn Baldwin <jhb@FreeBSD.org>2015-10-08 13:15:38 -0700
commit5e33d9fe748efc8df49c123fe01d486b04c20f57 (patch)
tree7c2522eb3ac8a690a47b8b97fdedcc26b66ce70a /src
parent974f95702efbf1ae15882777d191c7fde99bfb9b (diff)
Properly detect the battery charging status under FreeBSD.
The hw.acpi.battery.state sysctl returns a bitmask of flags as defined in <dev/acpica/acpiio.h>. Use constants from this header to examine the state and check for the charging flag to determine if the battery is charging.
Diffstat (limited to 'src')
-rw-r--r--src/print_battery_info.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/src/print_battery_info.c b/src/print_battery_info.c
index f65090a..90db4bb 100644
--- a/src/print_battery_info.c
+++ b/src/print_battery_info.c
@@ -12,6 +12,7 @@
#if defined(__FreeBSD__) || defined(__FreeBSD_kernel__) || defined(__DragonFly__)
#include <sys/types.h>
#include <sys/sysctl.h>
+#include <dev/acpica/acpiio.h>
#endif
#if defined(__OpenBSD__)
@@ -236,7 +237,7 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
state = sysctl_rslt;
if (state == 0 && present_rate == 100)
status = CS_FULL;
- else if (state == 0 && present_rate < 100)
+ else if ((state & ACPI_BATT_STAT_CHARGING) && present_rate < 100)
status = CS_CHARGING;
else
status = CS_DISCHARGING;
@@ -248,7 +249,7 @@ void print_battery_info(yajl_gen json_gen, char *buffer, int number, const char
(void)snprintf(percentagebuf, sizeof(percentagebuf), "%02d%s",
present_rate, pct_mark);
- if (state == 1) {
+ if (state == ACPI_BATT_STAT_DISCHARG) {
int hours, minutes;
minutes = remaining;
hours = minutes / 60;