regulator: tps65090: Hand over GPIO to regulator core
authorLinus Walleij <linus.walleij@linaro.org>
Thu, 6 Dec 2018 12:43:50 +0000 (13:43 +0100)
committerMark Brown <broonie@kernel.org>
Tue, 11 Dec 2018 01:04:56 +0000 (01:04 +0000)
The GPIO descriptors used by the TPS65090 driver are retrieved
during probe() and it is really helpful to have those under
devres management because of all the errorpaths in the
intialization.

Using the new dev_gpiod_unhinge() call we can remove the
devres management of the descriptor right before handing
it over to the regulators core.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
Reviewed-by: Marek Szyprowski <m.szyprowski@samsung.com>
Tested-by: Marek Szyprowski <m.szyprowski@samsung.com>
Reviewed-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/tps65090-regulator.c

index db714d5edafc8846285b2bf500e4eb1a4ab334aa..0614551796a19a5877d0875d1097fecfeeb998a7 100644 (file)
@@ -480,6 +480,12 @@ static int tps65090_regulator_probe(struct platform_device *pdev)
                else
                        config.of_node = NULL;
 
+               /*
+                * Hand the GPIO descriptor management over to the regulator
+                * core, remove it from devres management.
+                */
+               if (config.ena_gpiod)
+                       devm_gpiod_unhinge(&pdev->dev, config.ena_gpiod);
                rdev = devm_regulator_register(&pdev->dev, ri->desc, &config);
                if (IS_ERR(rdev)) {
                        dev_err(&pdev->dev, "failed to register regulator %s\n",