[coreboot-gerrit] Change in coreboot[master]: nb/i945: Reorder sdram_setup_processor_side()

[Angel Pons (Code Review)]

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

Change subject: nb/i945: Reorder sdram_setup_processor_side()
......................................................................

Patch Set 3: Code-Review-1

(1 comment)

https://review.coreboot.org/c/coreboot/+/40368/3/src/northbridge/intel/i945/... 
File src/northbridge/intel/i945/raminit.c:

https://review.coreboot.org/c/coreboot/+/40368/3/src/northbridge/intel/i945/... 
PS3, Line 2717: 	if (i945_silicon_revision() == 0) {
Changing the order of undocumented register writes is not a good idea. What if there's an electrical dependency between these?