[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge: Refactor ODT stretch table code

[Felix Held (Code Review)]

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

Change subject: nb/intel/sandybridge: Refactor ODT stretch table code
......................................................................

Patch Set 2: Code-Review+2

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/48613/2/src/northbridge/intel/sandy... 
PS2, Line 240: NUM_SLOTS
out of scope for this patch, but the NUM_SLOTS define in the header file could really use a comment that that is per channel and not per socket/system; found that a bit confusing and had to look that up