Elyes HAOUAS has posted comments on this change. ( https://review.coreboot.org/27590 )
Change subject: [test]i945: Fix sdram_rcomp_buffer_strength_and_slew for i945gc ......................................................................
Patch Set 2:
(1 comment)
https://review.coreboot.org/#/c/27590/2/src/northbridge/intel/i945/raminit.c File src/northbridge/intel/i945/raminit.c:
https://review.coreboot.org/#/c/27590/2/src/northbridge/intel/i945/raminit.c... PS2, Line 965:
you change IDX for this option...
in fact for dual channel, we need 2 idx : the 1st one is for channel0 : idx= 5 * dimm0 + dimm1 and the 2nd for channel1, called idx_channel1 = 5 * dimm2 + dimm3