[coreboot-gerrit] Change in coreboot[master]: nb/intel/sandybridge: Use arrays to program IOSAV

[Angel Pons (Code Review)]

Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/47492 )

Change subject: nb/intel/sandybridge: Use arrays to program IOSAV
......................................................................

Patch Set 1:

(2 comments)

https://review.coreboot.org/c/coreboot/+/47492/1/src/northbridge/intel/sandy... 
File src/northbridge/intel/sandybridge/raminit_iosav.c:

https://review.coreboot.org/c/coreboot/+/47492/1/src/northbridge/intel/sandy... 
PS1, Line 32: ssq_count
> > I specifically want to remove this assignment. […]

I still added a check for ssq_count == 0, since that would overflow within the register write.

https://review.coreboot.org/c/coreboot/+/47492/1/src/northbridge/intel/sandy... 
PS1, Line 21: ssq_count
> d'oh, good catch

Done