[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 5:

(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
> I think I did not express myself well. […]

Please see datasheet page 151
[Bit 2]: Single-Channel Selector (SCS):
When in Single-channel mode, this is the populated channel.
0: "Channel 0"
1: "Channel 1"