[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge: Use bitfields to program MCMAIN timings

[Felix Held (Code Review)]

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

Change subject: nb/intel/sandybridge: Use bitfields to program MCMAIN timings
......................................................................

Patch Set 7:

(1 comment)

https://review.coreboot.org/c/coreboot/+/47512/7/src/northbridge/intel/sandy... 
File src/northbridge/intel/sandybridge/raminit_common.h:

https://review.coreboot.org/c/coreboot/+/47512/7/src/northbridge/intel/sandy... 
PS7, Line 165: [12..12]
i'd use [12] here instead; some more instances above and below. should be fixed in a follow-up patch to avoid rebases