[coreboot-gerrit] Change in coreboot[master]: [WIP]nb/intel/sandybridge/raminit: Add ECC support

[Felix Held (Code Review)]

Felix Held has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/22215 )

Change subject: [WIP]nb/intel/sandybridge/raminit: Add ECC support
......................................................................

Patch Set 8:

(2 comments)

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

https://review.coreboot.org/c/coreboot/+/22215/4/src/northbridge/intel/sandy... 
PS4, Line 290:
> Very minor nit: Remove the space before the punctuation.

Done

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

https://review.coreboot.org/c/coreboot/+/22215/4/src/northbridge/intel/sandy... 
PS4, Line 354: 	udelay(10);
> Why?

not sure why, but i added a TODO comment