diff options
author | johnwc_yeh <johnwc_yeh@compal.corp-partner.google.com> | 2022-12-13 16:34:47 +0800 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-12-14 04:47:48 +0000 |
commit | 28052e2944fe30729d584cb2d900c39073399cd0 (patch) | |
tree | 9e8421bde28e72157070e096ce417ced3a66a115 | |
parent | 5247059d1e3c767392ff75bec6dd7945b35b180d (diff) | |
download | chrome-ec-28052e2944fe30729d584cb2d900c39073399cd0.tar.gz |
Xivu: Implement LED behavior for RSOC
The EC modifies the raw battery level to a 'Display'
level that is then sent to the AP.
Battery Display 0-94%:Amber
Battery Low Display 0-10%
The Display values map to raw values as:
Display Raw
10 13
11 14
94 91
95 92
BUG=b:237224850
BRANCH=none
TEST=zmake build xivu
Change-Id: I28417287710057ac7f37d932251d9e8dc392dfd2
Signed-off-by: johnwc_yeh <johnwc_yeh@compal.corp-partner.google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4100346
Code-Coverage: Zoss <zoss-cl-coverage@prod.google.com>
Reviewed-by: SamSP Liu <samsp_liu2@compal.corp-partner.google.com>
Reviewed-by: Andrew McRae <amcrae@google.com>
-rw-r--r-- | zephyr/program/nissa/xivu/led_policy.dtsi | 33 |
1 files changed, 25 insertions, 8 deletions
diff --git a/zephyr/program/nissa/xivu/led_policy.dtsi b/zephyr/program/nissa/xivu/led_policy.dtsi index 562e361ec5..dd501b2ee4 100644 --- a/zephyr/program/nissa/xivu/led_policy.dtsi +++ b/zephyr/program/nissa/xivu/led_policy.dtsi @@ -4,8 +4,22 @@ power-state-charge { charge-state = "PWR_STATE_CHARGE"; - /* Battery percent range (>= Empty, <= 94%) */ - batt-lvl = <0 94>; + /* + * The EC modifies the raw battery level to a 'Display' + * level that is then sent to the AP. + * The battery percentages that the LED driver uses for + * comparison is the raw level, but the Display values + * are the ones used to determine the LED thresholds. + * The Display values map to raw values as: + * Display Raw + * 10% 13% + * 11% 14% + * 94% 91% + * 95% 92% + */ + + /* Battery display percent range (>= Empty, <= 94%) */ + batt-lvl = <0 91>; color-0 { led-color = <&color_amber>; @@ -14,8 +28,11 @@ power-state-charge-lvl-2 { charge-state = "PWR_STATE_CHARGE"; - /* Battery percent range (>= 95%, <= Near Full) */ - batt-lvl = <95 97>; + /* + * Battery display percent range + * (>= 95%, <= Near Full) + */ + batt-lvl = <92 97>; color-0 { led-color = <&color_white>; @@ -33,8 +50,8 @@ power-state-discharge-s0 { charge-state = "PWR_STATE_DISCHARGE"; chipset-state = "POWER_S0"; - /* Battery percent range (>= 11%, <= Full) */ - batt-lvl = <11 100>; + /* Battery display percent range (>= 11%, <= Full) */ + batt-lvl = <14 100>; color-0 { led-color = <&color_white>; @@ -68,8 +85,8 @@ power-state-discharge-s0-batt-low { charge-state = "PWR_STATE_DISCHARGE"; chipset-state = "POWER_S0"; - /* Battery percent range (>= Empty, <= 10%) */ - batt-lvl = <0 10>; + /* Battery display percent range (>= Empty, <= 10%) */ + batt-lvl = <0 13>; /* Amber 1 sec, off 3 sec */ color-0 { |