[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge/raminit: Fix ECC scrub

[Patrick Rudolph (Code Review)]

Patrick Rudolph has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40721 )

Change subject: nb/intel/sandybridge/raminit: Fix ECC scrub
......................................................................

Patch Set 5:

(2 comments)

https://review.coreboot.org/c/coreboot/+/40721/4/src/northbridge/intel/sandy... 
File src/northbridge/intel/sandybridge/raminit_common.c:

https://review.coreboot.org/c/coreboot/+/40721/4/src/northbridge/intel/sandy... 
PS4, Line 2964: 		 *  Writes 128 * 8 bytes in a loop
> So this is almost true for a single iteration of the innermost loop. […]

Replaced the comment with a description why it's done like that.

https://review.coreboot.org/c/coreboot/+/40721/4/src/northbridge/intel/sandy... 
PS4, Line 2965: 		 *  Bank bits are located at bit 13:15 in physical memory map
              : 		 *  Rowbits start at bit 20 in physical memory map
> They help understand what's wrong by observing the patterns in the ecc scrub test. […]

Moved to CB:40946