summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWai-Hong Tam <waihong@google.com>2022-12-15 10:44:38 -0800
committerChromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com>2022-12-15 20:55:12 +0000
commitcddfff5e590d0d6015563e68eade601db93f03df (patch)
tree5cb160004af840c93c6cdee3b3c0cc566b09be15
parent037afbe8dfeeaa18e40b1efd26592b98b309401d (diff)
downloadchrome-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.dtsi2
-rw-r--r--zephyr/program/herobrine/gpio.dtsi2
-rw-r--r--zephyr/program/herobrine/hoglin/gpio.dtsi2
-rw-r--r--zephyr/program/herobrine/villager/gpio.dtsi2
-rw-r--r--zephyr/program/herobrine/zombie/gpio.dtsi2
-rw-r--r--zephyr/test/herobrine/boards/native_posix.overlay2
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";
};
};