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 35:

(1 comment)

current change isn't a big deal.
it will not break anything and it just fix the values of CxODT per channel.

we lost a important time for this patch and maybe we don't need to loos again time for this ...

so if you agree, I'll move this to abandoned and save your time.

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

https://review.coreboot.org/c/coreboot/+/18548/34/src/northbridge/intel/i945... 
PS34, Line 2450: one
> We are talking about `(populated_dimms & 3) == 1 != (populated_dimms & 3) == 2)`, right? […]

I'm confused with this : `(populated_dimms & 3) == 1 != (populated_dimms & 3) == 2)`