Attention is currently required from: Scott Chao, Eric Lai. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/63682 )
Change subject: mb/google/brya/var/crota: limit dram speed at 4800 ......................................................................
Patch Set 5: Code-Review+1
(5 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/63682/comment/3895f0bd_ca9e2281 PS5, Line 7: at to
https://review.coreboot.org/c/coreboot/+/63682/comment/86b4dd31_fdad6272 PS5, Line 7: limit Limit
https://review.coreboot.org/c/coreboot/+/63682/comment/a44f6674_ca1599dc PS5, Line 7: 4800 4800 MT/s
https://review.coreboot.org/c/coreboot/+/63682/comment/69cbefe6_e83f19f4 PS5, Line 9: Intel PDG Please add the revision and section.
File src/mainboard/google/brya/variants/crota/overridetree.cb:
https://review.coreboot.org/c/coreboot/+/63682/comment/bcb5126e_003e805b PS5, Line 22: max_dram_speed The unit should be appended to the variable name. Could you please create a separate change-set/commit doing that?
-
Paul Menzel (Code Review)