[PATCH 6/6] media: staging: rkisp1: cap: add warning when link validation fails
Dafna Hirschfeld
dafna.hirschfeld at collabora.com
Fri Oct 2 14:42:22 EDT 2020
The link validation between the capture and resizer might
fail because of various reasons. Add an informative warning
to make it easier to debug.
Signed-off-by: Dafna Hirschfeld <dafna.hirschfeld at collabora.com>
---
drivers/staging/media/rkisp1/rkisp1-capture.c | 10 +++++++++-
1 file changed, 9 insertions(+), 1 deletion(-)
diff --git a/drivers/staging/media/rkisp1/rkisp1-capture.c b/drivers/staging/media/rkisp1/rkisp1-capture.c
index 9b4a12e13f13..75321280ebf0 100644
--- a/drivers/staging/media/rkisp1/rkisp1-capture.c
+++ b/drivers/staging/media/rkisp1/rkisp1-capture.c
@@ -1269,8 +1269,16 @@ static int rkisp1_capture_link_validate(struct media_link *link)
if (sd_fmt.format.height != cap->pix.fmt.height ||
sd_fmt.format.width != cap->pix.fmt.width ||
- sd_fmt.format.code != fmt->mbus)
+ sd_fmt.format.code != fmt->mbus) {
+ dev_warn(cap->rkisp1->dev,
+ "%s: failed for '%s' sd:cap: height %u:%u, width %u:%u mbus: 0x%x:0x%x\n",
+ __func__,
+ cap->vnode.vdev.name,
+ sd_fmt.format.height, cap->pix.fmt.height,
+ sd_fmt.format.width, cap->pix.fmt.width,
+ sd_fmt.format.code, fmt->mbus);
return -EPIPE;
+ }
return 0;
}
--
2.17.1
More information about the Linux-rockchip
mailing list