[coreboot-gerrit] [XS] Change in coreboot[main]: soc/intel/ehl/fsp_params: Do not re-enable 'PchPwrOptEnable' for real...

[Mario Scheithauer (Code Review)]

Attention is currently required from: Lean Sheng Tan, Paul Menzel.

Mario Scheithauer has posted comments on this change by Mario Scheithauer. ( https://review.coreboot.org/c/coreboot/+/84399?usp=email )

Change subject: soc/intel/ehl/fsp_params: Do not re-enable 'PchPwrOptEnable' for real-time tuning
......................................................................

Patch Set 4:

(2 comments)

Commit Message:

https://review.coreboot.org/c/coreboot/+/84399/comment/b5ae8120_bd332e13?usp... :
PS3, Line 7: Fix double setting of 'PchPwrOptEnable'
> I misunderstood the summary. Maybe something like: […]

Done

https://review.coreboot.org/c/coreboot/+/84399/comment/4e0df6df_4f5436fb?usp... :
PS3, Line 9: If real-time tuning was enabled, 'PchPwrOptEnable' was set two times.
> … with different values.

Done