mfd: twl4030-madc: Remove unnecessary platform_set_drvdata()
authorJingoo Han <jg1.han@samsung.com>
Mon, 6 May 2013 04:08:08 +0000 (13:08 +0900)
committerSamuel Ortiz <sameo@linux.intel.com>
Tue, 11 Jun 2013 17:27:52 +0000 (19:27 +0200)
The driver core clears the driver data to NULL after device_release
or on probe failure, since commit 0998d0631001288a5974afc0b2a5f568bcdecb4d
(device-core: Ensure drvdata = NULL when no driver is bound).
Thus, it is not needed to manually clear the device driver data to NULL.

Signed-off-by: Jingoo Han <jg1.han@samsung.com>
Acked-by: Lee Jones <lee.jones@linaro.org>
Signed-off-by: Samuel Ortiz <sameo@linux.intel.com>
drivers/mfd/twl4030-madc.c

index 42bd3ea..1ea54d4 100644 (file)
@@ -775,12 +775,10 @@ static int twl4030_madc_probe(struct platform_device *pdev)
                                   IRQF_TRIGGER_RISING, "twl4030_madc", madc);
        if (ret) {
                dev_dbg(&pdev->dev, "could not request irq\n");
-               goto err_irq;
+               goto err_i2c;
        }
        twl4030_madc = madc;
        return 0;
-err_irq:
-       platform_set_drvdata(pdev, NULL);
 err_i2c:
        twl4030_madc_set_current_generator(madc, 0, 0);
 err_current_generator:
@@ -796,7 +794,6 @@ static int twl4030_madc_remove(struct platform_device *pdev)
        struct twl4030_madc_data *madc = platform_get_drvdata(pdev);
 
        free_irq(platform_get_irq(pdev, 0), madc);
-       platform_set_drvdata(pdev, NULL);
        twl4030_madc_set_current_generator(madc, 0, 0);
        twl4030_madc_set_power(madc, 0);
        kfree(madc);