raspi/linux
1
0
Fork 1
mirror of https://github.com/raspberrypi/linux.git synced 2025-12-06 10:00:17 +00:00
Code Issues Projects Releases Wiki Activity

media: i2c: imx290: Use dev_err_probe()

Browse Source 
Improve error handling in the probe() function with dev_err_probe().

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Alexander Stein <alexander.stein@ew.tq-group.com>
Signed-off-by: Sakari Ailus <sakari.ailus@linux.intel.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
This commit is contained in:
Laurent Pinchart
2023-01-16 15:44:46 +01:00
committed by Mauro Carvalho Chehab
parent 6b69c52277
commit 63127235be
1 changed files with 8 additions and 12 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
drivers/media/i2c/imx290.c
View File

@@ -1224,10 +1224,9 @@ static int imx290_probe(struct i2c_client *client)
/* get system clock (xclk) */ /* get system clock (xclk) */
imx290->xclk = devm_clk_get(dev, "xclk"); imx290->xclk = devm_clk_get(dev, "xclk");
if (IS_ERR(imx290->xclk)) { if (IS_ERR(imx290->xclk))
dev_err(dev, "Could not get xclk"); return dev_err_probe(dev, PTR_ERR(imx290->xclk),
return PTR_ERR(imx290->xclk); "Could not get xclk");
}
ret = fwnode_property_read_u32(dev_fwnode(dev), "clock-frequency", ret = fwnode_property_read_u32(dev_fwnode(dev), "clock-frequency",
&xclk_freq); &xclk_freq);
@@ -1250,17 +1249,14 @@ static int imx290_probe(struct i2c_client *client)
} }
ret = imx290_get_regulators(dev, imx290); ret = imx290_get_regulators(dev, imx290);
if (ret < 0) { if (ret < 0)
dev_err(dev, "Cannot get regulators\n"); return dev_err_probe(dev, ret, "Cannot get regulators\n");
return ret;
}
imx290->rst_gpio = devm_gpiod_get_optional(dev, "reset", imx290->rst_gpio = devm_gpiod_get_optional(dev, "reset",
GPIOD_OUT_HIGH); GPIOD_OUT_HIGH);
if (IS_ERR(imx290->rst_gpio)) { if (IS_ERR(imx290->rst_gpio))
dev_err(dev, "Cannot get reset gpio\n"); return dev_err_probe(dev, PTR_ERR(imx290->rst_gpio),
return PTR_ERR(imx290->rst_gpio); "Cannot get reset gpio\n");
}
mutex_init(&imx290->lock); mutex_init(&imx290->lock);