[coreboot-gerrit] [S] Change in coreboot[master]: soc/intel/meteorlake: Add PM Energy Report feature option

[Yong Zhi (Code Review)]

Attention is currently required from: Tarun Tuli, Subrata Banik, Ravishankar Sarawadi, Kapil Porwal, Eric Lai.

Yong Zhi has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/73061 )

Change subject: soc/intel/meteorlake: Add PM Energy Report feature option
......................................................................

Patch Set 1:

(1 comment)

File src/soc/intel/meteorlake/fsp_params.c:

https://review.coreboot.org/c/coreboot/+/73061/comment/7d6d02fe_807c9ac7 
PS1, Line 649: 	s_cfg->PchPmDisableEnergyReport = !config->pch_pm_energy_report_enable;
> Do you want default is disable? If no one set this, it will be true.

That's correct, default value of PchPmDisableEnergyReport in FSP is 1, the feature is intended for power instrumentation of eng build/RVP.