Attention is currently required from: Furquan Shaikh. Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/51057 )
Change subject: util/spd_tools/lp4x: Add 2 new parts to global memory definition ......................................................................
Patch Set 2:
(4 comments)
File util/spd_tools/lp4x/global_lp4x_mem_parts.json.txt:
https://review.coreboot.org/c/coreboot/+/51057/comment/5581af42_7bbc135b PS1, Line 239: 2
I think this is 1. […]
Done
https://review.coreboot.org/c/coreboot/+/51057/comment/0bee8bd1_db3e8e3a PS1, Line 242: "speedMbps": 4267
"casLatencies": "14 20 24 28 32 36" […]
Done
https://review.coreboot.org/c/coreboot/+/51057/comment/c6ca6b59_af204197 PS1, Line 248: 16
Isn't this 8?
Done
https://review.coreboot.org/c/coreboot/+/51057/comment/c372533f_2106e82f PS1, Line 250: 4
2?
Done