Messages in this thread | | | Date | Wed, 25 Sep 2013 15:50:31 +0530 | Subject | Re: [PATCH 02/20] regulator: ab8500: use devm_regulator_register() | From | Sachin Kamat <> |
| |
On 25 September 2013 13:21, Jingoo Han <jg1.han@samsung.com> wrote: > Use devm_regulator_register() to make cleanup paths simpler.
> - for (i = 0; i < abx500_regulator.info_size; i++) { > - struct ab8500_regulator_info *info = NULL; > - info = &abx500_regulator.info[i]; > - > - dev_vdbg(rdev_get_dev(info->regulator), > - "%s-remove\n", info->desc.name); > - > - regulator_unregister(info->regulator); > - } > + int err; > > /* remove regulator debug */ > err = ab8500_regulator_debug_exit(pdev);
Wouldn't re-ordering (unregistering after ab8500_regulator_debug_exit) cause any problems here ?
-- With warm regards, Sachin
| |