soc/intel/cannonlake: Disable ACPI mode on BS_DEV_INIT exit
commit67a489fdb058acfeda6e453ef145d8ac4cdc5971
authorFurquan Shaikh <furquan@google.com>
Wed, 27 Feb 2019 08:59:06 +0000 (27 00:59 -0800)
committerPatrick Georgi <pgeorgi@google.com>
Thu, 28 Feb 2019 13:43:45 +0000 (28 13:43 +0000)
treeb1bd819f4f3b66f1d0ab09799615b36d8ee2a3e9
parenta198c9d732e1eedd801952108823a5a94ede2839
soc/intel/cannonlake: Disable ACPI mode on BS_DEV_INIT exit

Change ac8c60e (soc/intel/cannonlake: Disable ACPI mode as part of
pmc_soc_init) moved disabling of ACPI mode to pmc_soc_init to keep it
more aligned with the behavior on other Intel SoCs. However, as the
PMC device is hidden, it never gets enumerated and so init function
does not get called for it. This change moves the call to disable ACPI
mode to exit of BS_DEV_INIT instead.

BUG=b:126016602
TEST=Verified that:
1. pmc_set_acpi_mode is actually getting called.
2. EC panic event gets logged to eventlog correctly.

Change-Id: Ie7025e322fa0abc21367a520184a4c7741eba1e6
Signed-off-by: Furquan Shaikh <furquan@google.com>
Reviewed-on: https://review.coreboot.org/c/31633
Reviewed-by: Subrata Banik <subrata.banik@intel.com>
Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
src/soc/intel/cannonlake/pmc.c