Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45278 )
Change subject: WIP: Use SPDs from PSP BIOS table type 0x69 based on GPIO straps ......................................................................
Patch Set 1:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45278/1/src/soc/amd/picasso/Makefil... File src/soc/amd/picasso/Makefile.inc:
https://review.coreboot.org/c/coreboot/+/45278/1/src/soc/amd/picasso/Makefil... PS1, Line 265: # SPD data array blob (with SPDs for all possible memory IDs for this platform) : SPD_BLOB:=$(obj)/SPD_array.bin : : # APCB binary without SPD data : APCB_NO_SPD_BLOB:=$(FIRMWARE_LOCATE)/APCB_magic.bin We just removed all the SPD information from the picasso Makefile and put it into the Mainboard files because it's really mainboard specific. A lot of these changes undo what was done there.
https://review.coreboot.org/c/coreboot/+/45222