functionality.  If in doubt, say Y here; it will only load on
          supported platforms.
 
-config INTEL_SMARTCONNECT
-        tristate "Intel Smart Connect disabling driver"
-       depends on ACPI
-       help
-         Intel Smart Connect is a technology intended to permit devices to
-         update state by resuming for a short period of time at regular
-         intervals. If a user enables this functionality under Windows and
-         then reboots into Linux, the system may remain configured to resume
-         on suspend. In the absence of any userspace to support it, the system
-         will then remain awake until something triggers another suspend.
-
-         This driver checks to determine whether the device has Intel Smart
-         Connect enabled, and if so disables it.
-
 source "drivers/platform/x86/intel_speed_select_if/Kconfig"
 
 config INTEL_TURBO_MAX_3
 
 
 # Intel uncore drivers
 obj-$(CONFIG_INTEL_IPS)                                += intel_ips.o
-obj-$(CONFIG_INTEL_SMARTCONNECT)               += intel-smartconnect.o
 obj-$(CONFIG_INTEL_SPEED_SELECT_INTERFACE)     += intel_speed_select_if/
 obj-$(CONFIG_INTEL_TURBO_MAX_3)                        += intel_turbo_max_3.o
 obj-$(CONFIG_INTEL_UNCORE_FREQ_CONTROL)                += intel-uncore-frequency.o
 
+++ /dev/null
-// SPDX-License-Identifier: GPL-2.0+
-/*
- *  Copyright 2013 Matthew Garrett <mjg59@srcf.ucam.org>
- */
-
-#include <linux/acpi.h>
-#include <linux/module.h>
-
-MODULE_LICENSE("GPL");
-
-static int smartconnect_acpi_init(struct acpi_device *acpi)
-{
-       unsigned long long value;
-       acpi_status status;
-
-       status = acpi_evaluate_integer(acpi->handle, "GAOS", NULL, &value);
-       if (ACPI_FAILURE(status))
-               return -EINVAL;
-
-       if (value & 0x1) {
-               dev_info(&acpi->dev, "Disabling Intel Smart Connect\n");
-               status = acpi_execute_simple_method(acpi->handle, "SAOS", 0);
-       }
-
-       return 0;
-}
-
-static const struct acpi_device_id smartconnect_ids[] = {
-       {"INT33A0", 0},
-       {"", 0}
-};
-MODULE_DEVICE_TABLE(acpi, smartconnect_ids);
-
-static struct acpi_driver smartconnect_driver = {
-       .owner = THIS_MODULE,
-       .name = "intel_smart_connect",
-       .class = "intel_smart_connect",
-       .ids = smartconnect_ids,
-       .ops = {
-               .add = smartconnect_acpi_init,
-       },
-};
-
-module_acpi_driver(smartconnect_driver);
 
          firmware will copy the memory contents back to RAM and resume the OS
          as usual.
 
+config INTEL_SMARTCONNECT
+       tristate "Intel Smart Connect disabling driver"
+       depends on ACPI
+       help
+         Intel Smart Connect is a technology intended to permit devices to
+         update state by resuming for a short period of time at regular
+         intervals. If a user enables this functionality under Windows and
+         then reboots into Linux, the system may remain configured to resume
+         on suspend. In the absence of any userspace to support it, the system
+         will then remain awake until something triggers another suspend.
+
+         This driver checks to determine whether the device has Intel Smart
+         Connect enabled, and if so disables it.
+
 endif # X86_PLATFORM_DRIVERS_INTEL
 
 # Intel Uncore drivers
 intel-rst-y                            := rst.o
 obj-$(CONFIG_INTEL_RST)                        += intel-rst.o
+intel-smartconnect-y                   := smartconnect.o
+obj-$(CONFIG_INTEL_SMARTCONNECT)       += intel-smartconnect.o
 
--- /dev/null
+// SPDX-License-Identifier: GPL-2.0+
+/*
+ *  Copyright 2013 Matthew Garrett <mjg59@srcf.ucam.org>
+ */
+
+#include <linux/acpi.h>
+#include <linux/module.h>
+
+MODULE_LICENSE("GPL");
+
+static int smartconnect_acpi_init(struct acpi_device *acpi)
+{
+       unsigned long long value;
+       acpi_status status;
+
+       status = acpi_evaluate_integer(acpi->handle, "GAOS", NULL, &value);
+       if (ACPI_FAILURE(status))
+               return -EINVAL;
+
+       if (value & 0x1) {
+               dev_info(&acpi->dev, "Disabling Intel Smart Connect\n");
+               status = acpi_execute_simple_method(acpi->handle, "SAOS", 0);
+       }
+
+       return 0;
+}
+
+static const struct acpi_device_id smartconnect_ids[] = {
+       {"INT33A0", 0},
+       {"", 0}
+};
+MODULE_DEVICE_TABLE(acpi, smartconnect_ids);
+
+static struct acpi_driver smartconnect_driver = {
+       .owner = THIS_MODULE,
+       .name = "intel_smart_connect",
+       .class = "intel_smart_connect",
+       .ids = smartconnect_ids,
+       .ops = {
+               .add = smartconnect_acpi_init,
+       },
+};
+
+module_acpi_driver(smartconnect_driver);