Patch Set 31: Code-Review-1

(1 comment)

There seems to be something wrong with Kconfig... here's an error I see when I try to `make menuconfig`:
$ make menuconfig
warning: (BOARD_SPECIFIC_OPTIONS && BOARD_SPECIFIC_OPTIONS && CPU_SPECIFIC_OPTIONS) selects FSP_USE_REPO which has unmet direct dependencies (PLATFORM_USES_FSP1_1 && SOC_INTEL_BRASWELL && !USE_GOOGLE_FSP || PLATFORM_USES_FSP2_0 && ADD_FSP_BINARIES && (SOC_INTEL_APOLLOLAKE || SOC_INTEL_SKYLAKE || SOC_INTEL_KABYLAKE || SOC_INTEL_COFFEELAKE || SOC_INTEL_ICELAKE || SOC_INTEL_WHISKEYLAKE || SOC_INTEL_XEON_SP))

*** ERROR: 1 warnings encountered, and warnings are errors.

Your configuration changes were NOT saved.

Turns out we just need to select ADD_FSP_BINARIES in the mainboard's Kconfig.

View Change

1 comment:

To view, visit change 38549. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I29868f03037d1887b90dfb19d15aee83c456edce
Gerrit-Change-Number: 38549
Gerrit-PatchSet: 31
Gerrit-Owner: Jonathan Zhang <jonzhang@fb.com>
Gerrit-Reviewer: Anjaneya "Reddy" Chagam <anjaneya.chagam@intel.com>
Gerrit-Reviewer: David Hendricks <david.hendricks@gmail.com>
Gerrit-Reviewer: HAOUAS Elyes <ehaouas@noos.fr>
Gerrit-Reviewer: Johnny Lin
Gerrit-Reviewer: Jonathan Zhang <jonzhang@fb.com>
Gerrit-Reviewer: Martin Roth <martinroth@google.com>
Gerrit-Reviewer: Patrick Georgi <pgeorgi@google.com>
Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org>
Gerrit-CC: Andrey Petrov <anpetrov@fb.com>
Gerrit-CC: Angel Pons <th3fanbus@gmail.com>
Gerrit-CC: Maxim Polyakov <max.senia.poliak@gmail.com>
Gerrit-CC: Paul Menzel <paulepanter@users.sourceforge.net>
Gerrit-Comment-Date: Thu, 20 Feb 2020 04:55:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment