ACPI: SBS: Simplify the driver init code
authorHanjun Guo <guohanjun@huawei.com>
Fri, 13 Nov 2020 02:25:10 +0000 (10:25 +0800)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Tue, 17 Nov 2020 17:12:34 +0000 (18:12 +0100)
acpi_bus_register_driver() will check acpi_disable and return
-ENODEV, so the acpi_disable check outside the
acpi_bus_register_driver() is duplicated, can be removed.

Also we can just return from acpi_bus_register_driver() then
we can simplify the code further.

Signed-off-by: Hanjun Guo <guohanjun@huawei.com>
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/sbs.c

index e6d9f4d..cbd1af7 100644 (file)
@@ -714,16 +714,7 @@ static struct acpi_driver acpi_sbs_driver = {
 
 static int __init acpi_sbs_init(void)
 {
-       int result = 0;
-
-       if (acpi_disabled)
-               return -ENODEV;
-
-       result = acpi_bus_register_driver(&acpi_sbs_driver);
-       if (result < 0)
-               return -ENODEV;
-
-       return 0;
+       return acpi_bus_register_driver(&acpi_sbs_driver);
 }
 
 static void __exit acpi_sbs_exit(void)