[media] s5c73m3: Use devm_regulator_bulk_get API
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>
This commit is contained in:
parent
0e23cbbe47
commit
33fba5de1e
|
@ -1627,7 +1627,7 @@ static int __devinit s5c73m3_probe(struct i2c_client *client,
|
||||||
for (i = 0; i < S5C73M3_MAX_SUPPLIES; i++)
|
for (i = 0; i < S5C73M3_MAX_SUPPLIES; i++)
|
||||||
state->supplies[i].supply = s5c73m3_supply_names[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);
|
state->supplies);
|
||||||
if (ret) {
|
if (ret) {
|
||||||
dev_err(dev, "failed to get regulators\n");
|
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);
|
ret = s5c73m3_init_controls(state);
|
||||||
if (ret)
|
if (ret)
|
||||||
goto out_err3;
|
goto out_err2;
|
||||||
|
|
||||||
state->sensor_pix_size[RES_ISP] = &s5c73m3_isp_resolutions[1];
|
state->sensor_pix_size[RES_ISP] = &s5c73m3_isp_resolutions[1];
|
||||||
state->sensor_pix_size[RES_JPEG] = &s5c73m3_jpeg_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);
|
ret = s5c73m3_register_spi_driver(state);
|
||||||
if (ret < 0)
|
if (ret < 0)
|
||||||
goto out_err3;
|
goto out_err2;
|
||||||
|
|
||||||
state->i2c_client = client;
|
state->i2c_client = client;
|
||||||
|
|
||||||
v4l2_info(sd, "%s: completed succesfully\n", __func__);
|
v4l2_info(sd, "%s: completed succesfully\n", __func__);
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
out_err3:
|
|
||||||
regulator_bulk_free(S5C73M3_MAX_SUPPLIES, state->supplies);
|
|
||||||
out_err2:
|
out_err2:
|
||||||
s5c73m3_free_gpios(state);
|
s5c73m3_free_gpios(state);
|
||||||
out_err1:
|
out_err1:
|
||||||
|
@ -1679,7 +1677,6 @@ static int __devexit s5c73m3_remove(struct i2c_client *client)
|
||||||
media_entity_cleanup(&sd->entity);
|
media_entity_cleanup(&sd->entity);
|
||||||
|
|
||||||
s5c73m3_unregister_spi_driver(state);
|
s5c73m3_unregister_spi_driver(state);
|
||||||
regulator_bulk_free(S5C73M3_MAX_SUPPLIES, state->supplies);
|
|
||||||
s5c73m3_free_gpios(state);
|
s5c73m3_free_gpios(state);
|
||||||
|
|
||||||
return 0;
|
return 0;
|
||||||
|
|
Loading…
Reference in New Issue