Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40493 )
Change subject: mb/asus/am1i-a/buildOpts.c: improve the RAM-related defines ......................................................................
Patch Set 2:
(3 comments)
https://review.coreboot.org/c/coreboot/+/40493/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/40493/2//COMMIT_MSG@7 PS2, Line 7: mb/asus/am1i-a/buildOpts.c: improve the RAM-related defines Please be more specific. Maybe:
mb/asus/am1i-a: Configure AGESA to support 1600 MHz RAM speeds
https://review.coreboot.org/c/coreboot/+/40493/2//COMMIT_MSG@9 PS2, Line 9: This is needed for getting a pair of Crucial BLT8G3D1869DT1TX0 sticks : to work at 1600 MHz CL8 speeds, What are they currently detected at? What does the vendor firmware support?
https://review.coreboot.org/c/coreboot/+/40493/2//COMMIT_MSG@12 PS2, Line 12: Please elaborate why the interleaving options need to be changed.