fwk: fix power on AC attach requiring OS boot to take effect

Previously, you needed to boot an OS first for the settings to kick in.
So if you remove SSDs, the setting will not kick in.

Add a host command EC_CMD_POWER_ON_AC_ATTACH(0x3E2A) only for power
on AC attach, use for BIOS to notify EC to save it.

BRANCH=fwk-dogwood-27111
BUG=https://app.clickup.com/t/86evbbru9
TEST=1.unplug all SSDs and other storage
2. enable "power on ac attach"  f10, save, reboot
3. unplug AC power and wait a few seconds
4. plug AC power can power on

Signed-off-by: johnwc_yeh <JohnWC_Yeh@compal.com>

Author: JohnWC-Yeh

zephyr/program/framework/include/board_host_command.h

@@ -641,5 +641,14 @@ struct ec_response_psu_control {
 	uint8_t force_enable_in_standby;
 } __ec_align1;
 
+/*****************************************************************************/
+/*
+ * This command uses to control the Power on AC attach enable/disable
+ */
+#define EC_CMD_POWER_ON_AC_ATTACH 0x3E2A
+
+struct ec_params_power_on_ac_attach {
+	uint8_t enable;
+} __ec_align1;
 
 #endif /* __BOARD_HOST_COMMAND_H */

zephyr/program/framework/src/board_function.c

@@ -57,14 +57,11 @@ int bios_function_status(uint16_t type, uint16_t addr, uint8_t flag)
  */
 int ac_boot_status(void)
 {
-	return bios_function_status(TYPE_MEMMAP, EC_CUSTOMIZED_MEMMAP_BIOS_SETUP_FUNC,
-		EC_AC_ATTACH_BOOT);
+	return bios_function_status(TYPE_FLASH, FLASH_FLAGS_ACPOWERON, 0);
 }
 
 void bios_function_detect(void)
 {
-	flash_storage_update(FLASH_FLAGS_ACPOWERON, ac_boot_status());
-
 #ifndef CONFIG_PLATFORM_EC_FRAMEWORK_MINI_PC
 	flash_storage_update(FLASH_FLAGS_STANDALONE, get_standalone_mode() ? 1 : 0);
 #else
@@ -121,10 +118,6 @@ void board_power_button_interrupt(enum gpio_signal signal)
 
 static void bios_function_init(void)
 {
-	/* restore the bios setup menu setting */
-	*host_get_memmap(EC_CUSTOMIZED_MEMMAP_BIOS_SETUP_FUNC) =
-		flash_storage_get(FLASH_FLAGS_ACPOWERON);
-
 #ifndef CONFIG_PLATFORM_EC_FRAMEWORK_MINI_PC
 	if (flash_storage_get(FLASH_FLAGS_STANDALONE))
 		set_standalone_mode(1);

zephyr/program/framework/src/board_host_command.c

@@ -454,6 +454,25 @@ static enum ec_status cmd_get_ap_throttle_status(struct host_cmd_handler_args *a
 }
 DECLARE_HOST_COMMAND(EC_CMD_GET_AP_THROTTLE_STATUS, cmd_get_ap_throttle_status, EC_VER_MASK(0));
 
+static enum ec_status power_on_ac_attach_control(struct host_cmd_handler_args *args)
+{
+	const struct ec_params_power_on_ac_attach *p = args->params;
+	int enable = p->enable;
+	int status;
+
+	status = flash_storage_get(FLASH_FLAGS_ACPOWERON);
+
+	/* Update the flash if status changes */
+	if (enable != status) {
+		/* Store the status in EC ROM and restore when EC power on */
+		flash_storage_update(FLASH_FLAGS_ACPOWERON, enable);
+		flash_storage_commit();
+	}
+
+	return EC_SUCCESS;
+}
+DECLARE_HOST_COMMAND(EC_CMD_POWER_ON_AC_ATTACH, power_on_ac_attach_control, EC_VER_MASK(0));
+
 static enum ec_status cmd_get_pd_port_state(struct host_cmd_handler_args *args)
 {
 	const struct ec_params_get_pd_port_state *p = args->params;