Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/20907 )
Change subject: nb/intel/sandybridge/raminit: Add Kconfig option for fuses ......................................................................
Patch Set 1: Code-Review+2
(2 comments)
https://review.coreboot.org/#/c/20907/1/src/northbridge/intel/sandybridge/ra... File src/northbridge/intel/sandybridge/raminit_common.c:
https://review.coreboot.org/#/c/20907/1/src/northbridge/intel/sandybridge/ra... PS1, Line 406: TCK_1333MHZ Is faster than both ivy and sandy can run but that's not an issue I suppose.
https://review.coreboot.org/#/c/20907/1/src/northbridge/intel/sandybridge/ra... PS1, Line 443: if (cfg->max_mem_clock_mhz >= 1066) : return TCK_1066MHZ; unrelated but should be adapted for ivy bridge since that one can run faster.