remove videobuf_set_pci_ops

From: Mauro Carvalho Chehab <mchehab@infradead.org>

Before the videobuf redesign, a procedure for re-using videobuf without PCI
scatter/gather where provided by changing the pci-dependent operations by
other operations.

With the newer approach, those methods are obsolete and can safelly be removed.

Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>

3 files changed, 8 insertions, 89 deletions

diff --git a/linux/drivers/media/video/videobuf-dma-sg.c b/linux/drivers/media/video/videobuf-dma-sg.c
index 33cd2830e..294ff62bb 100644
--- a/linux/drivers/media/video/videobuf-dma-sg.c
+++ b/linux/drivers/media/video/videobuf-dma-sg.c
@@ -221,7 +221,6 @@ int videobuf_dma_init_overlay(struct videobuf_dmabuf *dma, int direction,
 int videobuf_dma_map(struct videobuf_queue* q,struct videobuf_dmabuf *dma)
 {
 	void                   *dev=q->dev;
-	struct videobuf_dma_sg_ops *ops=q->priv_ops;
 
 	MAGIC_CHECK(dma->magic,MAGIC_DMABUF);
 	BUG_ON(0 == dma->nr_pages);
@@ -248,10 +247,8 @@ int videobuf_dma_map(struct videobuf_queue* q,struct videobuf_dmabuf *dma)
 		return -ENOMEM;
 	}
 	if (!dma->bus_addr) {
-		if (ops && ops->vb_map_sg) {
-			dma->sglen = ops->vb_map_sg(dev,dma->sglist,
+		dma->sglen = pci_map_sg(dev,dma->sglist,
 					dma->nr_pages, dma->direction);
-		}
 		if (0 == dma->sglen) {
 			printk(KERN_WARNING
 			       "%s: videobuf_map_sg failed\n",__FUNCTION__);
@@ -267,30 +264,28 @@ int videobuf_dma_map(struct videobuf_queue* q,struct videobuf_dmabuf *dma)
 int videobuf_dma_sync(struct videobuf_queue *q,struct videobuf_dmabuf *dma)
 {
 	void                   *dev=q->dev;
-	struct videobuf_dma_sg_ops *ops=q->priv_ops;
 
 	MAGIC_CHECK(dma->magic,MAGIC_DMABUF);
 	BUG_ON(!dma->sglen);
 
-	if (!dma->bus_addr && ops && ops->vb_dma_sync_sg)
-		ops->vb_dma_sync_sg(dev,dma->sglist,dma->nr_pages,
-							dma->direction);
-
+#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,5)
+	pci_dma_sync_sg	(dev,dma->sglist,dma->nr_pages,dma->direction);
+#else
+	pci_dma_sync_sg_for_cpu (dev,dma->sglist,dma->nr_pages,dma->direction);
+#endif
 	return 0;
 }
 
 int videobuf_dma_unmap(struct videobuf_queue* q,struct videobuf_dmabuf *dma)
 {
 	void                   *dev=q->dev;
-	struct videobuf_dma_sg_ops *ops=q->priv_ops;
 
 	MAGIC_CHECK(dma->magic,MAGIC_DMABUF);
 	if (!dma->sglen)
 		return 0;
 
-	if (!dma->bus_addr && ops && ops->vb_unmap_sg)
-			ops->vb_unmap_sg(dev,dma->sglist,dma->nr_pages,
-							dma->direction);
+	pci_unmap_sg (dev,dma->sglist,dma->nr_pages,dma->direction);
+
 	kfree(dma->sglist);
 	dma->sglist = NULL;
 	dma->sglen = 0;
@@ -326,12 +321,8 @@ int videobuf_dma_free(struct videobuf_dmabuf *dma)
 int videobuf_pci_dma_map(struct pci_dev *pci,struct videobuf_dmabuf *dma)
 {
 	struct videobuf_queue q;
-	struct videobuf_dma_sg_ops qops;
 
 	q.dev=pci;
-	qops.vb_map_sg=(vb_map_sg_t *)pci_map_sg;
-	qops.vb_unmap_sg=(vb_map_sg_t *)pci_unmap_sg;
-	q.priv_ops = &qops;
 
 	return (videobuf_dma_map(&q,dma));
 }
@@ -339,12 +330,8 @@ int videobuf_pci_dma_map(struct pci_dev *pci,struct videobuf_dmabuf *dma)
 int videobuf_pci_dma_unmap(struct pci_dev *pci,struct videobuf_dmabuf *dma)
 {
 	struct videobuf_queue q;
-	struct videobuf_dma_sg_ops qops;
 
 	q.dev=pci;
-	qops.vb_map_sg=(vb_map_sg_t *)pci_map_sg;
-	qops.vb_unmap_sg=(vb_map_sg_t *)pci_unmap_sg;
-	q.priv_ops = &qops;
 
 	return (videobuf_dma_unmap(&q,dma));
 }
@@ -729,55 +716,10 @@ void videobuf_queue_pci_init(struct videobuf_queue* q,
 			 unsigned int msize,
 			 void *priv)
 {
-	struct videobuf_dma_sg_ops *priv_ops;
-
 	videobuf_queue_init(q, ops, dev, irqlock, type, field, msize, priv);
 	q->int_ops=&pci_ops;
-
-	/* FIXME: the code bellow should be removed after having a proper
-	 * memory allocation method for vivi and tm6000
-	 */
-	q->priv_ops= kzalloc(sizeof(struct videobuf_dma_sg_ops), GFP_KERNEL);
-	BUG_ON (!q->priv_ops);
-
-	priv_ops=q->priv_ops;
-
-	/* Sets default methods for handling Scatter Gather mapping */
-	priv_ops->vb_map_sg=(vb_map_sg_t *)pci_map_sg;
-	priv_ops->vb_unmap_sg=(vb_map_sg_t *)pci_unmap_sg;
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,5)
-	priv_ops->vb_dma_sync_sg=(vb_map_sg_t *)pci_dma_sync_sg;
-#else
-	priv_ops->vb_dma_sync_sg=(vb_map_sg_t *)pci_dma_sync_sg_for_cpu;
-#endif
-}
-
-void videobuf_set_pci_ops (struct videobuf_queue* q,
-				struct videobuf_dma_sg_ops *ops)
-{
-	kfree (q->priv_ops);
-
-	q->priv_ops=ops;
-
-	if (!ops)
-		return;
-
-	/* If not specified, defaults to PCI map sg */
-	if (!ops->vb_map_sg)
-		ops->vb_map_sg=(vb_map_sg_t *)pci_map_sg;
-
-#if LINUX_VERSION_CODE < KERNEL_VERSION(2,6,5)
-	if (!ops->vb_dma_sync_sg)
-		ops->vb_dma_sync_sg=(vb_map_sg_t *)pci_dma_sync_sg;
-#else
-	if (!ops->vb_dma_sync_sg)
-		ops->vb_dma_sync_sg=(vb_map_sg_t *)pci_dma_sync_sg_for_cpu;
-#endif
-	if (!ops->vb_unmap_sg)
-		ops->vb_unmap_sg=(vb_map_sg_t *)pci_unmap_sg;
 }
 
-
 /* --------------------------------------------------------------------- */
 
 EXPORT_SYMBOL_GPL(videobuf_vmalloc_to_sg);
@@ -797,7 +739,6 @@ EXPORT_SYMBOL_GPL(videobuf_pci_dma_unmap);
 EXPORT_SYMBOL_GPL(videobuf_pci_alloc);
 
 EXPORT_SYMBOL_GPL(videobuf_queue_pci_init);
-EXPORT_SYMBOL_GPL(videobuf_set_pci_ops);
 
 /*
  * Local variables:
diff --git a/linux/include/media/videobuf-core.h b/linux/include/media/videobuf-core.h
index cf0759cb2..6e67c2c26 100644
--- a/linux/include/media/videobuf-core.h
+++ b/linux/include/media/videobuf-core.h
@@ -177,9 +177,6 @@ struct videobuf_queue {
 
 	/* driver private data */
 	void                       *priv_data;
-
-	/*FIXME: should be removed after completing the vb conversion */
-	void                       *priv_ops;
 };
 
 int videobuf_waiton(struct videobuf_buffer *vb, int non_blocking, int intr);
diff --git a/linux/include/media/videobuf-dma-sg.h b/linux/include/media/videobuf-dma-sg.h
index 206d9027b..38105031d 100644
--- a/linux/include/media/videobuf-dma-sg.h
+++ b/linux/include/media/videobuf-dma-sg.h
@@ -89,19 +89,6 @@ struct videbuf_pci_sg_memory
 	struct videobuf_dmabuf  dma;
 };
 
-/* FIXME: To be removed soon */
-typedef int (vb_map_sg_t)(void *dev, struct scatterlist *sglist, int nr_pages,
-					int direction);
-
-/* FIXME: To be removed soon */
-struct videobuf_dma_sg_ops
-{
-	vb_map_sg_t	*vb_map_sg;
-	vb_map_sg_t	*vb_dma_sync_sg;
-	vb_map_sg_t	*vb_unmap_sg;
-
-};
-
 void videobuf_dma_init(struct videobuf_dmabuf *dma);
 int videobuf_dma_init_user(struct videobuf_dmabuf *dma, int direction,
 			   unsigned long data, unsigned long size);
@@ -133,9 +120,3 @@ void videobuf_queue_pci_init(struct videobuf_queue* q,
 int videobuf_pci_dma_map(struct pci_dev *pci,struct videobuf_dmabuf *dma);
 int videobuf_pci_dma_unmap(struct pci_dev *pci,struct videobuf_dmabuf *dma);
 
-/* FIXME: temporary routine for vivi and tm6000, while lacking implementation
- * of videobuf-vmalloc
- */
-void videobuf_set_pci_ops (struct videobuf_queue* q,
-				struct videobuf_dma_sg_ops *ops);
-