net: rfkill: gpio: Switch to devm_acpi_dev_add_driver_gpios()
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Sat, 10 Jun 2017 19:10:34 +0000 (22:10 +0300)
committerJohannes Berg <johannes.berg@intel.com>
Tue, 13 Jun 2017 09:07:51 +0000 (11:07 +0200)
Switch to use managed variant of acpi_dev_add_driver_gpios() to simplify
error path and fix potentially wrong assingment if ->probe() fails.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
net/rfkill/rfkill-gpio.c

index 76c01cbd56e35b209bf5e2599298471263f14ded..41bd496531d45e8d43428584605bfb55ef249169 100644 (file)
@@ -81,8 +81,7 @@ static int rfkill_gpio_acpi_probe(struct device *dev,
 
        rfkill->type = (unsigned)id->driver_data;
 
-       return acpi_dev_add_driver_gpios(ACPI_COMPANION(dev),
-                                        acpi_rfkill_default_gpios);
+       return devm_acpi_dev_add_driver_gpios(dev, acpi_rfkill_default_gpios);
 }
 
 static int rfkill_gpio_probe(struct platform_device *pdev)
@@ -154,8 +153,6 @@ static int rfkill_gpio_remove(struct platform_device *pdev)
        rfkill_unregister(rfkill->rfkill_dev);
        rfkill_destroy(rfkill->rfkill_dev);
 
-       acpi_dev_remove_driver_gpios(ACPI_COMPANION(&pdev->dev));
-
        return 0;
 }