From 132fb3cc52884fa7634d7a9f6406eab3b4ea58d0 Mon Sep 17 00:00:00 2001 From: Sean Rhodes Date: Thu, 16 Feb 2023 08:20:20 +0000 Subject: Revert "mb/starlabs/*: Enable the Mirror flag for boards that support" This reverts commit 35354583cdc94ced026975ced5170e1c094b258e. Reason for revert: The mirror flag "0x01" is mirror once, which relies on the EC remembering that it's been mirrored. However, the EC forgets this if it's been without power for 20 minutes or so. Even if power is connected then, it'll instantly try to mirror and it can't charge whilst doing it. It can either result in incomplete EC firmware, or a loop where it's constantly trying to mirror. Change-Id: Ie82cbafd4bea2416526e2847738802a05ed45582 Signed-off-by: Sean Rhodes Reviewed-on: https://review.coreboot.org/c/coreboot/+/72677 Tested-by: build bot (Jenkins) Reviewed-by: Angel Pons --- src/mainboard/starlabs/starbook/Kconfig | 2 -- src/mainboard/starlabs/starbook/cmos.default | 1 - src/mainboard/starlabs/starbook/cmos.layout | 1 - src/mainboard/starlabs/starbook/variants/tgl/cmos.default | 1 - src/mainboard/starlabs/starbook/variants/tgl/cmos.layout | 1 - 5 files changed, 6 deletions(-) (limited to 'src/mainboard/starlabs') diff --git a/src/mainboard/starlabs/starbook/Kconfig b/src/mainboard/starlabs/starbook/Kconfig index 807b6a945ae8..15b2a585e2e0 100644 --- a/src/mainboard/starlabs/starbook/Kconfig +++ b/src/mainboard/starlabs/starbook/Kconfig @@ -29,7 +29,6 @@ config BOARD_STARLABS_LABTOP_CML select BOARD_ROMSIZE_KB_16384 select BOARD_STARLABS_STARBOOK_SERIES select EC_STARLABS_MAX_CHARGE - select EC_STARLABS_MIRROR_FLAG select EC_STARLABS_NEED_ITE_BIN select HAVE_INTEL_PTT select HAVE_SPD_IN_CBFS @@ -45,7 +44,6 @@ config BOARD_STARLABS_STARBOOK_TGL select DRIVERS_INTEL_USB4_RETIMER select EC_STARLABS_KBL_LEVELS select EC_STARLABS_MAX_CHARGE - select EC_STARLABS_MIRROR_FLAG select EC_STARLABS_NEED_ITE_BIN select MEMORY_MAPPED_TPM select SOC_INTEL_COMMON_BLOCK_TCSS diff --git a/src/mainboard/starlabs/starbook/cmos.default b/src/mainboard/starlabs/starbook/cmos.default index b7f77e4adcca..f1276cc6ed81 100644 --- a/src/mainboard/starlabs/starbook/cmos.default +++ b/src/mainboard/starlabs/starbook/cmos.default @@ -15,7 +15,6 @@ legacy_8254_timer=Disable # EC kbl_timeout=30 seconds fn_ctrl_swap=Disable -mirror_flag=Enable # southbridge power_on_after_fail=Disable # Functions diff --git a/src/mainboard/starlabs/starbook/cmos.layout b/src/mainboard/starlabs/starbook/cmos.layout index 8b6a463075ee..887c0b21b9f6 100644 --- a/src/mainboard/starlabs/starbook/cmos.layout +++ b/src/mainboard/starlabs/starbook/cmos.layout @@ -34,7 +34,6 @@ entries 608 1 e 1 fn_ctrl_swap 616 2 e 8 max_charge 624 2 e 9 fan_mode -632 1 e 1 mirror_flag # coreboot config options: southbridge 800 2 e 6 power_on_after_fail diff --git a/src/mainboard/starlabs/starbook/variants/tgl/cmos.default b/src/mainboard/starlabs/starbook/variants/tgl/cmos.default index 62bb28d411a1..cfc9a612fb3a 100644 --- a/src/mainboard/starlabs/starbook/variants/tgl/cmos.default +++ b/src/mainboard/starlabs/starbook/variants/tgl/cmos.default @@ -16,7 +16,6 @@ thunderbolt=Disable # EC kbl_timeout=30 seconds fn_ctrl_swap=Disable -mirror_flag=Enable # southbridge power_on_after_fail=Disable # Functions diff --git a/src/mainboard/starlabs/starbook/variants/tgl/cmos.layout b/src/mainboard/starlabs/starbook/variants/tgl/cmos.layout index d792426dc94c..c4a49c5dbad0 100644 --- a/src/mainboard/starlabs/starbook/variants/tgl/cmos.layout +++ b/src/mainboard/starlabs/starbook/variants/tgl/cmos.layout @@ -35,7 +35,6 @@ entries 608 1 e 1 fn_ctrl_swap 616 2 e 8 max_charge 624 2 e 9 fan_mode -632 1 e 1 mirror_flag # coreboot config options: southbridge 800 2 e 6 power_on_after_fail -- cgit v1.2.3