[coreboot-gerrit] Change in coreboot[master]: intel/i440bx: Resolve long standing raminit TODOs

[Angel Pons (Code Review)]

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

Change subject: intel/i440bx: Resolve long standing raminit TODOs
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/c/coreboot/+/38383/2/src/northbridge/intel/i440b... 
File src/northbridge/intel/i440bx/raminit.c:

https://review.coreboot.org/c/coreboot/+/38383/2/src/northbridge/intel/i440b... 
PS2, Line 361: 	// TODO: Only do this later?
> Agreed this should be dropped as it just mirrors power on defaults. […]

I'd do it on this change. It's at the top of the patch train, so it's easy to remove the comments on lines 361 and 362. I would change the comment on line 360:

Enable normal refresh, but not yet. Done after raminit.