Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/22806 )
Change subject: intel/sandybridge: Add RAM voltage hooks ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/22806/1/src/northbridge/intel/sandybridge/ra... File src/northbridge/intel/sandybridge/raminit_common.c:
https://review.coreboot.org/#/c/22806/1/src/northbridge/intel/sandybridge/ra... PS1, Line 3371: void __attribute__((weak)) mainboard_set_dram_voltage(dimm_flags_t flags) mainboard_set_dram_voltage(const size_t voltage)
where voltage is in units of mV to be compatible with XMP memory profiles that allows overvolting. The raminit code has to select the voltage and mainboard will set the most suitable. A future improvement would be a devicetree setting that lists supported dram voltages.