Tim Chu has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/43338 )
Change subject: soc/intel/common/block/pmc: Select PMC on mainboard basis ......................................................................
soc/intel/common/block/pmc: Select PMC on mainboard basis
Currently the common soc code automatically selects POWER_STATE_DEFAULT_ON_AFTER_FAILURE which makes other options can not be selected in mainboard.
However, there're some cases that power state should change to different states after power reapplying.
Make POWER_STATE_DEFAULT_ON_AFTER_FAILURE default y but do not select it in soc code so that we can disable it and select other options in the mainboard code.
Tested on OCP Delta Lake.
Signed-off-by: Tim Chu Tim.Chu@quantatw.com Change-Id: Ifa853d81ee9477d2440ceaa67b2bd6b863ee52c2 --- M src/soc/intel/common/block/pmc/Kconfig 1 file changed, 3 insertions(+), 1 deletion(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/38/43338/1
diff --git a/src/soc/intel/common/block/pmc/Kconfig b/src/soc/intel/common/block/pmc/Kconfig index c2757e6..63e290f 100644 --- a/src/soc/intel/common/block/pmc/Kconfig +++ b/src/soc/intel/common/block/pmc/Kconfig @@ -4,11 +4,13 @@ bool select HAVE_POWER_STATE_AFTER_FAILURE select HAVE_POWER_STATE_PREVIOUS_AFTER_FAILURE - select POWER_STATE_DEFAULT_ON_AFTER_FAILURE help Intel Processor common code for Power Management controller(PMC) subsystem
+config POWER_STATE_DEFAULT_ON_AFTER_FAILURE + default y + config PMC_INVALID_READ_AFTER_WRITE bool default n