Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/19016 )
Change subject: [Not For Merge] nb/intel/i945: Remove unused #define REFRESH_* ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/#/c/19016/1/src/northbridge/intel/i945/raminit.c File src/northbridge/intel/i945/raminit.c:
PS1, Line 848: if (refresh == 2) { : sysinfo->refresh = 1;
I wonder, if this ever hit the 15.6us case. Remind me, how many rows do we
15.6 is pretty standard on ddr2. (and is set as default before loop here) I think there are typically 13-14 rows