diff options
| author | Parth Malkan <parthmalkan@google.com> | 2022-05-26 01:12:11 +0000 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-06-03 01:29:42 +0000 |
| commit | 407ddc17fdcb32042148d4e548324092da9d29c3 (patch) | |
| tree | fdcbebf0aff3ce7a4357d742c2a4a2cb0128339a | |
| parent | 8f2fa112010f6f52413975c72e7a22a3ecdbfdbe (diff) | |
| download | chrome-ec-407ddc17fdcb32042148d4e548324092da9d29c3.tar.gz | |
zephyr: herobrine: Define battery-level property as int range
Currently battery-level property is defined using enums. This patch
changes this to use int range instead.
BRANCH=None
BUG=b:233953617
TEST=Manual test on Lazor by modifying nodes to include battery-level
range, use battfake to set different levels and observe LED behavior.
Cq-Depend: chromium:3669088
Signed-off-by: Parth Malkan <parthmalkan@google.com>
Change-Id: I18ef922a41ce17d509f03a1774e9d9437fc7065d
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3669089
Reviewed-by: Wai-Hong Tam <waihong@google.com>
| -rw-r--r-- | zephyr/projects/herobrine/led_policy_herobrine.dts | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/zephyr/projects/herobrine/led_policy_herobrine.dts b/zephyr/projects/herobrine/led_policy_herobrine.dts index 0e8c617eed..7e296e6a3a 100644 --- a/zephyr/projects/herobrine/led_policy_herobrine.dts +++ b/zephyr/projects/herobrine/led_policy_herobrine.dts @@ -1,3 +1,5 @@ +#include <dt-bindings/battery.h> + / { led-colors { compatible = "cros-ec,led-colors"; @@ -30,9 +32,10 @@ }; }; - power-state-discharge-right-0 { + power-state-discharge-right-low { charge-state = "PWR_STATE_DISCHARGE"; - extra-flag = "LED_BATT_BELOW_10_PCT"; + /* Battery percent range (>= Empty, <= Low) */ + batt-lvl = <BATTERY_LEVEL_EMPTY BATTERY_LEVEL_LOW>; /* Turn off the left LED */ color-0 { @@ -49,9 +52,10 @@ }; }; - power-state-discharge-right-1 { + power-state-discharge-right { charge-state = "PWR_STATE_DISCHARGE"; - extra-flag = "LED_BATT_ABOVE_10_PCT"; + /* Battery percent range (> Low, <= Full) */ + batt-lvl = <(BATTERY_LEVEL_LOW + 1) BATTERY_LEVEL_FULL>; /* Turn off the left LED */ color-0 { |