V4L/DVB (9271): videobuf: data storage optimisation (2)
authorDarron Broad <darron@kewl.org>
Wed, 15 Oct 2008 23:26:34 +0000 (20:26 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Fri, 17 Oct 2008 20:29:59 +0000 (17:29 -0300)
To optimise data storage even further one other redundant
var has been removed.

This also removes a redundant assignment.

Signed-off-by: Steven Toth <stoth@linuxtv.org>
Signed-off-by: Darron Broad <darron@kewl.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/video/cx23885/cx23885-dvb.c
drivers/media/video/cx88/cx88-mpeg.c
drivers/media/video/saa7134/saa7134-dvb.c
drivers/media/video/videobuf-dvb.c
include/media/videobuf-dvb.h

index 8e9871106afa2929813be8f03bf1601fe42389d0..e1aac07b3158a42bd32210f1d3d0a123b09cb940 100644 (file)
@@ -571,7 +571,7 @@ int cx23885_dvb_register(struct cx23885_tsport *port)
                port->num_frontends);
 
        for (i = 1; i <= port->num_frontends; i++) {
-               if (videobuf_dvb_alloc_frontend(dev,
+               if (videobuf_dvb_alloc_frontend(
                        &port->frontends, i) == NULL) {
                        printk(KERN_ERR "%s() failed to alloc\n", __func__);
                        return -ENOMEM;
index 512041451d38301b00796bc9ab3be6613ece679d..6df5cf31418627025e16752fdecbab998040a7ed 100644 (file)
@@ -807,7 +807,7 @@ static int __devinit cx8802_probe(struct pci_dev *pci_dev,
        printk(KERN_INFO "%s() allocating %d frontend(s)\n", __func__, core->board.num_frontends);
 
        for (i = 1; i <= core->board.num_frontends; i++) {
-               demod = videobuf_dvb_alloc_frontend(dev, &dev->frontends, i);
+               demod = videobuf_dvb_alloc_frontend(&dev->frontends, i);
                if(demod == NULL) {
                        printk(KERN_ERR "%s() failed to alloc\n", __func__);
                        err = -ENOMEM;
index 7d95a6928772c6ba5ca9b4ffc070949e58edda65..8c46115d4c7980a34f672ec75a4179848a11d3d7 100644 (file)
@@ -958,7 +958,7 @@ static int dvb_init(struct saa7134_dev *dev)
 
        printk(KERN_INFO "%s() allocating 1 frontend\n", __func__);
 
-       if (videobuf_dvb_alloc_frontend(dev, &dev->frontends, 1) == NULL) {
+       if (videobuf_dvb_alloc_frontend(&dev->frontends, 1) == NULL) {
                printk(KERN_ERR "%s() failed to alloc\n", __func__);
                return -ENOMEM;
        }
index 7c74845af267e6267352121e0a45dea587bba8a4..36b1e67ac33051281c8d2fa3c0f9df2b909fa975 100644 (file)
@@ -307,10 +307,8 @@ void videobuf_dvb_unregister_bus(struct videobuf_dvb_frontends *f)
                        dvb_dmx_release(&fe->dvb.demux);
                        dvb_unregister_frontend(fe->dvb.frontend);
                }
-               if(fe->dvb.frontend) {  /* always allocated, may have been reset */
+               if(fe->dvb.frontend) /* always allocated, may have been reset */
                        dvb_frontend_detach(fe->dvb.frontend);
-                       fe->dvb.frontend = NULL;
-               }
                list_del(list);
                kfree(fe);
        }
@@ -360,7 +358,7 @@ int videobuf_dvb_find_frontend(struct videobuf_dvb_frontends *f, struct dvb_fron
        return ret;
 }
 
-struct videobuf_dvb_frontend * videobuf_dvb_alloc_frontend(void *private, struct videobuf_dvb_frontends *f, int id)
+struct videobuf_dvb_frontend * videobuf_dvb_alloc_frontend(struct videobuf_dvb_frontends *f, int id)
 {
        struct videobuf_dvb_frontend *fe;
 
@@ -368,7 +366,6 @@ struct videobuf_dvb_frontend * videobuf_dvb_alloc_frontend(void *private, struct
        if (fe == NULL)
                goto fail_alloc;
 
-       fe->dev = private;
        fe->id = id;
        mutex_init(&fe->dvb.lock);
 
index 069215c77a657e9e3404391e65796452f9a2df62..52ddda90d3f655fdb2892818f3e4c7d89171d28c 100644 (file)
@@ -25,7 +25,6 @@ struct videobuf_dvb {
 
 struct videobuf_dvb_frontend {
        struct list_head felist;
-       void *dev;
        int id;
        struct videobuf_dvb dvb;
 };
@@ -57,7 +56,7 @@ int videobuf_dvb_register_adapter(struct videobuf_dvb_frontends *f,
 
 int videobuf_dvb_register_frontend(struct dvb_adapter *adapter, struct videobuf_dvb *dvb);
 
-struct videobuf_dvb_frontend * videobuf_dvb_alloc_frontend(void *private, struct videobuf_dvb_frontends *f, int id);
+struct videobuf_dvb_frontend * videobuf_dvb_alloc_frontend(struct videobuf_dvb_frontends *f, int id);
 
 struct videobuf_dvb_frontend * videobuf_dvb_get_frontend(struct videobuf_dvb_frontends *f, int id);
 int videobuf_dvb_find_frontend(struct videobuf_dvb_frontends *f, struct dvb_frontend *p);