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;
regarding the datasheet JEDEC Standard No. 21-C, the value '2' is for Reduc
I think this is correct (northbridge only documents how to set 7.8µs) . FYI: I also refactored this code a bit here https://review.coreboot.org/#/c/18305/28/src/northbridge/intel/i945/raminit....