Commit 1047a838 authored by Andy Walls's avatar Andy Walls Committed by Mauro Carvalho Chehab

V4L/DVB (13431): cx18: Adjust an MDL's final buffer size to force encoder transfer size

The encoder was not honoring the MDL size sent in DE_SET_MDL mailbox commands.
This change adjusts the size of the last buffer in an MDL, as reported to the
firmware, so that the encoder will send the exact amount of bytes we specify
per MDL transfer.  This eliminates tearing in YUV playback when using
non-default YUV buffer sizes.
Signed-off-by: default avatarAndy Walls <awalls@radix.net>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent 22dce188
...@@ -280,6 +280,7 @@ void cx18_load_queues(struct cx18_stream *s) ...@@ -280,6 +280,7 @@ void cx18_load_queues(struct cx18_stream *s)
struct cx18_buffer *buf; struct cx18_buffer *buf;
int mdl_id; int mdl_id;
int i; int i;
u32 partial_buf_size;
/* /*
* Attach buffers to MDLs, give the MDLs ids, and add MDLs to q_free * Attach buffers to MDLs, give the MDLs ids, and add MDLs to q_free
...@@ -308,10 +309,24 @@ void cx18_load_queues(struct cx18_stream *s) ...@@ -308,10 +309,24 @@ void cx18_load_queues(struct cx18_stream *s)
&cx->scb->cpu_mdl[mdl_id + i].length); &cx->scb->cpu_mdl[mdl_id + i].length);
} }
if (i == s->bufs_per_mdl) if (i == s->bufs_per_mdl) {
/*
* The encoder doesn't honor s->mdl_size. So in the
* case of a non-integral number of buffers to meet
* mdl_size, we lie about the size of the last buffer
* in the MDL to get the encoder to really only send
* us mdl_size bytes per MDL transfer.
*/
partial_buf_size = s->mdl_size % s->buf_size;
if (partial_buf_size) {
cx18_writel(cx, partial_buf_size,
&cx->scb->cpu_mdl[mdl_id + i - 1].length);
}
cx18_enqueue(s, mdl, &s->q_free); cx18_enqueue(s, mdl, &s->q_free);
else } else {
cx18_push(s, mdl, &s->q_idle); /* not enough buffers */ /* Not enough buffers for this MDL; we won't use it */
cx18_push(s, mdl, &s->q_idle);
}
mdl_id += i; mdl_id += i;
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment