eeepc-laptop: clean up control flow in *_rfkill_notifier
authorFrans Klaver <fransklaver@gmail.com>
Sat, 20 Sep 2014 22:22:17 +0000 (00:22 +0200)
committerDarren Hart <dvhart@linux.intel.com>
Mon, 22 Sep 2014 21:17:54 +0000 (14:17 -0700)
Handle errors immediately in eeepc_register_rfkill_notifier and
eeepc_unregister_rfkill_notifier. This clears up the control flow for the
reader. It also removes unnecessary indentation.

Signed-off-by: Frans Klaver <fransklaver@gmail.com>
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
drivers/platform/x86/eeepc-laptop.c

index 3f6c762fb560ac35bc460eb72f3c2bde47bfdb9e..db79902c4a8ee86ee17c8c27039fd0d868d0792b 100644 (file)
@@ -668,23 +668,21 @@ static int eeepc_register_rfkill_notifier(struct eeepc_laptop *eeepc,
 
        status = acpi_get_handle(NULL, node, &handle);
 
-       if (ACPI_SUCCESS(status)) {
-               status = acpi_install_notify_handler(handle,
-                                                    ACPI_SYSTEM_NOTIFY,
-                                                    eeepc_rfkill_notify,
-                                                    eeepc);
-               if (ACPI_FAILURE(status))
-                       pr_warn("Failed to register notify on %s\n", node);
-
-               /*
-                * Refresh pci hotplug in case the rfkill state was
-                * changed during setup.
-                */
-               eeepc_rfkill_hotplug(eeepc, handle);
-       } else {
+       if (ACPI_FAILURE(status))
                return -ENODEV;
-       }
 
+       status = acpi_install_notify_handler(handle,
+                                            ACPI_SYSTEM_NOTIFY,
+                                            eeepc_rfkill_notify,
+                                            eeepc);
+       if (ACPI_FAILURE(status))
+               pr_warn("Failed to register notify on %s\n", node);
+
+       /*
+        * Refresh pci hotplug in case the rfkill state was
+        * changed during setup.
+        */
+       eeepc_rfkill_hotplug(eeepc, handle);
        return 0;
 }
 
@@ -696,20 +694,21 @@ static void eeepc_unregister_rfkill_notifier(struct eeepc_laptop *eeepc,
 
        status = acpi_get_handle(NULL, node, &handle);
 
-       if (ACPI_SUCCESS(status)) {
-               status = acpi_remove_notify_handler(handle,
-                                                    ACPI_SYSTEM_NOTIFY,
-                                                    eeepc_rfkill_notify);
-               if (ACPI_FAILURE(status))
-                       pr_err("Error removing rfkill notify handler %s\n",
-                               node);
-                       /*
-                        * Refresh pci hotplug in case the rfkill
-                        * state was changed after
-                        * eeepc_unregister_rfkill_notifier()
-                        */
-               eeepc_rfkill_hotplug(eeepc, handle);
-       }
+       if (ACPI_FAILURE(status))
+               return;
+
+       status = acpi_remove_notify_handler(handle,
+                                            ACPI_SYSTEM_NOTIFY,
+                                            eeepc_rfkill_notify);
+       if (ACPI_FAILURE(status))
+               pr_err("Error removing rfkill notify handler %s\n",
+                       node);
+               /*
+                * Refresh pci hotplug in case the rfkill
+                * state was changed after
+                * eeepc_unregister_rfkill_notifier()
+                */
+       eeepc_rfkill_hotplug(eeepc, handle);
 }
 
 static int eeepc_get_adapter_status(struct hotplug_slot *hotplug_slot,