Mike Banon has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/40533 )
Change subject: mb/asus/am1i-a/buildOpts.c: choose the 1600 MHz RAM frequency ......................................................................
Patch Set 2:
(1 comment)
Patch Set 2:
is this tested on hardware?
Yes, and I test all these RAM-related changes on my hardware before submitting them.
https://review.coreboot.org/c/coreboot/+/40533/2//COMMIT_MSG Commit Message:
https://review.coreboot.org/c/coreboot/+/40533/2//COMMIT_MSG@11 PS2, Line 11: Earlier only 1333 MHz CL9 (9-9-10-27) has been possible with coreboot.
Please update the commit message that it only works with one DIMM in the second slot.
Perhaps I'd try fixing it for two DIMMs first: really hope that a tiny change at ./src/vendorcode/amd/agesa/f16kb/Proc/Mem/Ps/KB/mpUkb3.c will be enough. There I'm going to some limit values from 1333 to 1600 while looking at its' f15tn's relative ./agesa/f15tn/Proc/Mem/Ps/TN/mpUtn3.c (two DIMMs are working fine on 1600MHz and even 1866MHz at f15tn platform).