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

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes 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:

(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. […]

I've checked, apple macbook21 did the same on vendore rom.

I don't have that hardware to test, and I'm fine to drop current changes