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

[Paul Menzel (Code Review)]

Paul Menzel 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 32:

(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");
> yes, if 'zero' = not populated , that it means :  Not(zero & zero) = True  […]

True. My bad.

(sysinfo->dimm[0] != SYSINFO_DIMM_NOT_POPULATED) != (sysinfo->dimm[1] !=  SYSINFO_DIMM_NOT_POPULATED)

should work then [1].

[1]: https://stackoverflow.com/questions/1596668/logical-xor-operator-in-c