Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/46249 )
Change subject: mb/clevo/l140cu: rework memcfg ......................................................................
Patch Set 2: Code-Review+1
(2 comments)
https://review.coreboot.org/c/coreboot/+/46249/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/46249/2//COMMIT_MSG@12 PS2, Line 12: ect ECT (Early Command Training) is an acronym, please spell it in uppercase.
If you're curious, MRC will never run ECT on DDR4. This is because PDA (Per DRAM Addressability) is not stable before basic training (e.g. Receive Enable, Read MPR, JEDEC Write Leveling). At that point, one might as well do LCT (Late Command Training) instead.
https://review.coreboot.org/c/coreboot/+/46249/2//COMMIT_MSG@14 PS2, Line 14: correct or `trim down`
-
Angel Pons (Code Review)