Hung-Te Lin has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/44520 )
Change subject: mb/google/kukui: Add LPDDR4X MT53E2G32D4NQ-046 8GB support for burnet/esche ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/44520/1/src/mainboard/google/kukui/... File src/mainboard/google/kukui/sdram_configs.c:
https://review.coreboot.org/c/coreboot/+/44520/1/src/mainboard/google/kukui/... PS1, Line 31: 0x16
Hi Hung-Te, […]
Hi Kevin
If no boards have been manufactured with that ID and we're not going to use that module in future (given SOC can't support it), I see no reason we want to keep #5 unused. Please check with Google HW team to see if they agree to re-allocate. Please remember we have only 12 IDs, unless if you are going to have #5 defined for something else very soon.