mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
synced 2026-02-26 08:46:02 -05:00
media: imx-jpeg: Remove some redundant error logs
If the picture size parsed by decoder is different from those previously
established, it's a normal flow of dynamic resolution change, not an
error case, the log may mislead that some error occurs in decoding, so
remove the error log in this case.
Fixes: 2db16c6ed7 ("media: imx-jpeg: Add V4L2 driver for i.MX8 JPEG Encoder/Decoder")
Signed-off-by: Ming Qian <ming.qian@nxp.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
This commit is contained in:
@@ -1840,17 +1840,6 @@ static int mxc_jpeg_parse(struct mxc_jpeg_ctx *ctx, struct vb2_buffer *vb)
|
||||
|
||||
q_data_out = mxc_jpeg_get_q_data(ctx,
|
||||
V4L2_BUF_TYPE_VIDEO_OUTPUT_MPLANE);
|
||||
if (q_data_out->w == 0 && q_data_out->h == 0) {
|
||||
dev_warn(dev, "Invalid user resolution 0x0");
|
||||
dev_warn(dev, "Keeping resolution from JPEG: %dx%d",
|
||||
header.frame.width, header.frame.height);
|
||||
} else if (header.frame.width != q_data_out->w ||
|
||||
header.frame.height != q_data_out->h) {
|
||||
dev_err(dev,
|
||||
"Resolution mismatch: %dx%d (JPEG) versus %dx%d(user)",
|
||||
header.frame.width, header.frame.height,
|
||||
q_data_out->w, q_data_out->h);
|
||||
}
|
||||
q_data_out->w = header.frame.width;
|
||||
q_data_out->h = header.frame.height;
|
||||
if (header.frame.width > MXC_JPEG_MAX_WIDTH ||
|
||||
|
||||
Reference in New Issue
Block a user