[coreboot-gerrit] Change in coreboot[master]: nb/intel/i945: Refactor sdram_set_channel_mode()

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/39537 )

Change subject: nb/intel/i945: Refactor sdram_set_channel_mode()
......................................................................

Patch Set 3:

(1 comment)

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

https://review.coreboot.org/c/coreboot/+/39537/3/src/northbridge/intel/i945/... 
PS3, Line 1584: 	} else
> What if sysinfo->dimm[1] is populated?

Here we are in Single-Channel. so if DIMM0 is not populated, it means that DIMM1 is populated 😊
so here you are 😊