Arthur Heymans has posted comments on this change. ( https://review.coreboot.org/23088 )
Change subject: nb/intel/sandybridge: Reduce storage size ......................................................................
Patch Set 1: Code-Review+1
(1 comment)
https://review.coreboot.org/#/c/23088/1/src/northbridge/intel/sandybridge/ra... File src/northbridge/intel/sandybridge/raminit_common.c:
https://review.coreboot.org/#/c/23088/1/src/northbridge/intel/sandybridge/ra... PS1, Line 2106: MIN(MAX(timC, 0), MAX_TIMC); I haven't check what exactly this function is supposed to do but wouldn't you want a warning on overflows?