Change in coreboot[master]: nb/intel/i945: Program CxODT value for each channel

[HAOUAS Elyes (Code Review)]

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

Change subject: nb/intel/i945: Program CxODT value for each channel
......................................................................

Patch Set 31:

(1 comment)

https://review.coreboot.org/c/coreboot/+/18548/31/src/northbridge/intel/i945... 
File src/northbridge/intel/i945/raminit.c:

https://review.coreboot.org/c/coreboot/+/18548/31/src/northbridge/intel/i945... 
PS31, Line 2442: 		printk(BIOS_DEBUG, "one dimm per channel config..\n");
> But it could also be, that there is no DIMM in that channel, right? What would be wrong with […]

yes, if 'zero' = not populated , that it means :  Not(zero & zero) = True

sysinfo->dimm[0] != SYSINFO_DIMM_NOT_POPULATED || sysinfo->dimm[1] !=  SYSINFO_DIMM_NOT_POPULATED 
is also true if both DIMMs are populated