[coreboot-gerrit] Change in ...coreboot[master]: soc/intel/cannonlake: Clear PMCON status bits

[Lijian Zhao (Code Review)]

Lijian Zhao has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31902 )

Change subject: soc/intel/cannonlake: Clear PMCON status bits
......................................................................

Patch Set 4:

(1 comment)

https://review.coreboot.org/#/c/31902/3/src/soc/intel/cannonlake/pmutil.c 
File src/soc/intel/cannonlake/pmutil.c:

https://review.coreboot.org/#/c/31902/3/src/soc/intel/cannonlake/pmutil.c@14... 
PS3, Line 146: pmc_clear_suspwrflr
> the appropriate place would be pch_finalize(), by that point FSP notify is also done.

I think FSP will clear the PWR_FLR status itself anyway right? In PchOnPciEnumCompleteCommon function