[coreboot-gerrit] [S] Change in coreboot[main]: sb/intel/bd82x6x: Remove dummy SOUTH_BRIDGE_OPTIONS Kconfig symbol

[Felix Singer (Code Review)]

Felix Singer has submitted this change. ( https://review.coreboot.org/c/coreboot/+/77532?usp=email )

Change subject: sb/intel/bd82x6x: Remove dummy SOUTH_BRIDGE_OPTIONS Kconfig symbol
......................................................................

sb/intel/bd82x6x: Remove dummy SOUTH_BRIDGE_OPTIONS Kconfig symbol

Change-Id: If4ae9c800325f5de8f170f8922bacffad1c9bb20
Signed-off-by: Elyes Haouas 
Reviewed-on: https://review.coreboot.org/c/coreboot/+/77532
Reviewed-by: Felix Singer 
Tested-by: build bot (Jenkins) 
Reviewed-by: Nico Huber 
---
M src/southbridge/intel/bd82x6x/Kconfig
1 file changed, 12 insertions(+), 10 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Nico Huber: Looks good to me, but someone else must approve
  Felix Singer: Looks good to me, approved

diff --git a/src/southbridge/intel/bd82x6x/Kconfig b/src/southbridge/intel/bd82x6x/Kconfig
index 41732bd..1a578dd 100644
--- a/src/southbridge/intel/bd82x6x/Kconfig
+++ b/src/southbridge/intel/bd82x6x/Kconfig
@@ -1,15 +1,7 @@
 ## SPDX-License-Identifier: GPL-2.0-only

-config SOUTHBRIDGE_INTEL_BD82X6X
-	bool
-
-config SOUTHBRIDGE_INTEL_C216
-	bool
-
-if SOUTHBRIDGE_INTEL_BD82X6X || SOUTHBRIDGE_INTEL_C216
-
-config SOUTH_BRIDGE_OPTIONS
-	def_bool y
+config SOUTHBRIDGE_INTEL_BD82X6X_COMMON
+	def_bool n
    select ACPI_COMMON_MADT_IOAPIC
    select ACPI_COMMON_MADT_LAPIC
    select ACPI_INTEL_HARDWARE_SLEEP_VALUES
@@ -39,6 +31,16 @@
    select SOUTHBRIDGE_INTEL_COMMON_USB_DEBUG
    select TCO_SPACE_NOT_YET_SPLIT

+config SOUTHBRIDGE_INTEL_BD82X6X
+	bool
+	select SOUTHBRIDGE_INTEL_BD82X6X_COMMON
+
+config SOUTHBRIDGE_INTEL_C216
+	bool
+	select SOUTHBRIDGE_INTEL_BD82X6X_COMMON
+
+if SOUTHBRIDGE_INTEL_BD82X6X_COMMON
+
 config EHCI_BAR
    hex
    default 0xfef00000