Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/42677 )
Change subject: soc/intel/tigerlake: Change pmc_set_acpi_mode() run after BS_DEV_INIT ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/c/coreboot/+/42677/2/src/soc/intel/tigerlake/pmc... File src/soc/intel/tigerlake/pmc.c:
https://review.coreboot.org/c/coreboot/+/42677/2/src/soc/intel/tigerlake/pmc... PS2, Line 146: /* : * PMC initialization happens earlier for this SoC because FSP-Silicon : * init hides PMC from PCI bus. However, pmc_set_acpi_mode, which : * disables ACPI mode doesn't need to happen that early and can be : * delayed till typical BS_DEV_INIT. This ensures that ACPI mode : * disabling happens the same way for all SoCs and hence the ordering of : * events is the same. : * : * This is important to ensure that the ordering does not break the : * assumptions of any other drivers (e.g. ChromeEC) which could be : * taking different actions based on disabling of ACPI (e.g. flushing of : * all EC hostevent bits). : * : * P.S.: This cannot be done as part of pmc_soc_init as PMC device is : * hidden and hence the PMC driver never gets enumerated and so init is : * not called for it. : */
It looks like this comment is from Cannonlake/pmc.c. […]
yes cnl onwards all pch