[coreboot-gerrit] Change in coreboot[master]: util/spd_tools/intel/lp4x: Add a global list of LP4x memory parts

[Nick Vaccaro (Code Review)]

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:

(3 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
> No, this is the physical dies per package. So, it has to be 1. […]

Done

https://review.coreboot.org/c/coreboot/+/41874/3/util/spd_tools/intel/lp4x/g... 
PS3, Line 45: 2
> Same as above: […]

Thanks for the explanation.  Is diesPerPackage only used to determine total size?

https://review.coreboot.org/c/coreboot/+/41874/3/util/spd_tools/intel/lp4x/g... 
PS3, Line 112:          }
> I already did that last night here: https://review.coreboot. […]

You rock !