mirror of
https://github.com/linux-msm/laptops-kernel.git
synced 2026-08-13 14:19:53 -07:00
media: i2c: imx274: Use %pe format specifier
The %pe format specifier is designed to print error pointers. It prints a symbolic error name (eg. -EINVAL) and it makes the code simpler by omitting PTR_ERR(). This patch fixes this cocci report: ./i2c/imx274.c:2038:32-39: WARNING: Consider using %pe to print PTR_ERR() Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com> Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
This commit is contained in:
committed by
Hans Verkuil
parent
e1fa799310
commit
c268a1e9af
@@ -2034,8 +2034,7 @@ static int imx274_probe(struct i2c_client *client)
|
||||
/* initialize regmap */
|
||||
imx274->regmap = devm_regmap_init_i2c(client, &imx274_regmap_config);
|
||||
if (IS_ERR(imx274->regmap)) {
|
||||
dev_err(dev,
|
||||
"regmap init failed: %ld\n", PTR_ERR(imx274->regmap));
|
||||
dev_err(dev, "regmap init failed: %pe\n", imx274->regmap);
|
||||
ret = -ENODEV;
|
||||
goto err_regmap;
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user