[coreboot-gerrit] Change in ...coreboot[master]: nb/intel/nehalem: Tidy quickpath_reserved calculation

[Jacob Garber (Code Review)]

Jacob Garber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33381 )

Change subject: nb/intel/nehalem: Tidy quickpath_reserved calculation
......................................................................

Patch Set 1:

(1 comment)

https://review.coreboot.org/#/c/33381/1/src/northbridge/intel/nehalem/ramini... 
File src/northbridge/intel/nehalem/raminit.c:

https://review.coreboot.org/#/c/33381/1/src/northbridge/intel/nehalem/ramini... 
PS1, Line 1442: (u32)1
> I've usually seen '1UL' or '1ULL'

I've also seen those, but their widths depend on the target architecture. Currently 1UL would also work since sizeof(unsigned long) == 32, but there is no guarantee this will stay that way in the future, so I prefer explicit widths.