Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/28010
Change subject: soc/intel/braswell/Kconfig: Clean up redefined config options
......................................................................
soc/intel/braswell/Kconfig: Clean up redefined config options
HAVE_IFD_BIN and the likes are redefined in southbridge/intel/common/Kconfig
which presents options under the 'Intel Firmware' section which gets sourced
later than the 'southbridge' section therefore overriding this anyway.
FAKE_IFD depends on out tree flashrom patches for which there are better
alternatives available now, so don't build with FAKE_IFD by default.
Change-Id: Icd41137a1bbfe519c89a71cc0c7c3755558bd834
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/soc/intel/braswell/Kconfig
1 file changed, 0 insertions(+), 9 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/10/28010/1
diff --git a/src/soc/intel/braswell/Kconfig b/src/soc/intel/braswell/Kconfig
index 23e5990..607d78c 100644
--- a/src/soc/intel/braswell/Kconfig
+++ b/src/soc/intel/braswell/Kconfig
@@ -120,15 +120,6 @@
configure the pads and enable it. This serial port can be used for
the debug console.
-config HAVE_IFD_BIN
- def_bool n
-
-config BUILD_WITH_FAKE_IFD
- def_bool !HAVE_IFD_BIN
-
-config HAVE_ME_BIN
- def_bool n
-
config IED_REGION_SIZE
hex
default 0x400000
--
To view, visit https://review.coreboot.org/28010
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Icd41137a1bbfe519c89a71cc0c7c3755558bd834
Gerrit-Change-Number: 28010
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/28009
Change subject: sb/intel/lynxpoint: Don't build with FAKE_IFD by default
......................................................................
sb/intel/lynxpoint: Don't build with FAKE_IFD by default
FAKE_IFD depends on out tree flashrom patches for which there are better
alternatives available now, so don't build with FAKE_IFD by default.
Change-Id: I21bc5bdc8b733fbfdb1b2a4fbcb572c76701074a
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/southbridge/intel/lynxpoint/Kconfig
1 file changed, 0 insertions(+), 4 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/09/28009/1
diff --git a/src/southbridge/intel/lynxpoint/Kconfig b/src/southbridge/intel/lynxpoint/Kconfig
index 4e5527a..ee870fc 100644
--- a/src/southbridge/intel/lynxpoint/Kconfig
+++ b/src/southbridge/intel/lynxpoint/Kconfig
@@ -57,10 +57,6 @@
If you set this option to y, the serial IRQ machine will be
operated in continuous mode.
-config BUILD_WITH_FAKE_IFD
- bool
- default y if !HAVE_IFD_BIN
-
config ME_MBP_CLEAR_LATE
bool "Defer wait for ME MBP Cleared"
default y
--
To view, visit https://review.coreboot.org/28009
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I21bc5bdc8b733fbfdb1b2a4fbcb572c76701074a
Gerrit-Change-Number: 28009
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/28008
Change subject: mb/google/rambi: Don't set defaults for HAVE_IFD_BIN
......................................................................
mb/google/rambi: Don't set defaults for HAVE_IFD_BIN
There is no need set the default HAVE_IFD_BIN explicitly to n.
Change-Id: I4a5fe45e7f8f6dd018937861b0fb92a8da49904e
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/mainboard/google/rambi/Kconfig
1 file changed, 0 insertions(+), 8 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/08/28008/1
diff --git a/src/mainboard/google/rambi/Kconfig b/src/mainboard/google/rambi/Kconfig
index 440276d..725d69d 100644
--- a/src/mainboard/google/rambi/Kconfig
+++ b/src/mainboard/google/rambi/Kconfig
@@ -109,12 +109,4 @@
string
default "pci8086,0f31.rom"
-config HAVE_IFD_BIN
- bool
- default n
-
-config HAVE_ME_BIN
- bool
- default n
-
endif # BOARD_GOOGLE_BASEBOARD_RAMBI
--
To view, visit https://review.coreboot.org/28008
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4a5fe45e7f8f6dd018937861b0fb92a8da49904e
Gerrit-Change-Number: 28008
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>
Arthur Heymans has uploaded this change for review. ( https://review.coreboot.org/28007
Change subject: sb/intel/lynxpoint/Kconfig: Clean up redefined config options
......................................................................
sb/intel/lynxpoint/Kconfig: Clean up redefined config options
HAVE_IFD_BIN and the likes are redefined in southbridge/intel/common/Kconfig
which presents options under the 'Intel Firmware' section which gets sourced
later than the 'southbridge' section therefore overriding this anyway.
Also all board using this target override HAVE_IFD_BIN to default n, so it is
simpler to just use the defaults in southbridge/intel/common/firmware.
Change-Id: I9999440031b07006e2df11e00dfb9f3dbe04f832
Signed-off-by: Arthur Heymans <arthur(a)aheymans.xyz>
---
M src/mainboard/google/beltino/Kconfig
M src/mainboard/google/slippy/Kconfig
M src/mainboard/intel/baskingridge/Kconfig
M src/southbridge/intel/lynxpoint/Kconfig
4 files changed, 0 insertions(+), 42 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/07/28007/1
diff --git a/src/mainboard/google/beltino/Kconfig b/src/mainboard/google/beltino/Kconfig
index 428db73..aaec9f7 100644
--- a/src/mainboard/google/beltino/Kconfig
+++ b/src/mainboard/google/beltino/Kconfig
@@ -65,12 +65,4 @@
string
default "pci8086,0406.rom"
-config HAVE_IFD_BIN
- bool
- default n
-
-config HAVE_ME_BIN
- bool
- default n
-
endif # BOARD_GOOGLE_BASEBOARD_BELTINO
diff --git a/src/mainboard/google/slippy/Kconfig b/src/mainboard/google/slippy/Kconfig
index c43b713..6891c0b 100644
--- a/src/mainboard/google/slippy/Kconfig
+++ b/src/mainboard/google/slippy/Kconfig
@@ -73,12 +73,4 @@
string
default "pci8086,0406.rom"
-config HAVE_IFD_BIN
- bool
- default n
-
-config HAVE_ME_BIN
- bool
- default n
-
endif # BOARD_GOOGLE_BASEBOARD_SLIPPY
diff --git a/src/mainboard/intel/baskingridge/Kconfig b/src/mainboard/intel/baskingridge/Kconfig
index 445dbcb..1fd07c2 100644
--- a/src/mainboard/intel/baskingridge/Kconfig
+++ b/src/mainboard/intel/baskingridge/Kconfig
@@ -37,12 +37,4 @@
string
default "pci8086,0166.rom"
-config HAVE_IFD_BIN
- bool
- default n
-
-config HAVE_ME_BIN
- bool
- default n
-
endif # BOARD_INTEL_BASKING_RIDGE
diff --git a/src/southbridge/intel/lynxpoint/Kconfig b/src/southbridge/intel/lynxpoint/Kconfig
index 8d5cbbf..4e5527a 100644
--- a/src/southbridge/intel/lynxpoint/Kconfig
+++ b/src/southbridge/intel/lynxpoint/Kconfig
@@ -57,28 +57,10 @@
If you set this option to y, the serial IRQ machine will be
operated in continuous mode.
-config HAVE_IFD_BIN
- bool
- default y
-
config BUILD_WITH_FAKE_IFD
bool
default y if !HAVE_IFD_BIN
-config IFD_BIN_PATH
- string
- depends on !BUILD_WITH_FAKE_IFD
- default "3rdparty/blobs/mainboard/$(MAINBOARDDIR)/descriptor.bin"
-
-config HAVE_ME_BIN
- bool
- default y
-
-config ME_BIN_PATH
- string
- depends on HAVE_ME_BIN
- default "3rdparty/blobs/mainboard/$(MAINBOARDDIR)/me.bin"
-
config ME_MBP_CLEAR_LATE
bool "Defer wait for ME MBP Cleared"
default y
--
To view, visit https://review.coreboot.org/28007
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings
Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9999440031b07006e2df11e00dfb9f3dbe04f832
Gerrit-Change-Number: 28007
Gerrit-PatchSet: 1
Gerrit-Owner: Arthur Heymans <arthur(a)aheymans.xyz>