[coreboot-gerrit] Change in coreboot[master]: soc/intel/denverton_ns: Allow using FSP repo

[Felix Singer (Code Review)]

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

Change subject: soc/intel/denverton_ns: Allow using FSP repo
......................................................................

Patch Set 12:

(2 comments)

https://review.coreboot.org/c/coreboot/+/30931/3/src/soc/intel/denverton_ns/... 
File src/soc/intel/denverton_ns/Kconfig:

https://review.coreboot.org/c/coreboot/+/30931/3/src/soc/intel/denverton_ns/... 
PS3, Line 82: 	string "Location of FSP headers"
> I don't understand it. Why does it need a prompt now? and didn't need […]

Done

https://review.coreboot.org/c/coreboot/+/30931/3/src/soc/intel/denverton_ns/... 
PS3, Line 88: 	default "3rdparty/fsp/DenvertonNSFspBinPkg/FspBin/DenvertonNSFsp.fd"
> I really don't care about the existing weirdness. If you don't want to […]

I can do that in a follow-up patch, but I don't think this is the right patch to change this, since it would break the consistency compared to the other FSP platforms.