[media] coda: clamp frame sequence counters to 16 bit
authorLucas Stach <l.stach@pengutronix.de>
Thu, 9 Jul 2015 10:10:12 +0000 (07:10 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Fri, 17 Jul 2015 13:50:55 +0000 (10:50 -0300)
This is already done for one side of the comparison with the expectation
that the HW counter rolls over at the 16 bit boundary. This is true when
decoding a h.264 stream, but doesn't hold for at least MJPEG. As we don't
know the exact wrap-around point for this format just clamp the HW counter
to the same 16 bits. This should be enough to detect most of the errors
and saves us from doing different comparisons based on the decoded format.

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
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>
drivers/media/platform/coda/coda-bit.c

index 109797bb8fbb417c3499be722a9401617a496d29..9fbff248d7fa214fdaca72749302658b7d504ef4 100644 (file)
@@ -1902,7 +1902,14 @@ static void coda_finish_decode(struct coda_ctx *ctx)
                        meta = list_first_entry(&ctx->buffer_meta_list,
                                              struct coda_buffer_meta, list);
                        list_del(&meta->list);
-                       if (val != (meta->sequence & 0xffff)) {
+                       /*
+                        * Clamp counters to 16 bits for comparison, as the HW
+                        * counter rolls over at this point for h.264. This
+                        * may be different for other formats, but using 16 bits
+                        * should be enough to detect most errors and saves us
+                        * from doing different things based on the format.
+                        */
+                       if ((val & 0xffff) != (meta->sequence & 0xffff)) {
                                v4l2_err(&dev->v4l2_dev,
                                         "sequence number mismatch (%d(%d) != %d)\n",
                                         val, ctx->sequence_offset,