[coreboot-gerrit] Change in coreboot[master]: x86/tsc: Only call tsc_freq_mhz once to init clock

[Aaron Durbin (Code Review)]

Aaron Durbin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/36664 )

Change subject: x86/tsc: Only call tsc_freq_mhz once to init clock
......................................................................

Patch Set 1:
> Patch Set 1:
>
> I need to see the arguments why this was abandoned. Or in other words, where was the fix? Are we hitting some CPU errata or undisclosed silicon bug?

The failure was occurring after FSP-S requested a reset. Inside of FSP it was applying power management configuration. Prior to the mainboard policy being deployed the reset would happen. On that subsequent boot we were missing h1 interrupts because of the of power management policy. I cannot claim that I understand why this CL would help in such scenarios, though.

https://review.coreboot.org/c/coreboot/+/37319 was the solution people came up with for ensuring the mainboard power management policy was provisioned.