From 18fc3aab5262dc0e66f0b84f4feaebbacfd818a1 Mon Sep 17 00:00:00 2001 From: Parth Malkan Date: Thu, 26 May 2022 01:17:28 +0000 Subject: zephyr: skyrim: 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 Change-Id: I502b7a85420b0c017494739a977c468fd009dfde Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/3669090 Reviewed-by: Wai-Hong Tam --- zephyr/projects/skyrim/led_policy_skyrim.dts | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/zephyr/projects/skyrim/led_policy_skyrim.dts b/zephyr/projects/skyrim/led_policy_skyrim.dts index 55e1da330b..da3d817925 100644 --- a/zephyr/projects/skyrim/led_policy_skyrim.dts +++ b/zephyr/projects/skyrim/led_policy_skyrim.dts @@ -1,3 +1,5 @@ +#include + / { led-colors { compatible = "cros-ec,led-colors"; @@ -13,7 +15,8 @@ power-state-discharge-s0 { charge-state = "PWR_STATE_DISCHARGE"; chipset-state = "POWER_S0"; - extra-flag = "LED_BATT_ABOVE_10_PCT"; + /* Battery percent range (> Low, <= Full) */ + batt-lvl = <(BATTERY_LEVEL_LOW + 1) BATTERY_LEVEL_FULL>; color-0 { led-color = <&color_white>; @@ -23,7 +26,8 @@ power-state-discharge-s0-batt-low { charge-state = "PWR_STATE_DISCHARGE"; chipset-state = "POWER_S0"; - extra-flag = "LED_BATT_BELOW_10_PCT"; + /* Battery percent range (>= Empty, <= Low) */ + batt-lvl = ; /* White 2 sec, off 1 sec */ color-0 { -- cgit v1.2.1