Skip to content

Commit 14326eb

Browse files
ribaldawenmliu
authored andcommitted
UPSTREAM: media: i2c: imx412: 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/imx412.c:931:3-10: WARNING: Consider using %pe to print PTR_ERR() Link: https://lore.kernel.org/all/20251013-ptr_err-v1-11-2c5efbd82952@chromium.org/ Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> Acked-by: Sakari Ailus <sakari.ailus@linux.intel.com>
1 parent c4d35c7 commit 14326eb

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

drivers/media/i2c/imx412.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -927,8 +927,8 @@ static int imx412_parse_hw_config(struct imx412 *imx412)
927927
imx412->reset_gpio = devm_gpiod_get_optional(imx412->dev, "reset",
928928
GPIOD_OUT_LOW);
929929
if (IS_ERR(imx412->reset_gpio)) {
930-
dev_err(imx412->dev, "failed to get reset gpio %ld\n",
931-
PTR_ERR(imx412->reset_gpio));
930+
dev_err(imx412->dev, "failed to get reset gpio %pe\n",
931+
imx412->reset_gpio);
932932
return PTR_ERR(imx412->reset_gpio);
933933
}
934934

0 commit comments

Comments
 (0)