Use "devm_regulator_register" instead of "regulator_register" which simplifies the code. Signed-off-by: Manish Badarkhe <badarkhe.manish@xxxxxxxxx> --- :100644 100644 5ea64b9... 6d32341... M drivers/regulator/fixed.c drivers/regulator/fixed.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/regulator/fixed.c b/drivers/regulator/fixed.c index 5ea64b9..6d32341 100644 --- a/drivers/regulator/fixed.c +++ b/drivers/regulator/fixed.c @@ -186,7 +186,8 @@ static int reg_fixed_voltage_probe(struct platform_device *pdev) cfg.driver_data = drvdata; cfg.of_node = pdev->dev.of_node; - drvdata->dev = regulator_register(&drvdata->desc, &cfg); + drvdata->dev = devm_regulator_register(&pdev->dev, &drvdata->desc, + &cfg); if (IS_ERR(drvdata->dev)) { ret = PTR_ERR(drvdata->dev); dev_err(&pdev->dev, "Failed to register regulator: %d\n", ret); @@ -212,7 +213,6 @@ static int reg_fixed_voltage_remove(struct platform_device *pdev) { struct fixed_voltage_data *drvdata = platform_get_drvdata(pdev); - regulator_unregister(drvdata->dev); kfree(drvdata->desc.supply_name); kfree(drvdata->desc.name); -- 1.7.10.4 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html