diff options
author | James Buren <braewoods+rb@braewoods.net> | 2021-03-10 04:09:46 -0600 |
---|---|---|
committer | James Buren <braewoods+rb@braewoods.net> | 2021-03-10 04:09:46 -0600 |
commit | 0ba174789b45cd36fed61403898bbe5c2a01dcfb (patch) | |
tree | 63914375916ce9b9f8330d5f4aac4954d574a8d4 /firmware/powermgmt.c | |
parent | 777f63d5292df98253bbb00db9362ddbe55b778e (diff) | |
download | rockbox-0ba174789b45cd36fed61403898bbe5c2a01dcfb.tar.gz rockbox-0ba174789b45cd36fed61403898bbe5c2a01dcfb.zip |
usb_hid: switch Battery Strength to use feature reports
This means we will no longer send them routinely and instead rely on
the HID driver to send them when the host requests it. This also
moves the reporting out of the power management code where it probably
did not belong in the first place.
Change-Id: I9c8420e81897f1f6caaa55ffacc7525589f1ef75
Diffstat (limited to 'firmware/powermgmt.c')
-rw-r--r-- | firmware/powermgmt.c | 21 |
1 files changed, 0 insertions, 21 deletions
diff --git a/firmware/powermgmt.c b/firmware/powermgmt.c index 969d6167da..6cac300cdf 100644 --- a/firmware/powermgmt.c +++ b/firmware/powermgmt.c | |||
@@ -48,9 +48,6 @@ | |||
48 | #if (CONFIG_PLATFORM & PLATFORM_HOSTED) | 48 | #if (CONFIG_PLATFORM & PLATFORM_HOSTED) |
49 | #include <time.h> | 49 | #include <time.h> |
50 | #endif | 50 | #endif |
51 | #ifdef USB_ENABLE_HID | ||
52 | #include "usbstack/usb_hid.h" | ||
53 | #endif | ||
54 | 51 | ||
55 | #if (defined(IAUDIO_X5) || defined(IAUDIO_M5) || defined(COWON_D2)) \ | 52 | #if (defined(IAUDIO_X5) || defined(IAUDIO_M5) || defined(COWON_D2)) \ |
56 | && !defined (SIMULATOR) | 53 | && !defined (SIMULATOR) |
@@ -640,17 +637,6 @@ static void collect_power_history(void) | |||
640 | power_history[0] = power_hist_item(); | 637 | power_history[0] = power_hist_item(); |
641 | } | 638 | } |
642 | 639 | ||
643 | #ifdef USB_ENABLE_HID | ||
644 | static bool battery_reporting = false; | ||
645 | static int battery_report_percent = -1; | ||
646 | |||
647 | void set_battery_reporting(bool enable) | ||
648 | { | ||
649 | battery_reporting = enable; | ||
650 | battery_report_percent = -1; | ||
651 | } | ||
652 | #endif | ||
653 | |||
654 | /* | 640 | /* |
655 | * Monitor the presence of a charger and perform critical frequent steps | 641 | * Monitor the presence of a charger and perform critical frequent steps |
656 | * such as running the battery voltage filter. | 642 | * such as running the battery voltage filter. |
@@ -764,13 +750,6 @@ static void power_thread(void) | |||
764 | next_power_hist += HZ*60; | 750 | next_power_hist += HZ*60; |
765 | collect_power_history(); | 751 | collect_power_history(); |
766 | } | 752 | } |
767 | |||
768 | #ifdef USB_ENABLE_HID | ||
769 | if (battery_reporting && battery_report_percent != battery_percent) { | ||
770 | battery_report_percent = battery_percent; | ||
771 | usb_hid_send(HID_USAGE_PAGE_GENERIC_DEVICE_CONTROLS, battery_report_percent); | ||
772 | } | ||
773 | #endif | ||
774 | } | 753 | } |
775 | } /* power_thread */ | 754 | } /* power_thread */ |
776 | 755 | ||