diff options
author | johnwc_yeh <johnwc_yeh@compal.corp-partner.google.com> | 2022-12-14 13:31:44 +0800 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-12-15 02:31:45 +0000 |
commit | 3a71dfa983df103a7435c0f1ff0108beb1d99e77 (patch) | |
tree | 03ad90a9d3a92417fa661a5e505c1593a0004be7 | |
parent | f672c6d633a40e5390c6113f0b4bb684ffd719f9 (diff) | |
download | chrome-ec-3a71dfa983df103a7435c0f1ff0108beb1d99e77.tar.gz |
Frostflow: 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:258153920
BRANCH=none
TEST=zmake build frostflow
Change-Id: Iec114b47109f57f15db09dcc80cdb077c4ab884b
Signed-off-by: johnwc_yeh <johnwc_yeh@compal.corp-partner.google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4105800
Reviewed-by: Diana Z <dzigterman@chromium.org>
Code-Coverage: Zoss <zoss-cl-coverage@prod.google.com>
Reviewed-by: SamSP Liu <samsp_liu2@compal.corp-partner.google.com>
-rw-r--r-- | zephyr/program/skyrim/frostflow/led_policy.dtsi | 32 |
1 files changed, 24 insertions, 8 deletions
diff --git a/zephyr/program/skyrim/frostflow/led_policy.dtsi b/zephyr/program/skyrim/frostflow/led_policy.dtsi index e5875640fb..b5ee415d6d 100644 --- a/zephyr/program/skyrim/frostflow/led_policy.dtsi +++ b/zephyr/program/skyrim/frostflow/led_policy.dtsi @@ -4,8 +4,21 @@ 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 +27,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 +49,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>; @@ -44,8 +60,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 { |