[media] s5p-mfc: Only set timestamp/timecode for new frames
authorIlja Friedel <ihf@chromium.org>
Tue, 21 Oct 2014 11:06:58 +0000 (08:06 -0300)
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>
Tue, 28 Oct 2014 17:43:25 +0000 (15:43 -0200)
commitbb21c54af7c6d7896f218cc8e4e5d4a466b8e137
tree42095b67924bab4887e16bd7126f5fd1fe07e1db
parent643709941d152a8dee58f5ea9f9e19d3f1725722
[media] s5p-mfc: Only set timestamp/timecode for new frames

Timestamp i of a previously decoded buffer was overwritten for some
H.264 streams with timestamp i+1 of the next buffer. This happened when
encountering frame_type S5P_FIMV_DECODE_FRAME_SKIPPED, indicating no
new frame.

In most cases this wrong indexing might not have been noticed except
for a one frame delay in frame presentation. For H.264 streams though
that require reordering of frames for presentation, it caused a slightly
erratic presentation time lookup and consequently dropped frames in the
Pepper Flash plugin.

Signed-off-by: Ilja H. Friedel <ihf@google.com>
Signed-off-by: Kiran AVND <avnd.kiran@samsung.com>
Signed-off-by: Arun Kumar K <arun.kk@samsung.com>
Signed-off-by: Kamil Debski <k.debski@samsung.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
drivers/media/platform/s5p-mfc/s5p_mfc.c