V4L/DVB (9893): cx18: Convert some list manipulations to emphasize entries not lists
authorAndy Walls <awalls@radix.net>
Mon, 15 Dec 2008 00:05:36 +0000 (21:05 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 30 Dec 2008 11:39:08 +0000 (09:39 -0200)
Signed-off-by: Andy Walls <awalls@radix.net>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/cx18/cx18-queue.c
drivers/media/video/cx18/cx18-streams.c

index 2c58b0c478c0e2861065112a819aef777173e2d8..8d9441e88c4ef4f2a62b0a564576c20f78f6560d 100644 (file)
@@ -77,8 +77,8 @@ struct cx18_buffer *cx18_dequeue(struct cx18_stream *s, struct cx18_queue *q)
 
        mutex_lock(&s->qlock);
        if (!list_empty(&q->list)) {
-               buf = list_entry(q->list.next, struct cx18_buffer, list);
-               list_del_init(q->list.next);
+               buf = list_first_entry(&q->list, struct cx18_buffer, list);
+               list_del_init(&buf->list);
                q->bytesused -= buf->bytesused - buf->readpos;
                buf->skipped = 0;
                atomic_dec(&q->buffers);
@@ -92,13 +92,11 @@ struct cx18_buffer *cx18_queue_get_buf(struct cx18_stream *s, u32 id,
 {
        struct cx18 *cx = s->cx;
        struct cx18_buffer *buf;
+       struct cx18_buffer *tmp;
        struct cx18_buffer *ret = NULL;
-       struct list_head *p, *t;
 
        mutex_lock(&s->qlock);
-       list_for_each_safe(p, t, &s->q_busy.list) {
-               buf = list_entry(p, struct cx18_buffer, list);
-
+       list_for_each_entry_safe(buf, tmp, &s->q_busy.list, list) {
                if (buf->id != id) {
                        buf->skipped++;
                        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)
 
        mutex_lock(&s->qlock);
        while (!list_empty(&q->list)) {
-               buf = list_entry(q->list.next, struct cx18_buffer, list);
-               list_move_tail(q->list.next, &s->q_free.list);
+               buf = list_first_entry(&q->list, struct cx18_buffer, list);
+               list_move_tail(&buf->list, &s->q_free.list);
                buf->bytesused = buf->readpos = buf->b_flags = buf->skipped = 0;
                atomic_inc(&s->q_free.buffers);
        }
index 8fc655d3ced97074af8cb4a9758387f0fa96d97a..600a5fcac3d0dcc178bb18338bd114d1f10d2dc7 100644 (file)
@@ -442,7 +442,6 @@ int cx18_start_v4l2_encode_stream(struct cx18_stream *s)
 {
        u32 data[MAX_MB_ARGUMENTS];
        struct cx18 *cx = s->cx;
-       struct list_head *p;
        struct cx18_buffer *buf;
        int ts = 0;
        int captype = 0;
@@ -529,8 +528,7 @@ int cx18_start_v4l2_encode_stream(struct cx18_stream *s)
        /* Init all the cpu_mdls for this stream */
        cx18_flush_queues(s);
        mutex_lock(&s->qlock);
-       list_for_each(p, &s->q_free.list) {
-               buf = list_entry(p, struct cx18_buffer, list);
+       list_for_each_entry(buf, &s->q_free.list, list) {
                cx18_writel(cx, buf->dma_handle,
                                        &cx->scb->cpu_mdl[buf->id].paddr);
                cx18_writel(cx, s->buf_size, &cx->scb->cpu_mdl[buf->id].length);