Hello Raul Rangel, Jack Rosenthal,
I'd like you to reexamine a change. Please visit
https://review.coreboot.org/c/coreboot/+/45177
to look at the new patch set (#2).
Change subject: soc/amd/picasso: Adjust for memory Mhz fraction ......................................................................
soc/amd/picasso: Adjust for memory Mhz fraction
Memory speed is given as an integer, but in some cases it has an implicit fractional speed. Specifically, speeds ending in 33 are implicitly 33.333 and speeds ending in 67 are implicitly 66.666. So when multiplying by 2 to get MT/s, the result needs to be rounded up or down by one to account for this implicit fraction.
BUG=b:167155849 TEST=None
Signed-off-by: Rob Barnes robbarnes@google.com Change-Id: Icc77c21932c68ee9f0ff0b8e35ae7b1a3732b322 --- M src/soc/amd/picasso/dmi.c 1 file changed, 10 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/77/45177/2