[COMMON] media: mfc: DRV4.0: clean up duplicate functions
authorSunyoung Kang <sy0816.kang@samsung.com>
Mon, 2 Jul 2018 23:14:14 +0000 (08:14 +0900)
committerSunyoung Kang <sy0816.kang@samsung.com>
Mon, 23 Jul 2018 06:18:57 +0000 (15:18 +0900)
This removes mfc_move_first_buf_used() and replaces
mfc_get_move_buf_used().

Change-Id: Ia704c8917cd6a1552dff6851e0904329c9b23aa8
Signed-off-by: Sunyoung Kang <sy0816.kang@samsung.com>
drivers/media/platform/exynos/mfc/mfc_isr.c
drivers/media/platform/exynos/mfc/mfc_queue.c
drivers/media/platform/exynos/mfc/mfc_queue.h

index 22ba05c4aa9ddfb1ef68ae530a23eadd32209737..25d7dc69d9b0492e56ca4d3ff028b0de541042f3 100644 (file)
@@ -849,8 +849,8 @@ static void __mfc_handle_stream_input(struct mfc_ctx *ctx)
 move_buf:
        /* move enqueued src buffer: src queue -> ref queue */
        if (!found_in_src_queue && ctx->state != MFCINST_FINISHING) {
-               mfc_move_first_buf_used(&ctx->buf_queue_lock,
-                               &ctx->ref_buf_queue, &ctx->src_buf_queue, MFC_QUEUE_ADD_BOTTOM);
+               mfc_get_move_buf_used(&ctx->buf_queue_lock,
+                               &ctx->ref_buf_queue, &ctx->src_buf_queue);
 
                mfc_debug(2, "enc src_buf_queue(%d) -> ref_buf_queue(%d)\n",
                                mfc_get_queue_count(&ctx->buf_queue_lock, &ctx->src_buf_queue),
index e4ee0dad230d26ffd27c1d19eceb5d5c7071fcb1..2b07ba7f9d63087c9b5e2ce68ad8b1181fa18f67 100644 (file)
@@ -435,36 +435,6 @@ struct mfc_buf *mfc_find_move_buf_used(spinlock_t *plock,
        }
 }
 
-void mfc_move_first_buf_used(spinlock_t *plock, struct mfc_buf_queue *to_queue,
-               struct mfc_buf_queue *from_queue, enum mfc_queue_top_type top)
-{
-       unsigned long flags;
-       struct mfc_buf *mfc_buf = NULL;
-
-       spin_lock_irqsave(plock, flags);
-
-       if (list_empty(&from_queue->head)) {
-               mfc_err_dev("from_queue is empty\n");
-               spin_unlock_irqrestore(plock, flags);
-               return;
-       }
-       mfc_buf = list_entry(from_queue->head.next, struct mfc_buf, list);
-
-       if (mfc_buf->used) {
-               list_del(&mfc_buf->list);
-               from_queue->count--;
-
-               if (top == MFC_QUEUE_ADD_TOP)
-                       list_add(&mfc_buf->list, &to_queue->head);
-               else
-                       list_add_tail(&mfc_buf->list, &to_queue->head);
-
-               to_queue->count++;
-       }
-
-       spin_unlock_irqrestore(plock, flags);
-}
-
 void mfc_move_all_bufs(spinlock_t *plock, struct mfc_buf_queue *to_queue,
                struct mfc_buf_queue *from_queue, enum mfc_queue_top_type top)
 {
index 392d0b38185e416a1eb5fdd3f6852ce5f4def4aa..42330f3d57ea3ed6c49574ba255176e54e107c7c 100644 (file)
@@ -133,8 +133,6 @@ struct mfc_buf *mfc_find_move_buf_used(spinlock_t *plock,
                struct mfc_buf_queue *to_queue, struct mfc_buf_queue *from_queue,
                dma_addr_t addr);
 
-void mfc_move_first_buf_used(spinlock_t *plock, struct mfc_buf_queue *to_queue,
-               struct mfc_buf_queue *from_queue, enum mfc_queue_top_type top);
 void mfc_move_all_bufs(spinlock_t *plock, struct mfc_buf_queue *to_queue,
                struct mfc_buf_queue *from_queue, enum mfc_queue_top_type top);