[coreboot-gerrit] Change in coreboot[master]: soc/amd/picasso: Update northbridge

[Marshall Dawson (Code Review)]

Marshall Dawson has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34424 )

Change subject: soc/amd/picasso: Update northbridge
......................................................................

Patch Set 14:

(1 comment)

https://review.coreboot.org/c/coreboot/+/34424/12/src/soc/amd/picasso/includ... 
File src/soc/amd/picasso/include/soc/pci_devs.h:

https://review.coreboot.org/c/coreboot/+/34424/12/src/soc/amd/picasso/includ... 
PS12, Line 108: #define DF_F0_DEVID		0x15e8
> Could you add comments defining what each data fabric is about? Like DRAM/MISC/PM and so on?

No, I prefer not to.  The features don't line up nicely like in previous generations, and it's doubtful they'll get documented in the public PPR.