diff options
| author | Michael Tretter <m.tretter@pengutronix.de> | 2020-03-16 16:26:22 +0100 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab+huawei@kernel.org> | 2020-03-20 09:08:09 +0100 |
| commit | b646f99642b7499b469eaff25829a0b91a69692a (patch) | |
| tree | 8667c6709b15b2c78c25a363016c45d4261f59bd | |
| parent | 04d2a1101871030e0b80ab3bfd97d407d7804569 (diff) | |
media: allegro: fail encoding only on actual errors
Only negative values are actual errors and positive values are used for
warnings. Warnings should not fail the encoding process.
Signed-off-by: Michael Tretter <m.tretter@pengutronix.de>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
| -rw-r--r-- | drivers/staging/media/allegro-dvt/allegro-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/staging/media/allegro-dvt/allegro-core.c b/drivers/staging/media/allegro-dvt/allegro-core.c index 9e601a602ac9..56d199c9312e 100644 --- a/drivers/staging/media/allegro-dvt/allegro-core.c +++ b/drivers/staging/media/allegro-dvt/allegro-core.c @@ -1536,7 +1536,7 @@ static void allegro_channel_finish_frame(struct allegro_channel *channel, dst_buf = v4l2_m2m_dst_buf_remove(channel->fh.m2m_ctx); dst_buf->sequence = channel->csequence++; - if (msg->error_code) { + if (msg->error_code & AL_ERROR) { v4l2_err(&dev->v4l2_dev, "channel %d: failed to encode frame: %s (%x)\n", channel->mcu_channel_id, |