[media] s5c73m3: Use devm_regulator_bulk_get API
authorSachin Kamat <sachin.kamat@linaro.org>
Thu, 31 Jan 2013 04:12:46 +0000 (01:12 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 5 Feb 2013 19:54:22 +0000 (17:54 -0200)
devm_regulator_bulk_get saves some cleanup and exit code.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Sylwester Nawrocki <s.nawrocki@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/i2c/s5c73m3/s5c73m3-core.c

index b063b4d..c143c9e 100644 (file)
@@ -1627,7 +1627,7 @@ static int __devinit s5c73m3_probe(struct i2c_client *client,
        for (i = 0; i < S5C73M3_MAX_SUPPLIES; i++)
                state->supplies[i].supply = s5c73m3_supply_names[i];
 
-       ret = regulator_bulk_get(dev, S5C73M3_MAX_SUPPLIES,
+       ret = devm_regulator_bulk_get(dev, S5C73M3_MAX_SUPPLIES,
                               state->supplies);
        if (ret) {
                dev_err(dev, "failed to get regulators\n");
@@ -1636,7 +1636,7 @@ static int __devinit s5c73m3_probe(struct i2c_client *client,
 
        ret = s5c73m3_init_controls(state);
        if (ret)
-               goto out_err3;
+               goto out_err2;
 
        state->sensor_pix_size[RES_ISP] = &s5c73m3_isp_resolutions[1];
        state->sensor_pix_size[RES_JPEG] = &s5c73m3_jpeg_resolutions[1];
@@ -1652,15 +1652,13 @@ static int __devinit s5c73m3_probe(struct i2c_client *client,
 
        ret = s5c73m3_register_spi_driver(state);
        if (ret < 0)
-               goto out_err3;
+               goto out_err2;
 
        state->i2c_client = client;
 
        v4l2_info(sd, "%s: completed succesfully\n", __func__);
        return 0;
 
-out_err3:
-       regulator_bulk_free(S5C73M3_MAX_SUPPLIES, state->supplies);
 out_err2:
        s5c73m3_free_gpios(state);
 out_err1:
@@ -1679,7 +1677,6 @@ static int __devexit s5c73m3_remove(struct i2c_client *client)
        media_entity_cleanup(&sd->entity);
 
        s5c73m3_unregister_spi_driver(state);
-       regulator_bulk_free(S5C73M3_MAX_SUPPLIES, state->supplies);
        s5c73m3_free_gpios(state);
 
        return 0;