Messages in this thread | | | Date | Thu, 26 Sep 2013 11:50:17 +0530 | Subject | Re: [PATCH V2 06/18] regulator: gpio-regulator: use devm_regulator_register() | From | Sachin Kamat <> |
| |
On 26 September 2013 07:36, Jingoo Han <jg1.han@samsung.com> wrote: > Use devm_regulator_register() to make cleanup paths simpler. > @@ -349,8 +350,6 @@ static int gpio_regulator_remove(struct platform_device *pdev) > { > struct gpio_regulator_data *drvdata = platform_get_drvdata(pdev); > > - regulator_unregister(drvdata->dev); > - > gpio_free_array(drvdata->gpios, drvdata->nr_gpios); > > kfree(drvdata->states);
In most of the cases where unregister doesn't happen to be the only or last call in the remove path, I am not sure if change in ordering wouldn't cause any functional issues. For. e.g., in this patch we are freeing the gpios and the driver states even before unregistering the device which is logically not right.
-- With warm regards, Sachin
| |