Attention is currently required from: Dinesh Gehlot, Eric Lai, Kapil Porwal, Nick Vaccaro, Varshit Pandya.
Subrata Banik has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/81627?usp=email )
Change subject: mb/google/brya: Create trulo variant ......................................................................
Patch Set 10:
(4 comments)
File src/mainboard/google/brya/Kconfig:
https://review.coreboot.org/c/coreboot/+/81627/comment/99893f84_280a5f10 : PS10, Line 498: select BOARD_GOOGLE_BASEBOARD_TRULO select SOC_INTEL_TWINLAKE
File src/mainboard/google/brya/variants/trulo/memory/Makefile.mk:
https://review.coreboot.org/c/coreboot/+/81627/comment/527bbbe3_88a42462 : PS10, Line 7: SPD_SOURCES += spd/lp5/set-0/spd-2.hex # ID = 0(0b0000) Parts = MT62F1G32D4DR-031 WT:B : SPD_SOURCES += spd/lp5/set-0/spd-1.hex # ID = 1(0b0001) Parts = MT62F512M32D2DR-031 WT:B empty
File src/mainboard/google/brya/variants/trulo/memory/dram_id.generated.txt:
https://review.coreboot.org/c/coreboot/+/81627/comment/fd13648e_ad6b0373 : PS10, Line 7: MT62F1G32D4DR-031 WT:B 0 (0000) : MT62F512M32D2DR-031 WT:B 1 (0001) should be empty
File src/mainboard/google/brya/variants/trulo/memory/mem_parts_used.txt:
https://review.coreboot.org/c/coreboot/+/81627/comment/ed2bfd8c_c743b315 : PS10, Line 12: MT62F1G32D4DR-031 WT:B : MT62F512M32D2DR-031 WT:B empty