videobuf: data storage optimisation

From: Darron Broad <darron@kewl.org>

To optimise data storage redundant vars are removed.

Priority: normal

Signed-off-by: Darron Broad <darron@kewl.org>
Signed-off-by: Steven Toth <stoth@linuxtv.org>

4 files changed, 8 insertions, 9 deletions

diff --git a/linux/drivers/media/video/cx23885/cx23885-core.c b/linux/drivers/media/video/cx23885/cx23885-core.c
index 009e5cb01..4f68156d2 100644
--- a/linux/drivers/media/video/cx23885/cx23885-core.c
+++ b/linux/drivers/media/video/cx23885/cx23885-core.c
@@ -644,7 +644,7 @@ static int cx23885_init_tsport(struct cx23885_dev *dev,
 	init_timer(&port->mpegq.timeout);
 
 	mutex_init(&port->frontends.lock);
-	INIT_LIST_HEAD(&port->frontends.frontend.felist);
+	INIT_LIST_HEAD(&port->frontends.felist);
 	port->frontends.active_fe_id = 0;
 
 	/* This should be hardcoded allow a single frontend
diff --git a/linux/drivers/media/video/cx88/cx88-mpeg.c b/linux/drivers/media/video/cx88/cx88-mpeg.c
index 819d7fbdf..8df19ae43 100644
--- a/linux/drivers/media/video/cx88/cx88-mpeg.c
+++ b/linux/drivers/media/video/cx88/cx88-mpeg.c
@@ -842,7 +842,7 @@ static int __devinit cx8802_probe(struct pci_dev *pci_dev,
 	list_add_tail(&dev->devlist,&cx8802_devlist);
 
 	mutex_init(&dev->frontends.lock);
-	INIT_LIST_HEAD(&dev->frontends.frontend.felist);
+	INIT_LIST_HEAD(&dev->frontends.felist);
 
 	printk(KERN_INFO "%s() allocating %d frontend(s)\n", __func__, core->board.num_frontends);
 
diff --git a/linux/drivers/media/video/saa7134/saa7134-dvb.c b/linux/drivers/media/video/saa7134/saa7134-dvb.c
index 11a7c76bc..63f76afac 100644
--- a/linux/drivers/media/video/saa7134/saa7134-dvb.c
+++ b/linux/drivers/media/video/saa7134/saa7134-dvb.c
@@ -953,7 +953,7 @@ static int dvb_init(struct saa7134_dev *dev)
 
 	/* FIXME: add support for multi-frontend */
 	mutex_init(&dev->frontends.lock);
-	INIT_LIST_HEAD(&dev->frontends.frontend.felist);
+	INIT_LIST_HEAD(&dev->frontends.felist);
 	dev->frontends.active_fe_id = 0;
 
 	printk(KERN_INFO "%s() allocating 1 frontend\n", __func__);
diff --git a/linux/drivers/media/video/videobuf-dvb.c b/linux/drivers/media/video/videobuf-dvb.c
index 33e2f2b2b..2b7c10674 100644
--- a/linux/drivers/media/video/videobuf-dvb.c
+++ b/linux/drivers/media/video/videobuf-dvb.c
@@ -167,9 +167,8 @@ int videobuf_dvb_register_bus(struct videobuf_dvb_frontends *f,
 
 	/* Attach all of the frontends to the adapter */
 	mutex_lock(&f->lock);
-	list_for_each_safe(list, q, &f->frontend.felist) {
+	list_for_each_safe(list, q, &f->felist) {
 		fe = list_entry(list, struct videobuf_dvb_frontend, felist);
-
 		res = videobuf_dvb_register_frontend(&f->adapter, &fe->dvb);
 		if (res < 0) {
 			printk(KERN_WARNING "%s: videobuf_dvb_register_frontend failed (errno = %d)\n",
@@ -295,7 +294,7 @@ void videobuf_dvb_unregister_bus(struct videobuf_dvb_frontends *f)
 	struct videobuf_dvb_frontend *fe;
 
 	mutex_lock(&f->lock);
-	list_for_each_safe(list, q, &f->frontend.felist) {
+	list_for_each_safe(list, q, &f->felist) {
 		fe = list_entry(list, struct videobuf_dvb_frontend, felist);
 
 		dvb_net_release(&fe->dvb.net);
@@ -321,7 +320,7 @@ struct videobuf_dvb_frontend * videobuf_dvb_get_frontend(struct videobuf_dvb_fro
 
 	mutex_lock(&f->lock);
 
-	list_for_each_safe(list, q, &f->frontend.felist) {
+	list_for_each_safe(list, q, &f->felist) {
 		fe = list_entry(list, struct videobuf_dvb_frontend, felist);
 		if (fe->id == id) {
 			ret = fe;
@@ -342,7 +341,7 @@ int videobuf_dvb_find_frontend(struct videobuf_dvb_frontends *f, struct dvb_fron
 
 	mutex_lock(&f->lock);
 
-	list_for_each_safe(list, q, &f->frontend.felist) {
+	list_for_each_safe(list, q, &f->felist) {
 		fe = list_entry(list, struct videobuf_dvb_frontend, felist);
 		if (fe->dvb.frontend == p) {
 			ret = fe->id;
@@ -368,7 +367,7 @@ struct videobuf_dvb_frontend * videobuf_dvb_alloc_frontend(void *private, struct
 	mutex_init(&fe->dvb.lock);
 
 	mutex_lock(&f->lock);
-	list_add_tail(&fe->felist,&f->frontend.felist);
+	list_add_tail(&fe->felist,&f->felist);
 	mutex_unlock(&f->lock);
 
 fail_alloc: