diff options
author | Darron Broad <darron@kewl.org> | 2008-10-11 10:18:53 -0400 |
---|---|---|
committer | Mauro Carvalho Chehab <mchehab@redhat.com> | 2008-10-17 16:23:15 -0400 |
commit | 92abe9ee374599179033f039b095864a9cf74593 (patch) | |
tree | 2e4304cac9052a8848b972e678d372bd49d7db29 /drivers/media/video/cx23885/cx23885-dvb.c | |
parent | 363c35fc448943c3d6121332d28bcda2d2fbf87c (diff) |
V4L/DVB (9223): MFE: Fix a number of bugs and some tidying up
A number of reference to videobuf_dvb_get_frontend used an invalid
index. This has been fixed.
The section for the HVR3000 in advise_acquire was redundant as
the same logic is used on the HVR4000. This has been removed
and both cards now use the same function.
A number of small errors and whitespace errors are also fixed.
Signed-off-by: Darron Broad <darron@kewl.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/video/cx23885/cx23885-dvb.c')
-rw-r--r-- | drivers/media/video/cx23885/cx23885-dvb.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/media/video/cx23885/cx23885-dvb.c b/drivers/media/video/cx23885/cx23885-dvb.c index fe1218fd44cb..d8c25a802561 100644 --- a/drivers/media/video/cx23885/cx23885-dvb.c +++ b/drivers/media/video/cx23885/cx23885-dvb.c | |||
@@ -314,7 +314,7 @@ static int dvb_register(struct cx23885_tsport *port) | |||
314 | struct cx23885_i2c *i2c_bus = NULL; | 314 | struct cx23885_i2c *i2c_bus = NULL; |
315 | struct videobuf_dvb_frontend *fe0; | 315 | struct videobuf_dvb_frontend *fe0; |
316 | 316 | ||
317 | fe0 = videobuf_dvb_get_frontend(&port->frontends, 0); | 317 | fe0 = videobuf_dvb_get_frontend(&port->frontends, 1); |
318 | if (!fe0) | 318 | if (!fe0) |
319 | return -EINVAL; | 319 | return -EINVAL; |
320 | 320 | ||
@@ -336,7 +336,7 @@ static int dvb_register(struct cx23885_tsport *port) | |||
336 | break; | 336 | break; |
337 | case CX23885_BOARD_HAUPPAUGE_HVR1800: | 337 | case CX23885_BOARD_HAUPPAUGE_HVR1800: |
338 | i2c_bus = &dev->i2c_bus[0]; | 338 | i2c_bus = &dev->i2c_bus[0]; |
339 | switch (alt_tuner) { // XXXXXX multifrontend? | 339 | switch (alt_tuner) { |
340 | case 1: | 340 | case 1: |
341 | fe0->dvb.frontend = | 341 | fe0->dvb.frontend = |
342 | dvb_attach(s5h1409_attach, | 342 | dvb_attach(s5h1409_attach, |
@@ -554,7 +554,7 @@ int cx23885_dvb_register(struct cx23885_tsport *port) | |||
554 | struct cx23885_dev *dev = port->dev; | 554 | struct cx23885_dev *dev = port->dev; |
555 | int err; | 555 | int err; |
556 | 556 | ||
557 | fe0 = videobuf_dvb_get_frontend(&port->frontends, 0); | 557 | fe0 = videobuf_dvb_get_frontend(&port->frontends, 1); |
558 | if (!fe0) | 558 | if (!fe0) |
559 | err = -EINVAL; | 559 | err = -EINVAL; |
560 | 560 | ||
@@ -583,7 +583,7 @@ int cx23885_dvb_unregister(struct cx23885_tsport *port) | |||
583 | { | 583 | { |
584 | struct videobuf_dvb_frontend *fe0; | 584 | struct videobuf_dvb_frontend *fe0; |
585 | 585 | ||
586 | fe0 = videobuf_dvb_get_frontend(&port->frontends, 0); | 586 | fe0 = videobuf_dvb_get_frontend(&port->frontends, 1); |
587 | /* dvb */ | 587 | /* dvb */ |
588 | if(fe0->dvb.frontend) | 588 | if(fe0->dvb.frontend) |
589 | videobuf_dvb_unregister_bus(&port->frontends); | 589 | videobuf_dvb_unregister_bus(&port->frontends); |