[coreboot-gerrit] New patch to review for coreboot: intel/wifi: Include conditionally in the build

Kyösti Mälkki (kyosti.malkki@gmail.com) gerrit at coreboot.org
Thu Jul 28 16:29:16 CEST 2016


Kyösti Mälkki (kyosti.malkki at gmail.com) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15931

-gerrit

commit 399aaaf5e85fcab09ac2665e5e50a5376e7225fd
Author: Kyösti Mälkki <kyosti.malkki at gmail.com>
Date:   Thu Jul 28 17:26:39 2016 +0300

    intel/wifi: Include conditionally in the build
    
    Keep this enabled by default as most x86 platforms could have PCI-e
    slots equipped with one of these Intel WiFi adapters.
    
    The Kconfig entries under google boards had no function previously,
    the variable was never referenced.
    
    Change-Id: I728ce3fd83d51d4e5e32b848a2079c5fcee29349
    Signed-off-by: Kyösti Mälkki <kyosti.malkki at gmail.com>
---
 src/drivers/intel/wifi/Kconfig        | 6 ++++++
 src/drivers/intel/wifi/Makefile.inc   | 7 +------
 src/drivers/intel/wifi/wifi.c         | 4 ++++
 src/mainboard/google/chell/Kconfig    | 3 ---
 src/mainboard/google/glados/Kconfig   | 3 ---
 src/mainboard/google/lars/Kconfig     | 3 ---
 src/mainboard/intel/kunimitsu/Kconfig | 3 ---
 7 files changed, 11 insertions(+), 18 deletions(-)

diff --git a/src/drivers/intel/wifi/Kconfig b/src/drivers/intel/wifi/Kconfig
new file mode 100644
index 0000000..40a8528
--- /dev/null
+++ b/src/drivers/intel/wifi/Kconfig
@@ -0,0 +1,6 @@
+config DRIVERS_INTEL_WIFI
+	bool "Support Intel PCI-e WiFi adapters"
+	default y if PCIEXP_PLUGIN_SUPPORT
+	help
+	   When enabled, add identifiers in ACPI and SMBIOS tables to
+	   make OS drivers work with certain Intel PCI-e WiFi chipsets.
diff --git a/src/drivers/intel/wifi/Makefile.inc b/src/drivers/intel/wifi/Makefile.inc
index 1435f8b..aab811c 100644
--- a/src/drivers/intel/wifi/Makefile.inc
+++ b/src/drivers/intel/wifi/Makefile.inc
@@ -11,9 +11,4 @@
 # GNU General Public License for more details.
 #
 
-# really?
-ifeq ($(CONFIG_GENERATE_SMBIOS_TABLES),y)
-
-ramstage-$(CONFIG_PCIEXP_PLUGIN_SUPPORT) += wifi.c
-
-endif
+ramstage-$(CONFIG_DRIVERS_INTEL_WIFI) += wifi.c
diff --git a/src/drivers/intel/wifi/wifi.c b/src/drivers/intel/wifi/wifi.c
index 5661bf4..789d0d5 100644
--- a/src/drivers/intel/wifi/wifi.c
+++ b/src/drivers/intel/wifi/wifi.c
@@ -25,6 +25,7 @@
 #include <wrdd.h>
 #include "chip.h"
 
+#if IS_ENABLED(CONFIG_GENERATE_SMBIOS_TABLES)
 static int smbios_write_wifi(struct device *dev, int *handle,
 			     unsigned long *current)
 {
@@ -55,6 +56,7 @@ static int smbios_write_wifi(struct device *dev, int *handle,
 	*handle += 1;
 	return len;
 }
+#endif
 
 #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
 static void intel_wifi_fill_ssdt(struct device *dev)
@@ -125,7 +127,9 @@ struct device_operations device_ops = {
 	.set_resources            = pci_dev_set_resources,
 	.enable_resources         = pci_dev_enable_resources,
 	.init                     = pci_dev_init,
+#if IS_ENABLED(CONFIG_GENERATE_SMBIOS_TABLES)
 	.get_smbios_data          = smbios_write_wifi,
+#endif
 	.ops_pci                  = &pci_ops,
 #if IS_ENABLED(CONFIG_HAVE_ACPI_TABLES)
 	.acpi_name                = &intel_wifi_acpi_name,
diff --git a/src/mainboard/google/chell/Kconfig b/src/mainboard/google/chell/Kconfig
index 5ed211a..6fcdad4 100644
--- a/src/mainboard/google/chell/Kconfig
+++ b/src/mainboard/google/chell/Kconfig
@@ -28,9 +28,6 @@ config DRIVERS_I2C_GENERIC
 config DRIVERS_I2C_NAU8825
 	def_bool y
 
-config DRIVERS_INTEL_WIFI
-	def_bool y
-
 config IRQ_SLOT_COUNT
 	int
 	default 18
diff --git a/src/mainboard/google/glados/Kconfig b/src/mainboard/google/glados/Kconfig
index 3acb8e9..83049b7 100644
--- a/src/mainboard/google/glados/Kconfig
+++ b/src/mainboard/google/glados/Kconfig
@@ -28,9 +28,6 @@ config DRIVERS_I2C_GENERIC
 config DRIVERS_I2C_NAU8825
 	def_bool y
 
-config DRIVERS_INTEL_WIFI
-	def_bool y
-
 config IRQ_SLOT_COUNT
 	int
 	default 18
diff --git a/src/mainboard/google/lars/Kconfig b/src/mainboard/google/lars/Kconfig
index c602523..44c1018 100644
--- a/src/mainboard/google/lars/Kconfig
+++ b/src/mainboard/google/lars/Kconfig
@@ -32,9 +32,6 @@ config DRIVERS_I2C_GENERIC
 config DRIVERS_I2C_NAU8825
 	default y
 
-config DRIVERS_INTEL_WIFI
-	default y
-
 config IRQ_SLOT_COUNT
 	int
 	default 18
diff --git a/src/mainboard/intel/kunimitsu/Kconfig b/src/mainboard/intel/kunimitsu/Kconfig
index d2e6951..c203df0 100644
--- a/src/mainboard/intel/kunimitsu/Kconfig
+++ b/src/mainboard/intel/kunimitsu/Kconfig
@@ -31,9 +31,6 @@ config DRIVERS_I2C_GENERIC
 config DRIVERS_I2C_NAU8825
 	default y
 
-config DRIVERS_INTEL_WIFI
-	default y
-
 config IRQ_SLOT_COUNT
 	int
 	default 18



More information about the coreboot-gerrit mailing list