Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44462 )
Change subject: mb/asus/am1i-a/buildOpts.c: choose the 1600 MT/s RAM frequency ......................................................................
Patch Set 3:
(2 comments)
https://review.coreboot.org/c/coreboot/+/44462/1//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/44462/1//COMMIT_MSG@7 PS1, Line 7: 1600 MHz
DDR3-1600 != 1600 MHz […]
Done
https://review.coreboot.org/c/coreboot/+/44462/1//COMMIT_MSG@9 PS1, Line 9: Together with the "AMD_XMP" changes
Are they needed? Or does this patch work properly on its own? I'd say this patch should make DDR3-16 […]
Yes, AMD_XMP are needed at least for Crucial BLT8G3D1869DT1TX0 sticks mentioned below, because these sticks - like many others - don't have anything higher than DDR3-1333 at its' JEDEC half (lower 128 bytes) of SPD memory. So, in practice, this patch alone often isn't enough to get a desired DDR3-1600.