[coreboot-gerrit] Change in coreboot[master]: mb/amd,google/mandolin,zork: Set EFS SPI platform config

[Martin Roth (Code Review)]

Martin Roth has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43303 )

Change subject: mb/amd,google/mandolin,zork: Set EFS SPI platform config
......................................................................

Patch Set 16:

(2 comments)

https://review.coreboot.org/c/coreboot/+/43303/16/src/mainboard/amd/mandolin... 
File src/mainboard/amd/mandolin/Kconfig:

https://review.coreboot.org/c/coreboot/+/43303/16/src/mainboard/amd/mandolin... 
PS16, Line 110: 	default 6
              : 	default 0 if EM100
These are reversed for all 4 with an EM100.  Kconfig only pays attention to the FIRST default it reads.

I think we should just make the em100 Kconfig get loaded first though.  I'll discuss that with you on chat.

https://review.coreboot.org/c/coreboot/+/43303/16/src/mainboard/google/zork/... 
File src/mainboard/google/zork/Kconfig:

https://review.coreboot.org/c/coreboot/+/43303/16/src/mainboard/google/zork/... 
PS16, Line 232: default 0 if EM100
also needs to go first.