[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Adjust for memory Mhz fraction

[Furquan Shaikh (Code Review)]

Furquan Shaikh has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45177 )

Change subject: soc/amd/picasso: Adjust for memory Mhz fraction
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45177/2/src/soc/amd/picasso/dmi.c 
File src/soc/amd/picasso/dmi.c:

https://review.coreboot.org/c/coreboot/+/45177/2/src/soc/amd/picasso/dmi.c@3... 
PS2, Line 34: 	 * e.g. 1333 Mhz == 1333.333 Mhz == 2666.666 MT/s == 26667 MT/s
What if it is 1466MHz which comes out to be 2933MT/s. I think it might be better to just have a table that does the conversion instead of doing the math here? Or just handle it in mosys as Jack commented.