diff options
author | Wai-Hong Tam <waihong@google.com> | 2022-12-15 10:44:38 -0800 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2022-12-15 20:55:12 +0000 |
commit | cddfff5e590d0d6015563e68eade601db93f03df (patch) | |
tree | 5cb160004af840c93c6cdee3b3c0cc566b09be15 | |
parent | 037afbe8dfeeaa18e40b1efd26592b98b309401d (diff) | |
download | chrome-ec-cddfff5e590d0d6015563e68eade601db93f03df.tar.gz |
herobrine: Defer initializing 5V rail
The 5V rail enable or GPIO EN_PP5000_S5 is enabled later in the
board_chipset logic for better timing on the first power in.
The GPIO driver should initialize this GPIO pin to LOW.
BRANCH=None
BUG=b:215726554, b:260192201
TEST=Unplug the battery. On the first power in, boot up to the AP.
Change-Id: Ie05b4d4196061d245066e571a1df6c33b6c0da02
Signed-off-by: Wai-Hong Tam <waihong@google.com>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/4112423
Reviewed-by: Sam Hurst <shurst@google.com>
Code-Coverage: Zoss <zoss-cl-coverage@prod.google.com>
Commit-Queue: Sam Hurst <shurst@google.com>
-rw-r--r-- | zephyr/program/herobrine/evoker/gpio.dtsi | 2 | ||||
-rw-r--r-- | zephyr/program/herobrine/gpio.dtsi | 2 | ||||
-rw-r--r-- | zephyr/program/herobrine/hoglin/gpio.dtsi | 2 | ||||
-rw-r--r-- | zephyr/program/herobrine/villager/gpio.dtsi | 2 | ||||
-rw-r--r-- | zephyr/program/herobrine/zombie/gpio.dtsi | 2 | ||||
-rw-r--r-- | zephyr/test/herobrine/boards/native_posix.overlay | 2 |
6 files changed, 6 insertions, 6 deletions
diff --git a/zephyr/program/herobrine/evoker/gpio.dtsi b/zephyr/program/herobrine/evoker/gpio.dtsi index 22919efea1..935ce8c719 100644 --- a/zephyr/program/herobrine/evoker/gpio.dtsi +++ b/zephyr/program/herobrine/evoker/gpio.dtsi @@ -123,7 +123,7 @@ enum-name = "GPIO_SWITCHCAP_ON"; }; gpio_en_pp5000_s5: en_pp5000_s5 { - gpios = <&gpio7 3 GPIO_OUTPUT_HIGH>; + gpios = <&gpio7 3 GPIO_OUTPUT_LOW>; enum-name = "GPIO_EN_PP5000"; }; ec_bl_disable_l { diff --git a/zephyr/program/herobrine/gpio.dtsi b/zephyr/program/herobrine/gpio.dtsi index 60a60ec658..e484e335cf 100644 --- a/zephyr/program/herobrine/gpio.dtsi +++ b/zephyr/program/herobrine/gpio.dtsi @@ -126,7 +126,7 @@ enum-name = "GPIO_SWITCHCAP_ON"; }; gpio_en_pp5000_s5: en_pp5000_s5 { - gpios = <&gpio7 3 GPIO_OUTPUT_HIGH>; + gpios = <&gpio7 3 GPIO_OUTPUT_LOW>; enum-name = "GPIO_EN_PP5000"; }; ec_bl_disable_l { diff --git a/zephyr/program/herobrine/hoglin/gpio.dtsi b/zephyr/program/herobrine/hoglin/gpio.dtsi index c742393ff8..8f75141f95 100644 --- a/zephyr/program/herobrine/hoglin/gpio.dtsi +++ b/zephyr/program/herobrine/hoglin/gpio.dtsi @@ -126,7 +126,7 @@ enum-name = "GPIO_SWITCHCAP_ON"; }; gpio_en_pp5000_s5: en_pp5000_s5 { - gpios = <&gpio7 3 GPIO_OUTPUT_HIGH>; + gpios = <&gpio7 3 GPIO_OUTPUT_LOW>; enum-name = "GPIO_EN_PP5000"; }; ec_bl_disable_l { diff --git a/zephyr/program/herobrine/villager/gpio.dtsi b/zephyr/program/herobrine/villager/gpio.dtsi index 0ffabf88f3..049ac74bcb 100644 --- a/zephyr/program/herobrine/villager/gpio.dtsi +++ b/zephyr/program/herobrine/villager/gpio.dtsi @@ -126,7 +126,7 @@ enum-name = "GPIO_SWITCHCAP_ON"; }; gpio_en_pp5000_s5: en_pp5000_s5 { - gpios = <&gpio7 3 GPIO_OUTPUT_HIGH>; + gpios = <&gpio7 3 GPIO_OUTPUT_LOW>; enum-name = "GPIO_EN_PP5000"; }; ec_bl_disable_l { diff --git a/zephyr/program/herobrine/zombie/gpio.dtsi b/zephyr/program/herobrine/zombie/gpio.dtsi index 711854fb79..ad60b945b9 100644 --- a/zephyr/program/herobrine/zombie/gpio.dtsi +++ b/zephyr/program/herobrine/zombie/gpio.dtsi @@ -126,7 +126,7 @@ enum-name = "GPIO_SWITCHCAP_ON"; }; gpio_en_pp5000_s5: en_pp5000_s5 { - gpios = <&gpio7 3 GPIO_OUTPUT_HIGH>; + gpios = <&gpio7 3 GPIO_OUTPUT_LOW>; enum-name = "GPIO_EN_PP5000"; }; ec_bl_disable_l { diff --git a/zephyr/test/herobrine/boards/native_posix.overlay b/zephyr/test/herobrine/boards/native_posix.overlay index d2afb38db1..37febb9516 100644 --- a/zephyr/test/herobrine/boards/native_posix.overlay +++ b/zephyr/test/herobrine/boards/native_posix.overlay @@ -15,7 +15,7 @@ compatible = "named-gpios"; gpio_en_pp5000_s5: en_pp5000_s5 { - gpios = <&gpio0 2 GPIO_OUTPUT_HIGH>; + gpios = <&gpio0 2 GPIO_OUTPUT_LOW>; enum-name = "GPIO_EN_PP5000"; }; }; |