diff options
author | Darron Broad <darron@kewl.org> | 2008-10-15 12:43:41 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2008-10-17 16:28:31 -0400 |
commit | 7bdf84fc47f2d2ed2194b6ade480d043207c4098 (patch) | |
tree | 8f015d667fcba57e7223458c61c1de87f8d67413 /drivers/media/video/videobuf-dvb.c | |
parent | 6594690b39f9f9fcadafb1caf019bfd7a326e2e5 (diff) |
V4L/DVB (9265): videobuf: data storage optimisation
To optimise data storage redundant vars are removed.
Signed-off-by: Darron Broad <darron@kewl.org>
Signed-off-by: Steven Toth <stoth@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/videobuf-dvb.c')
-rw-r--r-- | drivers/media/video/videobuf-dvb.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/drivers/media/video/videobuf-dvb.c b/drivers/media/video/videobuf-dvb.c index af0b75cda6fd..fc4cfaa7bf5f 100644 --- a/drivers/media/video/videobuf-dvb.c +++ b/drivers/media/video/videobuf-dvb.c | |||
@@ -162,9 +162,8 @@ int videobuf_dvb_register_bus(struct videobuf_dvb_frontends *f, | |||
162 | 162 | ||
163 | /* Attach all of the frontends to the adapter */ | 163 | /* Attach all of the frontends to the adapter */ |
164 | mutex_lock(&f->lock); | 164 | mutex_lock(&f->lock); |
165 | list_for_each_safe(list, q, &f->frontend.felist) { | 165 | list_for_each_safe(list, q, &f->felist) { |
166 | fe = list_entry(list, struct videobuf_dvb_frontend, felist); | 166 | fe = list_entry(list, struct videobuf_dvb_frontend, felist); |
167 | |||
168 | res = videobuf_dvb_register_frontend(&f->adapter, &fe->dvb); | 167 | res = videobuf_dvb_register_frontend(&f->adapter, &fe->dvb); |
169 | if (res < 0) { | 168 | if (res < 0) { |
170 | printk(KERN_WARNING "%s: videobuf_dvb_register_frontend failed (errno = %d)\n", | 169 | printk(KERN_WARNING "%s: videobuf_dvb_register_frontend failed (errno = %d)\n", |
@@ -290,7 +289,7 @@ void videobuf_dvb_unregister_bus(struct videobuf_dvb_frontends *f) | |||
290 | struct videobuf_dvb_frontend *fe; | 289 | struct videobuf_dvb_frontend *fe; |
291 | 290 | ||
292 | mutex_lock(&f->lock); | 291 | mutex_lock(&f->lock); |
293 | list_for_each_safe(list, q, &f->frontend.felist) { | 292 | list_for_each_safe(list, q, &f->felist) { |
294 | fe = list_entry(list, struct videobuf_dvb_frontend, felist); | 293 | fe = list_entry(list, struct videobuf_dvb_frontend, felist); |
295 | 294 | ||
296 | dvb_net_release(&fe->dvb.net); | 295 | dvb_net_release(&fe->dvb.net); |
@@ -316,7 +315,7 @@ struct videobuf_dvb_frontend * videobuf_dvb_get_frontend(struct videobuf_dvb_fro | |||
316 | 315 | ||
317 | mutex_lock(&f->lock); | 316 | mutex_lock(&f->lock); |
318 | 317 | ||
319 | list_for_each_safe(list, q, &f->frontend.felist) { | 318 | list_for_each_safe(list, q, &f->felist) { |
320 | fe = list_entry(list, struct videobuf_dvb_frontend, felist); | 319 | fe = list_entry(list, struct videobuf_dvb_frontend, felist); |
321 | if (fe->id == id) { | 320 | if (fe->id == id) { |
322 | ret = fe; | 321 | ret = fe; |
@@ -337,7 +336,7 @@ int videobuf_dvb_find_frontend(struct videobuf_dvb_frontends *f, struct dvb_fron | |||
337 | 336 | ||
338 | mutex_lock(&f->lock); | 337 | mutex_lock(&f->lock); |
339 | 338 | ||
340 | list_for_each_safe(list, q, &f->frontend.felist) { | 339 | list_for_each_safe(list, q, &f->felist) { |
341 | fe = list_entry(list, struct videobuf_dvb_frontend, felist); | 340 | fe = list_entry(list, struct videobuf_dvb_frontend, felist); |
342 | if (fe->dvb.frontend == p) { | 341 | if (fe->dvb.frontend == p) { |
343 | ret = fe->id; | 342 | ret = fe->id; |
@@ -363,7 +362,7 @@ struct videobuf_dvb_frontend * videobuf_dvb_alloc_frontend(void *private, struct | |||
363 | mutex_init(&fe->dvb.lock); | 362 | mutex_init(&fe->dvb.lock); |
364 | 363 | ||
365 | mutex_lock(&f->lock); | 364 | mutex_lock(&f->lock); |
366 | list_add_tail(&fe->felist,&f->frontend.felist); | 365 | list_add_tail(&fe->felist,&f->felist); |
367 | mutex_unlock(&f->lock); | 366 | mutex_unlock(&f->lock); |
368 | 367 | ||
369 | fail_alloc: | 368 | fail_alloc: |