[coreboot-gerrit] Change in coreboot[master]: device/dram: Add method for converting MHz to MT/s

[Paul Fagerburg (Code Review)]

Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45343 )

Change subject: device/dram: Add method for converting MHz to MT/s
......................................................................

Patch Set 3:

(1 comment)

https://review.coreboot.org/c/coreboot/+/45343/3/src/device/dram/ddr4.c 
File src/device/dram/ddr4.c:

https://review.coreboot.org/c/coreboot/+/45343/3/src/device/dram/ddr4.c@69 
PS3, Line 69: 1466
> Table 170, DDR4-2933: min_tck >= 0.682 ns, max_tck < 0.750 ns --> max_mhz<1466.28 min_mhz>=1333.33. […]

No, just questioning since 1067 will get doubled to 2133, should we not have 2933 include 1467 MHz and not stop at 1466 MHz?