[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Fix TSC frequency calculation

[Nikolai Vyssotski (Code Review)]

Nikolai Vyssotski has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45055 )

Change subject: soc/amd/picasso: Fix TSC frequency calculation
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45055/1/src/soc/amd/picasso/tsc_fre... 
File src/soc/amd/picasso/tsc_freq.c:

https://review.coreboot.org/c/coreboot/+/45055/1/src/soc/amd/picasso/tsc_fre... 
PS1, Line 29: 		mhz = 0;
> Is this possible? Print an error?

It should not really happen. But 0 is a valid value which indicates that clock is off. We should not be able to run in this case. Do you think we still need to add printk or die here?