[coreboot-gerrit] Change in coreboot[master]: drivers/wifi/generic: Move SMBIOS functions to a separate file

[Furquan Shaikh (Code Review)]

Furquan Shaikh has uploaded this change for review. ( https://review.coreboot.org/c/coreboot/+/46859 )

Change subject: drivers/wifi/generic: Move SMBIOS functions to a separate file
......................................................................

drivers/wifi/generic: Move SMBIOS functions to a separate file

This change reorganizes the WiFi generic driver to move the SMBIOS
functions to a separate file. This change is done to reduce the noise
in generic.c file and improve readability of the file.

Change-Id: I38ed46f5ae1594945d2078b00e8315d9234f36d7
Signed-off-by: Furquan Shaikh 
---
M src/drivers/wifi/generic/Makefile.inc
M src/drivers/wifi/generic/generic.c
A src/drivers/wifi/generic/smbios.c
A src/drivers/wifi/generic/wifi_private.h
4 files changed, 58 insertions(+), 39 deletions(-)

git pull ssh://review.coreboot.org:29418/coreboot refs/changes/59/46859/1

diff --git a/src/drivers/wifi/generic/Makefile.inc b/src/drivers/wifi/generic/Makefile.inc
index 407041a..3683c3e 100644
--- a/src/drivers/wifi/generic/Makefile.inc
+++ b/src/drivers/wifi/generic/Makefile.inc
@@ -1,5 +1,11 @@
-ramstage-$(CONFIG_DRIVERS_WIFI_GENERIC) += generic.c
+
+ifeq ($(CONFIG_DRIVERS_WIFI_GENERIC),y)
+
+ramstage-y += generic.c
+ramstage-$(CONFIG_GENERATE_SMBIOS_TABLES) += smbios.c

cbfs-files-$(CONFIG_WIFI_SAR_CBFS)  += wifi_sar_defaults.hex
 wifi_sar_defaults.hex-file := $(call strip_quotes,$(CONFIG_WIFI_SAR_CBFS_FILEPATH))
 wifi_sar_defaults.hex-type := raw
+
+endif
diff --git a/src/drivers/wifi/generic/generic.c b/src/drivers/wifi/generic/generic.c
index 6ffa7f1..56944e61 100644
--- a/src/drivers/wifi/generic/generic.c
+++ b/src/drivers/wifi/generic/generic.c
@@ -9,10 +9,9 @@
 #include <device/pci_ids.h>
 #include <elog.h>
 #include <sar.h>
-#include <smbios.h>
-#include <string.h>
 #include <wrdd.h>
 #include "chip.h"
+#include "wifi_private.h"

/* WRDS Spec Revision */
 #define WRDS_REVISION 0x0
@@ -243,42 +242,6 @@
    	elog_add_event_wake(ELOG_WAKE_SOURCE_PME_WIFI, 0);
 }

-#if CONFIG(GENERATE_SMBIOS_TABLES)
-static int smbios_write_intel_wifi(struct device *dev, int *handle, unsigned long *current)
-{
-	struct smbios_type_intel_wifi {
-		u8 type;
-		u8 length;
-		u16 handle;
-		u8 str;
-		u8 eos[2];
-	} __packed;
-
-	struct smbios_type_intel_wifi *t = (struct smbios_type_intel_wifi *)*current;
-	int len = sizeof(struct smbios_type_intel_wifi);
-
-	memset(t, 0, sizeof(struct smbios_type_intel_wifi));
-	t->type = 0x85;
-	t->length = len - 2;
-	t->handle = *handle;
-	/* Intel wifi driver expects this string to be in the table 0x85. */
-	t->str = smbios_add_string(t->eos, "KHOIHGIUCCHHII");
-
-	len = t->length + smbios_string_table_len(t->eos);
-	*current += len;
-	*handle += 1;
-	return len;
-}
-
-static int smbios_write_wifi(struct device *dev, int *handle, unsigned long *current)
-{
-	if (dev->vendor == PCI_VENDOR_ID_INTEL)
-		return smbios_write_intel_wifi(dev, handle, current);
-
-	return 0;
-}
-#endif
-
 struct device_operations wifi_generic_ops = {
    .read_resources		= pci_dev_read_resources,
    .set_resources		= pci_dev_set_resources,
diff --git a/src/drivers/wifi/generic/smbios.c b/src/drivers/wifi/generic/smbios.c
new file mode 100644
index 0000000..0c1b976
--- /dev/null
+++ b/src/drivers/wifi/generic/smbios.c
@@ -0,0 +1,42 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#include <device/device.h>
+#include <device/pci_ids.h>
+#include <smbios.h>
+#include <string.h>
+
+#include "wifi_private.h"
+
+static int smbios_write_intel_wifi(struct device *dev, int *handle, unsigned long *current)
+{
+	struct smbios_type_intel_wifi {
+		u8 type;
+		u8 length;
+		u16 handle;
+		u8 str;
+		u8 eos[2];
+	} __packed;
+
+	struct smbios_type_intel_wifi *t = (struct smbios_type_intel_wifi *)*current;
+	int len = sizeof(struct smbios_type_intel_wifi);
+
+	memset(t, 0, sizeof(struct smbios_type_intel_wifi));
+	t->type = 0x85;
+	t->length = len - 2;
+	t->handle = *handle;
+	/* Intel wifi driver expects this string to be in the table 0x85. */
+	t->str = smbios_add_string(t->eos, "KHOIHGIUCCHHII");
+
+	len = t->length + smbios_string_table_len(t->eos);
+	*current += len;
+	*handle += 1;
+	return len;
+}
+
+int smbios_write_wifi(struct device *dev, int *handle, unsigned long *current)
+{
+	if (dev->vendor == PCI_VENDOR_ID_INTEL)
+		return smbios_write_intel_wifi(dev, handle, current);
+
+	return 0;
+}
diff --git a/src/drivers/wifi/generic/wifi_private.h b/src/drivers/wifi/generic/wifi_private.h
new file mode 100644
index 0000000..11b9198
--- /dev/null
+++ b/src/drivers/wifi/generic/wifi_private.h
@@ -0,0 +1,8 @@
+/* SPDX-License-Identifier: GPL-2.0-only */
+
+#ifndef _WIFI_GENERIC_PRIVATE_H_
+#define _WIFI_GENERIC_PRIVATE_H_
+
+int smbios_write_wifi(struct device *dev, int *handle, unsigned long *current);
+
+#endif