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

[Kyösti Mälkki (Code Review)]

Kyösti Mälkki has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33765 )

Change subject: soc/amd/picasso: Update calculation for TSC frequency
......................................................................

Patch Set 4:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/33765/4/src/soc/amd/picasso/tsc_fre... 
PS4, Line 38: 	return (100 * (cpufid + 0x10)) / (0x01 << cpudid);
If this is constant, you may want to cache the result (declare static variable for return value) as rdmsr is sometimes heavy for execution and worth skipping when possible.