[coreboot-gerrit] Change in ...coreboot[master]: mb/google/hatch/variants/helios: Use LPDDR3 memory

[Paul Fagerburg (Code Review)]

Paul Fagerburg has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/33456 )

Change subject: mb/google/hatch/variants/helios: Use LPDDR3 memory
......................................................................

Patch Set 2:

(1 comment)

https://review.coreboot.org/#/c/33456/1/src/mainboard/google/hatch/variants/... 
File src/mainboard/google/hatch/variants/helios/Makefile.inc:

https://review.coreboot.org/#/c/33456/1/src/mainboard/google/hatch/variants/... 
PS1, Line 17: LP_16G_2133
> Shouldn't this be at index 1?

According to the chart in comment #2 on the bug, 0000 is for the 512Mbit parts, and 0010 is for the 1024Mbit parts. Do we want to have them change the strapping, and then the other vendor's parts (if not covered by these SPDs) can have slots 0010 and 0011?