extcon: Convert extcon_gpio to devm_gpio_request_one
authorAxel Lin <axel.lin@gmail.com>
Sat, 16 Jun 2012 03:56:24 +0000 (11:56 +0800)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 18 Jun 2012 23:40:18 +0000 (16:40 -0700)
Also remove unneeded devm_kfree calls.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/extcon/extcon_gpio.c

index 8a0dcc11c7c73e9c1abb64b7537047d58115064b..fe3db45fa83c13604db19590cb29546dc38efae1 100644 (file)
@@ -105,25 +105,25 @@ static int __devinit gpio_extcon_probe(struct platform_device *pdev)
 
        ret = extcon_dev_register(&extcon_data->edev, &pdev->dev);
        if (ret < 0)
-               goto err_extcon_dev_register;
+               return ret;
 
        ret = gpio_request_one(extcon_data->gpio, GPIOF_DIR_IN, pdev->name);
        if (ret < 0)
-               goto err_request_gpio;
+               goto err;
 
        INIT_DELAYED_WORK(&extcon_data->work, gpio_extcon_work);
 
        extcon_data->irq = gpio_to_irq(extcon_data->gpio);
        if (extcon_data->irq < 0) {
                ret = extcon_data->irq;
-               goto err_detect_irq_num_failed;
+               goto err;
        }
 
        ret = request_any_context_irq(extcon_data->irq, gpio_irq_handler,
                                      pdata->irq_flags, pdev->name,
                                      extcon_data);
        if (ret < 0)
-               goto err_request_irq;
+               goto err;
 
        platform_set_drvdata(pdev, extcon_data);
        /* Perform initial detection */
@@ -131,13 +131,8 @@ static int __devinit gpio_extcon_probe(struct platform_device *pdev)
 
        return 0;
 
-err_request_irq:
-err_detect_irq_num_failed:
-       gpio_free(extcon_data->gpio);
-err_request_gpio:
+err:
        extcon_dev_unregister(&extcon_data->edev);
-err_extcon_dev_register:
-       devm_kfree(&pdev->dev, extcon_data);
 
        return ret;
 }
@@ -148,9 +143,7 @@ static int __devexit gpio_extcon_remove(struct platform_device *pdev)
 
        cancel_delayed_work_sync(&extcon_data->work);
        free_irq(extcon_data->irq, extcon_data);
-       gpio_free(extcon_data->gpio);
        extcon_dev_unregister(&extcon_data->edev);
-       devm_kfree(&pdev->dev, extcon_data);
 
        return 0;
 }