[media] coda: reset CODA960 hardware after sequence end

On i.MX6, sometimes after decoding a stream, encoding will produce macroblock
errors caused by missing 8-byte sequences in the output stream. Until the cause
for this is found, reset the hardware after sequence end, which seems to help.

Signed-off-by: Philipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>

diff --git a/drivers/media/platform/coda/coda-bit.c b/drivers/media/platform/coda/coda-bit.c
index 25910ccd2da080726501c0b5f916da73a50c6d73..bcb9911d5e3f1844cb0d72e17c26d0972600622d 100644 (file)
--- a/drivers/media/platform/coda/coda-bit.c
+++ b/drivers/media/platform/coda/coda-bit.c
@@ -1347,6 +1347,14 @@ static void coda_seq_end_work(struct work_struct *work)
                         "CODA_COMMAND_SEQ_END failed\n");
        }
 
+       /*
+        * FIXME: Sometimes h.264 encoding fails with 8-byte sequences missing
+        * from the output stream after the h.264 decoder has run. Resetting the
+        * hardware after the decoder has finished seems to help.
+        */
+       if (dev->devtype->product == CODA_960)
+               coda_hw_reset(ctx);
+
        kfifo_init(&ctx->bitstream_fifo,
                ctx->bitstream.vaddr, ctx->bitstream.size);