Attention is currently required from: Henry Sun, Stanley Wu, Paul Fagerburg, Karthik Ramasubramanian. Paul Menzel has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/57310 )
Change subject: mb/google/dedede/var/gooey: Generate SPD ID for supported part ......................................................................
Patch Set 1:
(4 comments)
Commit Message:
https://review.coreboot.org/c/coreboot/+/57310/comment/ed42b2fa_acc60ebb PS1, Line 2: stanley.wu Please use *Stanley Wu*.
$ git config --global user.name "Stanley Wu" $ git commit -s --amend "Stanley Wu stanley1.wu@lcfc.corp-partner.google.com"
https://review.coreboot.org/c/coreboot/+/57310/comment/97443419_0d2b42f7 PS1, Line 7: Generate SPD ID for supported part Maybe:
Add MT53E512M32D1NP-046 as supported mem module
https://review.coreboot.org/c/coreboot/+/57310/comment/219aeb72_05285d72 PS1, Line 11: MT53E512M32D1NP-046 WT:B 1. What is the difference to MT53E512M32D2NP-046? 2. Any idea, who the manufacturer is?
https://review.coreboot.org/c/coreboot/+/57310/comment/af8b9e13_8f32e382 PS1, Line 17: Signed-off-by: stanley.wu stanley1.wu@lcfc.corp-partner.google.com Ditto.