Angel Pons 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
Here we are in Single-Channel. so if DIMM0 is not populated, it means that DIMM1 is populated 😊 […]
I think I did not express myself well.
What I mean is that, if we are not in dual-channel mode, and DIMM0 is empty, it is possible for DIMM1 to be the only populated slot. And DIMM1 is on Channel 0, sooo... 😊