[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