[coreboot-gerrit] Change in coreboot[master]: soc/intel/apl: Enable FSP usage by default

[Felix Singer (Code Review)]

Felix Singer has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/34642 )

Change subject: soc/intel/apl: Enable FSP usage by default
......................................................................

Patch Set 3:

(2 comments)

https://review.coreboot.org/c/coreboot/+/34642/1/src/mainboard/up/squared/Kc... 
File src/mainboard/up/squared/Kconfig:

https://review.coreboot.org/c/coreboot/+/34642/1/src/mainboard/up/squared/Kc... 
PS1, Line 15: 	default y
> If we enable ADD_FSP_BINARIES AND FSP_USE_REPO by default, we need this option to get the build prop […]

Done

https://review.coreboot.org/c/coreboot/+/34642/1/src/mainboard/up/squared/Kc... 
PS1, Line 17: config ADD_FSP_BINARIES
            : 	bool
            : 	default y
            : 
            : config FSP_USE_REPO
            : 	bool
            : 	default y
> You are right, but see my comment before.

Done