Patrick Rudolph 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:
(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
Ivybridge supports this frequency, but there's no board in tree that does. At the end the value here doesn't matter, but I don't want to return 0...
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 faste
Yes, at the moment only 666mhz and 933mhz is being used. As this commit removes 933 I don't care that much.
-
Patrick Rudolph (Code Review)