Use devm_of_platform_populate() to be sure that of_platform_depopulate
is called when removing the driver.

Signed-off-by: Benjamin Gaignard <benjamin.gaign...@linaro.org>

CC: Lee Jones <lee.jo...@linaro.org>
CC: linux-kernel@vger.kernel.org
---
 drivers/mfd/fsl-imx25-tsadc.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/drivers/mfd/fsl-imx25-tsadc.c b/drivers/mfd/fsl-imx25-tsadc.c
index ac430a3..4258a42 100644
--- a/drivers/mfd/fsl-imx25-tsadc.c
+++ b/drivers/mfd/fsl-imx25-tsadc.c
@@ -129,7 +129,6 @@ static void mx25_tsadc_setup_clk(struct platform_device 
*pdev,
 static int mx25_tsadc_probe(struct platform_device *pdev)
 {
        struct device *dev = &pdev->dev;
-       struct device_node *np = dev->of_node;
        struct mx25_tsadc *tsadc;
        struct resource *res;
        int ret;
@@ -178,9 +177,7 @@ static int mx25_tsadc_probe(struct platform_device *pdev)
 
        platform_set_drvdata(pdev, tsadc);
 
-       of_platform_populate(np, NULL, NULL, dev);
-
-       return 0;
+       return devm_of_platform_populate(dev);
 }
 
 static const struct of_device_id mx25_tsadc_ids[] = {
-- 
1.9.1

Reply via email to