fjes: Do not load fjes driver if extended socket device is not power on.
authorYasuaki Ishimatsu <yasu.isimatu@gmail.com>
Tue, 21 Mar 2017 15:46:35 +0000 (11:46 -0400)
committerDavid S. Miller <davem@davemloft.net>
Wed, 22 Mar 2017 19:38:17 +0000 (12:38 -0700)
The extended device socket cannot turn on/off while system is running.
So when system boots up and the device is not power on, the fjes driver
does not need be loaded.

To check the status of the device, the patch adds ACPI _STA method check.

Signed-off-by: Yasuaki Ishimatsu <isimatu.yasuaki@jp.fujitsu.com>
CC: Taku Izumi <izumi.taku@jp.fujitsu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/fjes/fjes_main.c

index 7b58964..ae48c80 100644 (file)
@@ -144,6 +144,24 @@ static bool is_extended_socket_device(struct acpi_device *device)
        return true;
 }
 
+static int acpi_check_extended_socket_status(struct acpi_device *device)
+{
+       unsigned long long sta;
+       acpi_status status;
+
+       status = acpi_evaluate_integer(device->handle, "_STA", NULL, &sta);
+       if (ACPI_FAILURE(status))
+               return -ENODEV;
+
+       if (!((sta & ACPI_STA_DEVICE_PRESENT) &&
+             (sta & ACPI_STA_DEVICE_ENABLED) &&
+             (sta & ACPI_STA_DEVICE_UI) &&
+             (sta & ACPI_STA_DEVICE_FUNCTIONING)))
+               return -ENODEV;
+
+       return 0;
+}
+
 static int fjes_acpi_add(struct acpi_device *device)
 {
        struct platform_device *plat_dev;
@@ -152,6 +170,9 @@ static int fjes_acpi_add(struct acpi_device *device)
        if (!is_extended_socket_device(device))
                return -ENODEV;
 
+       if (acpi_check_extended_socket_status(device))
+               return -ENODEV;
+
        status = acpi_walk_resources(device->handle, METHOD_NAME__CRS,
                                     fjes_get_acpi_resource, fjes_resource);
        if (ACPI_FAILURE(status))
@@ -1503,6 +1524,9 @@ acpi_find_extended_socket_device(acpi_handle obj_handle, u32 level,
        if (!is_extended_socket_device(device))
                return AE_OK;
 
+       if (acpi_check_extended_socket_status(device))
+               return AE_OK;
+
        *found = true;
        return AE_CTRL_TERMINATE;
 }