Skip to content

Commit bc53bc3

Browse files
kbinghamjailuthra
authored andcommitted
media: imx335: Use dev_err_probe where appropriate
In the early probe we can fail with -EPROBE_DEFER. Use dev_err_probe to ignore these warnings before we successfully power on the camera. Signed-off-by: Kieran Bingham <[email protected]> Signed-off-by: Jai Luthra <[email protected]>
1 parent 1d4a142 commit bc53bc3

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

drivers/media/i2c/imx335.c

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1455,26 +1455,23 @@ static int imx335_probe(struct i2c_client *client)
14551455

14561456
imx335->dev = &client->dev;
14571457
imx335->cci = devm_cci_regmap_init_i2c(client, 16);
1458-
if (IS_ERR(imx335->cci)) {
1459-
dev_err(imx335->dev, "Unable to initialize I2C\n");
1460-
return -ENODEV;
1461-
}
1458+
if (IS_ERR(imx335->cci))
1459+
return dev_err_probe(imx335->dev, PTR_ERR(imx335->cci),
1460+
"Unable to initialize I2C\n");
14621461

14631462
/* Initialize subdev */
14641463
v4l2_i2c_subdev_init(&imx335->sd, client, &imx335_subdev_ops);
14651464
imx335->sd.internal_ops = &imx335_internal_ops;
14661465

14671466
ret = imx335_parse_hw_config(imx335);
1468-
if (ret) {
1469-
dev_err(imx335->dev, "HW configuration is not supported\n");
1470-
return ret;
1471-
}
1467+
if (ret)
1468+
return dev_err_probe(imx335->dev, ret,
1469+
"HW configuration is not supported\n");
14721470

14731471
ret = imx335_power_on(imx335->dev);
1474-
if (ret) {
1475-
dev_err(imx335->dev, "failed to power-on the sensor\n");
1476-
return ret;
1477-
}
1472+
if (ret)
1473+
return dev_err_probe(imx335->dev, ret,
1474+
"failed to power-on the sensor\n");
14781475

14791476
/* Check module identity */
14801477
ret = imx335_detect(imx335);

0 commit comments

Comments
 (0)