Merge pull request #1039 from FrameworkComputer/marigold.me_control

marigold: modify me_en control

Author: JohnAZoidberg

zephyr/program/framework/marigold/gpio.dtsi

@@ -200,7 +200,7 @@
 		};
 		gpio_me_en: me_en {
 			/* ME flash, strap default low before PLT_PWROK*/
-			gpios = <&gpio7 4 (GPIO_OUTPUT_LOW | GPIO_VOLTAGE_1P8)>;
+			gpios = <&gpio7 4 GPIO_OUTPUT_LOW>;
 		};
 		gpio_mux_sub_uart_flip: mux_sub_uart_flip {
 			gpios = <&gpiod 5 GPIO_OUTPUT_LOW>;	/*TBT SBU MUX*/

zephyr/program/framework/marigold/src/power_sequence.c

@@ -302,6 +302,11 @@ static void module_pwr_control_enable(bool state)
 		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_module_pwr_on), 0);
 }
 
+void me_gpio_change(uint32_t flags)
+{
+	gpio_pin_configure_dt(GPIO_DT_FROM_NODELABEL(gpio_me_en), flags);
+}
+
 enum power_state power_handle_state(enum power_state state)
 {
 	switch (state) {
@@ -320,10 +325,10 @@ enum power_state power_handle_state(enum power_state state)
 			return POWER_G3;
 		}
 
-		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_me_en), !!(me_change & ME_UNLOCK));
-
 		k_msleep(10);
 		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_ec_soc_rsmrst_l), 1);
+		me_gpio_change(me_change & ME_UNLOCK ? GPIO_OUTPUT_HIGH : GPIO_OUTPUT_LOW);
+
 		if (extpower_is_present())
 			gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_ac_present), 1);
 
@@ -407,6 +412,7 @@ enum power_state power_handle_state(enum power_state state)
 
 		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_pch_pwrok_ls), 1);
 		k_msleep(10);
+		me_gpio_change(GPIO_INPUT);
 		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_sys_pwrok_ls), 1);
 
 		/* Call hooks now that rails are up */
@@ -473,11 +479,10 @@ enum power_state power_handle_state(enum power_state state)
 	case POWER_S0S3:
 		k_msleep(5);
 		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_susp_l), 0);
+		me_gpio_change(GPIO_OUTPUT_LOW);
 		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_pch_pwrok_ls), 0);
 		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_sys_pwrok_ls), 0);
 
-		gpio_pin_set_dt(GPIO_DT_FROM_NODELABEL(gpio_me_en), 0);
-
 		lpc_s0ix_suspend_clear_masks();
 		/* Call hooks before we remove power rails */
 		hook_notify(HOOK_CHIPSET_SUSPEND);