Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39851 )
Change subject: nb/intel/sandybridge: Refactor get_mem_min_tck ......................................................................
Patch Set 7: Code-Review+2
(1 comment)
Looks good
https://review.coreboot.org/c/coreboot/+/39851/7/src/northbridge/intel/sandy... File src/northbridge/intel/sandybridge/raminit_native.c:
https://review.coreboot.org/c/coreboot/+/39851/7/src/northbridge/intel/sandy... PS7, Line 194: max_mem_clock_mhz Maybe outside the scope of this patch but why are values higher than 1066MHz not possible via devicetree override but are with Kconfig?