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

[Matt Papageorge (Code Review)]

Matt Papageorge 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 17:

(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
> +1 to default for EM100 to be placed first. […]

Done. For some reason I was thinking it was the other way around. I have learned a lot about kconfig while adding this feature, hopefully my future changes will benefit :)

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.

Done