upstream: [media] dvbdev: add a dvb_detach() macro
authorMauro Carvalho Chehab <m.chehab@samsung.com>
Thu, 29 May 2014 12:20:13 +0000 (09:20 -0300)
committerChanho Park <chanho61.park@samsung.com>
Tue, 18 Nov 2014 02:59:44 +0000 (11:59 +0900)
The dvb_attach() was unbalanced, as there was no dvb_dettach. Ok,
on current cases, the dettach is done by dvbdev, but that are some
future corner cases where we may need to do this before registering
the frontend.

So, add a dvb_detach() and use it at dvb_frontend.c.

Backports http://git.linuxtv.org/cgit.cgi/media_tree.git/commit/?id=4647f487083d
Signed-off-by: Mauro Carvalho Chehab <m.chehab@samsung.com>
drivers/media/dvb-core/dvb_frontend.c
drivers/media/dvb-core/dvbdev.h

index 6ce435a..6cc2631 100644 (file)
@@ -2666,20 +2666,20 @@ void dvb_frontend_detach(struct dvb_frontend* fe)
 
        if (fe->ops.release_sec) {
                fe->ops.release_sec(fe);
-               symbol_put_addr(fe->ops.release_sec);
+               dvb_detach(fe->ops.release_sec);
        }
        if (fe->ops.tuner_ops.release) {
                fe->ops.tuner_ops.release(fe);
-               symbol_put_addr(fe->ops.tuner_ops.release);
+               dvb_detach(fe->ops.tuner_ops.release);
        }
        if (fe->ops.analog_ops.release) {
                fe->ops.analog_ops.release(fe);
-               symbol_put_addr(fe->ops.analog_ops.release);
+               dvb_detach(fe->ops.analog_ops.release);
        }
        ptr = (void*)fe->ops.release;
        if (ptr) {
                fe->ops.release(fe);
-               symbol_put_addr(ptr);
+               dvb_detach(ptr);
        }
 }
 #else
index 93a9470..f96b28e 100644 (file)
@@ -136,11 +136,15 @@ extern int dvb_usercopy(struct file *file, unsigned int cmd, unsigned long arg,
        __r; \
 })
 
+#define dvb_detach(FUNC)       symbol_put_addr(FUNC)
+
 #else
 #define dvb_attach(FUNCTION, ARGS...) ({ \
        FUNCTION(ARGS); \
 })
 
+#define dvb_detach(FUNC)       {}
+
 #endif
 
 #endif /* #ifndef _DVBDEV_H_ */