Nick Vaccaro has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/41874 )
Change subject: util/spd_tools/intel/lp4x: Add a global list of LP4x memory parts ......................................................................
Patch Set 4:
(2 comments)
https://review.coreboot.org/c/coreboot/+/41874/3/util/spd_tools/intel/lp4x/g... File util/spd_tools/intel/lp4x/global_lp4x_mem_parts.json.txt:
https://review.coreboot.org/c/coreboot/+/41874/3/util/spd_tools/intel/lp4x/g... PS3, Line 33: 1 Should this be 2?
https://review.coreboot.org/c/coreboot/+/41874/3/util/spd_tools/intel/lp4x/g... PS3, Line 45: 2 Should this be 4?