On Mon Jun 16 00:58:01 2025 +0100, Dr. David Alan Gilbert wrote:
> vpdma_update_dma_addr() was added in 2016 as part of
> commit 2f88703a0bfd ("[media] media: ti-vpe: vpdma: Add multi-instance and
> multi-client support")
> 
> but has remained unused.
> 
> Remove it.
> 
> I did see that there was a VIP driver submitted in 2020 that
> doesn't seem to have got merged which did use this (and a bunch
> of other unused functions).
> 
> Signed-off-by: Dr. David Alan Gilbert <li...@treblig.org>
> Signed-off-by: Hans Verkuil <hverk...@xs4all.nl>

Patch committed.

Thanks,
Hans Verkuil

 drivers/media/platform/ti/vpe/vpdma.c | 32 --------------------------------
 drivers/media/platform/ti/vpe/vpdma.h |  3 ---
 2 files changed, 35 deletions(-)

---

diff --git a/drivers/media/platform/ti/vpe/vpdma.c 
b/drivers/media/platform/ti/vpe/vpdma.c
index da90d7f03f82..bb8a8bd7980c 100644
--- a/drivers/media/platform/ti/vpe/vpdma.c
+++ b/drivers/media/platform/ti/vpe/vpdma.c
@@ -552,38 +552,6 @@ EXPORT_SYMBOL(vpdma_submit_descs);
 
 static void dump_dtd(struct vpdma_dtd *dtd);
 
-void vpdma_update_dma_addr(struct vpdma_data *vpdma,
-       struct vpdma_desc_list *list, dma_addr_t dma_addr,
-       void *write_dtd, int drop, int idx)
-{
-       struct vpdma_dtd *dtd = list->buf.addr;
-       dma_addr_t write_desc_addr;
-       int offset;
-
-       dtd += idx;
-       vpdma_unmap_desc_buf(vpdma, &list->buf);
-
-       dtd->start_addr = dma_addr;
-
-       /* Calculate write address from the offset of write_dtd from start
-        * of the list->buf
-        */
-       offset = (void *)write_dtd - list->buf.addr;
-       write_desc_addr = list->buf.dma_addr + offset;
-
-       if (drop)
-               dtd->desc_write_addr = dtd_desc_write_addr(write_desc_addr,
-                                                          1, 1, 0);
-       else
-               dtd->desc_write_addr = dtd_desc_write_addr(write_desc_addr,
-                                                          1, 0, 0);
-
-       vpdma_map_desc_buf(vpdma, &list->buf);
-
-       dump_dtd(dtd);
-}
-EXPORT_SYMBOL(vpdma_update_dma_addr);
-
 void vpdma_set_max_size(struct vpdma_data *vpdma, int reg_addr,
                        u32 width, u32 height)
 {
diff --git a/drivers/media/platform/ti/vpe/vpdma.h 
b/drivers/media/platform/ti/vpe/vpdma.h
index 393fcbb3cb40..e4d7941c6207 100644
--- a/drivers/media/platform/ti/vpe/vpdma.h
+++ b/drivers/media/platform/ti/vpe/vpdma.h
@@ -222,9 +222,6 @@ void vpdma_free_desc_list(struct vpdma_desc_list *list);
 int vpdma_submit_descs(struct vpdma_data *vpdma, struct vpdma_desc_list *list,
                       int list_num);
 bool vpdma_list_busy(struct vpdma_data *vpdma, int list_num);
-void vpdma_update_dma_addr(struct vpdma_data *vpdma,
-       struct vpdma_desc_list *list, dma_addr_t dma_addr,
-       void *write_dtd, int drop, int idx);
 
 /* VPDMA hardware list funcs */
 int vpdma_hwlist_alloc(struct vpdma_data *vpdma, void *priv);

Reply via email to