diff options
| author | JohnWC Yeh <johnwc_yeh@compal.corp-partner.google.com> | 2023-04-24 02:56:40 +0000 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-04-24 04:35:38 +0000 |
| commit | e2bbc0a0587c04462c7a002ca59041bc36618968 (patch) | |
| tree | e52a76ade117ea308be850433772adcac9b26259 | |
| parent | de9944a9e66695f94916aa040d654a299198bf72 (diff) | |
| download | chrome-ec-e2bbc0a0587c04462c7a002ca59041bc36618968.tar.gz | |
Revert "Xivu: Implement LED behavior for RSOC"
This reverts commit 28052e2944fe30729d584cb2d900c39073399cd0.
Reason for revert: Already change the node depends on display SOC
Original change's description:
> 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>
Bug: b:237224850
Change-Id: I4159020e389af538fe4953aa86538e6d11d1c27e
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4461253
Reviewed-by: Peter Marheine <pmarheine@chromium.org>
Tested-by: JohnWC Yeh <johnwc_yeh@compal.corp-partner.google.com>
Commit-Queue: JohnWC Yeh <johnwc_yeh@compal.corp-partner.google.com>
| -rw-r--r-- | zephyr/program/nissa/xivu/led_policy.dtsi | 33 |
1 files changed, 8 insertions, 25 deletions
diff --git a/zephyr/program/nissa/xivu/led_policy.dtsi b/zephyr/program/nissa/xivu/led_policy.dtsi index dd501b2ee4..562e361ec5 100644 --- a/zephyr/program/nissa/xivu/led_policy.dtsi +++ b/zephyr/program/nissa/xivu/led_policy.dtsi @@ -4,22 +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>; @@ -28,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>; @@ -50,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>; @@ -85,8 +68,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 { |