diff options
author | JohnWC Yeh <johnwc_yeh@compal.corp-partner.google.com> | 2023-04-24 02:57:54 +0000 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-04-25 13:17:10 +0000 |
commit | 592ef3fc7b49a0bca9e9d59c5372571b3fe25c47 (patch) | |
tree | 35b7d22c7d5bd82427ed5fb54866203682958047 | |
parent | 229b251ffa8c104e190b1bfa6ba6cb011dcacad4 (diff) | |
download | chrome-ec-592ef3fc7b49a0bca9e9d59c5372571b3fe25c47.tar.gz |
Revert "Frostflow: Implement LED behavior for RSOC"
This reverts commit 3a71dfa983df103a7435c0f1ff0108beb1d99e77.
Reason for revert: Already change the node depends on display SOC
Original change's description:
> 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>
Bug: b:258153920
Change-Id: I918f72998686d9699a757b129bb095a5e1dfb96c
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4461254
Reviewed-by: Diana Z <dzigterman@chromium.org>
Reviewed-by: SamSP Liu <samsp_liu2@compal.corp-partner.google.com>
Tested-by: JohnWC Yeh <johnwc_yeh@compal.corp-partner.google.com>
Commit-Queue: JohnWC Yeh <johnwc_yeh@compal.corp-partner.google.com>
(cherry picked from commit 7659d9fe9486d0ef938d7fe4ba4e10dcce5aa4f5)
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4473611
Reviewed-by: Chao Gui <chaogui@google.com>
Commit-Queue: Chao Gui <chaogui@google.com>
-rw-r--r-- | zephyr/program/skyrim/frostflow/led_policy.dtsi | 32 |
1 files changed, 8 insertions, 24 deletions
diff --git a/zephyr/program/skyrim/frostflow/led_policy.dtsi b/zephyr/program/skyrim/frostflow/led_policy.dtsi index 901d4f4372..28e33219a0 100644 --- a/zephyr/program/skyrim/frostflow/led_policy.dtsi +++ b/zephyr/program/skyrim/frostflow/led_policy.dtsi @@ -4,21 +4,8 @@ power-state-charge { charge-state = "PWR_STATE_CHARGE"; - /* - * 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>; + /* Battery percent range (>= Empty, <= 94%) */ + batt-lvl = <0 94>; color-0 { led-color = <&color_amber>; @@ -27,11 +14,8 @@ power-state-charge-lvl-2 { charge-state = "PWR_STATE_CHARGE"; - /* - * Battery display percent range - * (>= 95%, <= Near Full) - */ - batt-lvl = <92 97>; + /* Battery percent range (>= 95%, <= Near Full) */ + batt-lvl = <95 97>; color-0 { led-color = <&color_white>; @@ -49,8 +33,8 @@ power-state-discharge-s0 { charge-state = "PWR_STATE_DISCHARGE"; chipset-state = "POWER_S0"; - /* Battery display percent range (>= 11%, <= Full) */ - batt-lvl = <14 100>; + /* Battery percent range (>= 11%, <= Full) */ + batt-lvl = <11 100>; color-0 { led-color = <&color_white>; @@ -60,8 +44,8 @@ power-state-discharge-s0-batt-low { charge-state = "PWR_STATE_DISCHARGE"; chipset-state = "POWER_S0"; - /* Battery display percent range (>= Empty, <= 10%) */ - batt-lvl = <0 13>; + /* Battery percent range (>= Empty, <= 10%) */ + batt-lvl = <0 10>; /* Amber 1 sec, off 3 sec */ color-0 { |