From 5500a4aee62998ca858a35f4749f9a79bb7f911a Mon Sep 17 00:00:00 2001 From: Shubham Chandra Date: Fri, 27 Jun 2025 21:54:39 +0000 Subject: [PATCH] media: camera: modify error log to info log v4l2-compliance has a negative test for image size, resulting in an error log. Modify this error log to info to avoid reporting kernel error/warning failure. Bug 5143480 Change-Id: I5ec459f97c5b09e130293b59c19c1eb3ea9a53ee Signed-off-by: Shubham Chandra Reviewed-on: https://git-master.nvidia.com/r/c/linux-nv-oot/+/3396425 Reviewed-by: Ankur Pawar Reviewed-by: Bitan Biswas Reviewed-by: svcacv Reviewed-by: Nithin Varghese GVS: buildbot_gerritrpt --- drivers/media/platform/tegra/camera/vi/channel.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/tegra/camera/vi/channel.c b/drivers/media/platform/tegra/camera/vi/channel.c index 71135185..c5d149bc 100644 --- a/drivers/media/platform/tegra/camera/vi/channel.c +++ b/drivers/media/platform/tegra/camera/vi/channel.c @@ -761,11 +761,12 @@ tegra_channel_queue_setup(struct vb2_queue *vq, * of the requested image size. Although this did not harm the * flow, according to "v4l2-compliance", we need to check if * the requested size is invalid. + * Printing this error as info, to avoid kernel error/warning failure. */ if (*nplanes) { if (sizes[0] < chan->format.sizeimage) { - dev_err(chan->vi->dev, "sizes[0] = %d chan->format.sizeimage = %d ...\n" - , sizes[0], chan->format.sizeimage); + dev_info(chan->vi->dev, "sizes[0] = %d chan->format.sizeimage = %d, for num_planes = %d ...\n" + , sizes[0], chan->format.sizeimage, *nplanes); return -EINVAL; } } else {