[git:media_stage/master] media: intel/ipu6: Don't print user-triggerable errors to kernel log

[Hans Verkuil]

This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: intel/ipu6: Don't print user-triggerable errors to kernel log
Author:  Sakari Ailus <sakari.ai...@linux.intel.com>
Date:    Thu May 2 18:21:30 2024 +0300

Use dev_dbg() for printing messages on user-triggerable conditions that
have no relation to driver or hardware issues.

Fixes: 3c1dfb5a69cf ("media: intel/ipu6: input system video nodes and buffer 
queues")
Signed-off-by: Sakari Ailus <sakari.ai...@linux.intel.com>
Reviewed-by: Bingbu Cao <bingbu....@intel.com>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>

 drivers/media/pci/intel/ipu6/ipu6-isys-queue.c | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

---

diff --git a/drivers/media/pci/intel/ipu6/ipu6-isys-queue.c 
b/drivers/media/pci/intel/ipu6/ipu6-isys-queue.c
index 11cd2c977abe..40a8ebfcfce2 100644
--- a/drivers/media/pci/intel/ipu6/ipu6-isys-queue.c
+++ b/drivers/media/pci/intel/ipu6/ipu6-isys-queue.c
@@ -34,7 +34,7 @@ static int queue_setup(struct vb2_queue *q, unsigned int 
*num_buffers,
        if (!*num_planes) {
                sizes[0] = size;
        } else if (sizes[0] < size) {
-               dev_err(dev, "%s: queue setup: size %u < %u\n",
+               dev_dbg(dev, "%s: queue setup: size %u < %u\n",
                        av->vdev.name, sizes[0], size);
                return -EINVAL;
        }
@@ -359,7 +359,7 @@ static void buf_queue(struct vb2_buffer *vb)
         */
        ret = buffer_list_get(stream, &bl);
        if (ret < 0) {
-               dev_warn(dev, "No buffers available\n");
+               dev_dbg(dev, "No buffers available\n");
                goto out;
        }
 
@@ -426,7 +426,7 @@ static int ipu6_isys_link_fmt_validate(struct 
ipu6_isys_queue *aq)
 
        if (format.width != ipu6_isys_get_frame_width(av) ||
            format.height != ipu6_isys_get_frame_height(av)) {
-               dev_err(dev, "wrong width or height %ux%u (%ux%u expected)\n",
+               dev_dbg(dev, "wrong width or height %ux%u (%ux%u expected)\n",
                        ipu6_isys_get_frame_width(av),
                        ipu6_isys_get_frame_height(av), format.width,
                        format.height);
@@ -521,13 +521,13 @@ static int start_streaming(struct vb2_queue *q, unsigned 
int count)
 
        ret = ipu6_isys_setup_video(av, &source_entity, &nr_queues);
        if (ret < 0) {
-               dev_err(dev, "failed to setup video\n");
+               dev_dbg(dev, "failed to setup video\n");
                goto out_return_buffers;
        }
 
        ret = ipu6_isys_link_fmt_validate(aq);
        if (ret) {
-               dev_err(dev,
+               dev_dbg(dev,
                        "%s: link format validation failed (%d)\n",
                        av->vdev.name, ret);
                goto out_pipeline_stop;