media: ti-vpe: cal: Use list_first_entry()

Use the list_first_entry() macro where appropriate to replace manual
usage of list_entry(head.next).

Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
Reviewed-by: Benoit Parrot <bparrot@ti.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>

diff --git a/drivers/media/platform/ti-vpe/cal-video.c b/drivers/media/platform/ti-vpe/cal-video.c
index bfc3ba4..c80c464 100644 (file)
--- a/drivers/media/platform/ti-vpe/cal-video.c
+++ b/drivers/media/platform/ti-vpe/cal-video.c
@@ -511,7 +511,7 @@ static int cal_start_streaming(struct vb2_queue *vq, unsigned int count)
        int ret;
 
        spin_lock_irq(&ctx->slock);
-       buf = list_entry(dma_q->active.next, struct cal_buffer, list);
+       buf = list_first_entry(&dma_q->active, struct cal_buffer, list);
        ctx->cur_frm = buf;
        ctx->next_frm = buf;
        list_del(&buf->list);

diff --git a/drivers/media/platform/ti-vpe/cal.c b/drivers/media/platform/ti-vpe/cal.c
index f1c2b8b..8f25e7a 100644 (file)
--- a/drivers/media/platform/ti-vpe/cal.c
+++ b/drivers/media/platform/ti-vpe/cal.c
@@ -496,7 +496,7 @@ static inline void cal_irq_wdma_start(struct cal_ctx *ctx)
                struct cal_buffer *buf;
                unsigned long addr;
 
-               buf = list_entry(dma_q->active.next, struct cal_buffer, list);
+               buf = list_first_entry(&dma_q->active, struct cal_buffer, list);
                addr = vb2_dma_contig_plane_dma_addr(&buf->vb.vb2_buf, 0);
                cal_ctx_wr_dma_addr(ctx, addr);