Attention is currently required from: Henry Sun, Paul Menzel, Karthik Ramasubramanian. Stanley Wu has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57310 )
Change subject: mb/google/dedede/var/gooey: Add MT53E512M32D1NP-046 as supported mem module ......................................................................
Patch Set 2:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/57310/comment/af5cf66d_49c546d2 PS1, Line 2: stanley.wu
Please use *Stanley Wu*. […]
Done
https://review.coreboot.org/c/coreboot/+/57310/comment/771540a2_9e1ce543 PS1, Line 7: Generate SPD ID for supported part
Maybe: […]
Done
https://review.coreboot.org/c/coreboot/+/57310/comment/b7c1e00c_da8f4626 PS1, Line 11: MT53E512M32D1NP-046 WT:B
- What is the difference to MT53E512M32D2NP-046? […]
From Micron AE feedback as below: 1. MT53E512M32D1NP-046 WT:B is 1anm Tech, MT53E512M32D2NP-046 is 1xnm Tech 2. Manufacturer is Micron
https://review.coreboot.org/c/coreboot/+/57310/comment/cccb5e5a_db00d375 PS1, Line 17: Signed-off-by: stanley.wu stanley1.wu@lcfc.corp-partner.google.com
Ditto.
Done