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

V4L/DVB (9893): cx18: Convert some list manipulations to emphasize entries not lists

Signed-off-by: default avatarAndy Walls <awalls@radix.net>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab@redhat.com>
parent aafc7706
...@@ -77,8 +77,8 @@ struct cx18_buffer *cx18_dequeue(struct cx18_stream *s, struct cx18_queue *q) ...@@ -77,8 +77,8 @@ struct cx18_buffer *cx18_dequeue(struct cx18_stream *s, struct cx18_queue *q)
mutex_lock(&s->qlock); mutex_lock(&s->qlock);
if (!list_empty(&q->list)) { if (!list_empty(&q->list)) {
buf = list_entry(q->list.next, struct cx18_buffer, list); buf = list_first_entry(&q->list, struct cx18_buffer, list);
list_del_init(q->list.next); list_del_init(&buf->list);
q->bytesused -= buf->bytesused - buf->readpos; q->bytesused -= buf->bytesused - buf->readpos;
buf->skipped = 0; buf->skipped = 0;
atomic_dec(&q->buffers); atomic_dec(&q->buffers);
...@@ -92,13 +92,11 @@ struct cx18_buffer *cx18_queue_get_buf(struct cx18_stream *s, u32 id, ...@@ -92,13 +92,11 @@ struct cx18_buffer *cx18_queue_get_buf(struct cx18_stream *s, u32 id,
{ {
struct cx18 *cx = s->cx; struct cx18 *cx = s->cx;
struct cx18_buffer *buf; struct cx18_buffer *buf;
struct cx18_buffer *tmp;
struct cx18_buffer *ret = NULL; struct cx18_buffer *ret = NULL;
struct list_head *p, *t;
mutex_lock(&s->qlock); mutex_lock(&s->qlock);
list_for_each_safe(p, t, &s->q_busy.list) { list_for_each_entry_safe(buf, tmp, &s->q_busy.list, list) {
buf = list_entry(p, struct cx18_buffer, list);
if (buf->id != id) { if (buf->id != id) {
buf->skipped++; buf->skipped++;
if (buf->skipped >= atomic_read(&s->q_busy.buffers)-1) { if (buf->skipped >= atomic_read(&s->q_busy.buffers)-1) {
...@@ -152,8 +150,8 @@ static void cx18_queue_flush(struct cx18_stream *s, struct cx18_queue *q) ...@@ -152,8 +150,8 @@ static void cx18_queue_flush(struct cx18_stream *s, struct cx18_queue *q)
mutex_lock(&s->qlock); mutex_lock(&s->qlock);
while (!list_empty(&q->list)) { while (!list_empty(&q->list)) {
buf = list_entry(q->list.next, struct cx18_buffer, list); buf = list_first_entry(&q->list, struct cx18_buffer, list);
list_move_tail(q->list.next, &s->q_free.list); list_move_tail(&buf->list, &s->q_free.list);
buf->bytesused = buf->readpos = buf->b_flags = buf->skipped = 0; buf->bytesused = buf->readpos = buf->b_flags = buf->skipped = 0;
atomic_inc(&s->q_free.buffers); atomic_inc(&s->q_free.buffers);
} }
......
...@@ -442,7 +442,6 @@ int cx18_start_v4l2_encode_stream(struct cx18_stream *s) ...@@ -442,7 +442,6 @@ int cx18_start_v4l2_encode_stream(struct cx18_stream *s)
{ {
u32 data[MAX_MB_ARGUMENTS]; u32 data[MAX_MB_ARGUMENTS];
struct cx18 *cx = s->cx; struct cx18 *cx = s->cx;
struct list_head *p;
struct cx18_buffer *buf; struct cx18_buffer *buf;
int ts = 0; int ts = 0;
int captype = 0; int captype = 0;
...@@ -529,8 +528,7 @@ int cx18_start_v4l2_encode_stream(struct cx18_stream *s) ...@@ -529,8 +528,7 @@ int cx18_start_v4l2_encode_stream(struct cx18_stream *s)
/* Init all the cpu_mdls for this stream */ /* Init all the cpu_mdls for this stream */
cx18_flush_queues(s); cx18_flush_queues(s);
mutex_lock(&s->qlock); mutex_lock(&s->qlock);
list_for_each(p, &s->q_free.list) { list_for_each_entry(buf, &s->q_free.list, list) {
buf = list_entry(p, struct cx18_buffer, list);
cx18_writel(cx, buf->dma_handle, cx18_writel(cx, buf->dma_handle,
&cx->scb->cpu_mdl[buf->id].paddr); &cx->scb->cpu_mdl[buf->id].paddr);
cx18_writel(cx, s->buf_size, &cx->scb->cpu_mdl[buf->id].length); cx18_writel(cx, s->buf_size, &cx->scb->cpu_mdl[buf->id].length);
......
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