mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2025-12-28 06:44:36 -05:00
media: imx8mq-mipi-csi2: 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: ./platform/nxp/imx8mq-mipi-csi2.c:422:23-30: WARNING: Consider using %pe to print PTR_ERR() Signed-off-by: Ricardo Ribalda <ribalda@chromium.org> Reviewed-by: Martin Kepplinger-Novakovic <martink@posteo.de> 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
610a0fcd80
commit
53d94ec2ea
@@ -418,8 +418,8 @@ static int imx8mq_mipi_csi_calc_hs_settle(struct csi_state *state,
|
||||
|
||||
src_pad = media_entity_remote_source_pad_unique(&sd_state->sd->entity);
|
||||
if (IS_ERR(src_pad)) {
|
||||
dev_err(state->dev, "can't get source pad of %s (%ld)\n",
|
||||
sd_state->sd->name, PTR_ERR(src_pad));
|
||||
dev_err(state->dev, "can't get source pad of %s (%pe)\n",
|
||||
sd_state->sd->name, src_pad);
|
||||
return PTR_ERR(src_pad);
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user