power: supply: twl4030-charger: move allocation of iio channel to the beginning
authorH. Nikolaus Schaller <hns@goldelico.com>
Wed, 14 Jun 2017 09:25:54 +0000 (11:25 +0200)
committerSebastian Reichel <sre@kernel.org>
Wed, 14 Jun 2017 20:10:44 +0000 (22:10 +0200)
This is in prepraration for EPROBE_DEFER handling because it is quite
likely that geting the (madc) iio channel is deferred more often than
later steps.

Signed-off-by: H. Nikolaus Schaller <hns@goldelico.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
drivers/power/supply/twl4030_charger.c

index 33f2415..3de802f 100644 (file)
@@ -980,6 +980,12 @@ static int twl4030_bci_probe(struct platform_device *pdev)
 
        platform_set_drvdata(pdev, bci);
 
+       bci->channel_vac = devm_iio_channel_get(&pdev->dev, "vac");
+       if (IS_ERR(bci->channel_vac)) {
+               bci->channel_vac = NULL;
+               dev_warn(&pdev->dev, "could not request vac iio channel");
+       }
+
        bci->ac = devm_power_supply_register(&pdev->dev, &twl4030_bci_ac_desc,
                                             NULL);
        if (IS_ERR(bci->ac)) {
@@ -1013,12 +1019,6 @@ static int twl4030_bci_probe(struct platform_device *pdev)
                return ret;
        }
 
-       bci->channel_vac = devm_iio_channel_get(&pdev->dev, "vac");
-       if (IS_ERR(bci->channel_vac)) {
-               bci->channel_vac = NULL;
-               dev_warn(&pdev->dev, "could not request vac iio channel");
-       }
-
        INIT_WORK(&bci->work, twl4030_bci_usb_work);
        INIT_DELAYED_WORK(&bci->current_worker, twl4030_current_worker);