[media] Mantis: use dvb_attach to avoid double dereferencing on module removal
authorBjørn Mork <bjorn@mork.no>
Sun, 14 Nov 2010 17:24:36 +0000 (14:24 -0300)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Wed, 29 Dec 2010 10:16:48 +0000 (08:16 -0200)
Convert the driver to use the dvb_attach macro to avoid the hard dependency
on the frontend drivers.  The hard dependecy will result in loading a
number of unused frontends, and unwanted automatic dereferencing.

This fixes a bug where unloading the mantis driver will derefence any
attached frontend twice, which will cause an oops if the same frontend is
used by another driver.

Signed-off-by: Bjørn Mork <bjorn@mork.no>
Signed-off-by: Manu Abraham <manu@linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/mantis/hopper_vp3028.c
drivers/media/dvb/mantis/mantis_vp1033.c
drivers/media/dvb/mantis/mantis_vp1034.c
drivers/media/dvb/mantis/mantis_vp1041.c
drivers/media/dvb/mantis/mantis_vp2033.c
drivers/media/dvb/mantis/mantis_vp2040.c
drivers/media/dvb/mantis/mantis_vp3030.c

index 96674c78e86be22a588839e0411e890b93b6c70e..567ed24f2de34d50666bc7069847f854e80ab9eb 100644 (file)
@@ -57,7 +57,7 @@ static int vp3028_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
        if (err == 0) {
                msleep(250);
                dprintk(MANTIS_ERROR, 1, "Probing for 10353 (DVB-T)");
-               fe = zl10353_attach(&hopper_vp3028_config, adapter);
+               fe = dvb_attach(zl10353_attach, &hopper_vp3028_config, adapter);
 
                if (!fe)
                        return -1;
index 4a723bda0031197f8cd897d0383cc670e7960f09..deec927c7f7a2f2fdddf3c954c3c55acfbdee878 100644 (file)
@@ -173,7 +173,7 @@ static int vp1033_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
                msleep(250);
 
                dprintk(MANTIS_ERROR, 1, "Probing for STV0299 (DVB-S)");
-               fe = stv0299_attach(&lgtdqcs001f_config, adapter);
+               fe = dvb_attach(stv0299_attach, &lgtdqcs001f_config, adapter);
 
                if (fe) {
                        fe->ops.tuner_ops.set_params = lgtdqcs001f_tuner_set;
index 8e6ae558ee57424b40f5622ce44ea86ca2c1a737..bf62338a1745c40a8eb25ef660bcf1549af9deaa 100644 (file)
@@ -82,7 +82,7 @@ static int vp1034_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
                msleep(250);
 
                dprintk(MANTIS_ERROR, 1, "Probing for MB86A16 (DVB-S/DSS)");
-               fe = mb86a16_attach(&vp1034_mb86a16_config, adapter);
+               fe = dvb_attach(mb86a16_attach, &vp1034_mb86a16_config, adapter);
                if (fe) {
                        dprintk(MANTIS_ERROR, 1,
                        "found MB86A16 DVB-S/DSS frontend @0x%02x",
index d1aa2bc0c155af4e5d07e4975b8fb839b6c87bcf..38a436ca2fdf352341c141b87fc5ec3ede2a4311 100644 (file)
@@ -316,14 +316,14 @@ static int vp1041_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
        if (err == 0) {
                mantis_frontend_soft_reset(mantis);
                msleep(250);
-               mantis->fe = stb0899_attach(&vp1041_stb0899_config, adapter);
+               mantis->fe = dvb_attach(stb0899_attach, &vp1041_stb0899_config, adapter);
                if (mantis->fe) {
                        dprintk(MANTIS_ERROR, 1,
                                "found STB0899 DVB-S/DVB-S2 frontend @0x%02x",
                                vp1041_stb0899_config.demod_address);
 
-                       if (stb6100_attach(mantis->fe, &vp1041_stb6100_config, adapter)) {
-                               if (!lnbp21_attach(mantis->fe, adapter, 0, 0))
+                       if (dvb_attach(stb6100_attach, mantis->fe, &vp1041_stb6100_config, adapter)) {
+                               if (!dvb_attach(lnbp21_attach, mantis->fe, adapter, 0, 0))
                                        dprintk(MANTIS_ERROR, 1, "No LNBP21 found!");
                        }
                } else {
index 10ce81790a8c2bc96afed737287e4e493153d27a..06da0ddf05a77b46bcb582a274019499a6cb5f67 100644 (file)
@@ -132,7 +132,7 @@ static int vp2033_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
                msleep(250);
 
                dprintk(MANTIS_ERROR, 1, "Probing for CU1216 (DVB-C)");
-               fe = tda10021_attach(&vp2033_tda1002x_cu1216_config,
+               fe = dvb_attach(tda10021_attach, &vp2033_tda1002x_cu1216_config,
                                     adapter,
                                     read_pwm(mantis));
 
@@ -141,7 +141,7 @@ static int vp2033_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
                                "found Philips CU1216 DVB-C frontend (TDA10021) @ 0x%02x",
                                vp2033_tda1002x_cu1216_config.demod_address);
                } else {
-                       fe = tda10023_attach(&vp2033_tda10023_cu1216_config,
+                       fe = dvb_attach(tda10023_attach, &vp2033_tda10023_cu1216_config,
                                             adapter,
                                             read_pwm(mantis));
 
index a7ca233e800b67b6dd2a6f7ade5bf3b7d46c5434..f72b137b7652b94eded9ddb3351b410c1993de0b 100644 (file)
@@ -132,7 +132,7 @@ static int vp2040_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
                msleep(250);
 
                dprintk(MANTIS_ERROR, 1, "Probing for CU1216 (DVB-C)");
-               fe = tda10021_attach(&vp2040_tda1002x_cu1216_config,
+               fe = dvb_attach(tda10021_attach, &vp2040_tda1002x_cu1216_config,
                                     adapter,
                                     read_pwm(mantis));
 
@@ -141,7 +141,7 @@ static int vp2040_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
                                "found Philips CU1216 DVB-C frontend (TDA10021) @ 0x%02x",
                                vp2040_tda1002x_cu1216_config.demod_address);
                } else {
-                       fe = tda10023_attach(&vp2040_tda10023_cu1216_config,
+                       fe = dvb_attach(tda10023_attach, &vp2040_tda10023_cu1216_config,
                                             adapter,
                                             read_pwm(mantis));
 
index 1f4334214953f4c43caa5d896340005f39278dad..be4d87c04149678c11db69557811b157e2d699e2 100644 (file)
@@ -68,12 +68,12 @@ static int vp3030_frontend_init(struct mantis_pci *mantis, struct dvb_frontend *
        if (err == 0) {
                msleep(250);
                dprintk(MANTIS_ERROR, 1, "Probing for 10353 (DVB-T)");
-               fe = zl10353_attach(&mantis_vp3030_config, adapter);
+               fe = dvb_attach(zl10353_attach, &mantis_vp3030_config, adapter);
 
                if (!fe)
                        return -1;
 
-               tda665x_attach(fe, &env57h12d5_config, adapter);
+               dvb_attach(tda665x_attach, fe, &env57h12d5_config, adapter);
        } else {
                dprintk(MANTIS_ERROR, 1, "Frontend on <%s> POWER ON failed! <%d>",
                        adapter->name,