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

[Paul Menzel (Code Review)]

Paul Menzel 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;
> It should not really happen. But 0 is a valid value which indicates that clock is off. […]

If it’s an condition, that shouldn’t happen, I think at least a warning or notice should be printed.