summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Krufky <mkrufky@linuxtv.org>2007-10-22 18:15:39 -0300
committerMauro Carvalho Chehab <mchehab@infradead.org>2008-01-25 19:01:11 -0200
commit4524c1aba5449d3519d1545f8106811eb73fc72c (patch)
tree9740467620d93b5fd488f246637c6a4fc588a0d6
parentaf3b0f3f01d299c46837a408e1b9510b8c304e43 (diff)
V4L/DVB (6445): tuner-core: improve comments inside function fe_release()
Explain who is responsible for freeing analog_demod_priv Signed-off-by: Michael Krufky <mkrufky@linuxtv.org> Signed-off-by: Mauro Carvalho Chehab <mchehab@infradead.org>
-rw-r--r--drivers/media/video/tuner-core.c10
1 files changed, 9 insertions, 1 deletions
diff --git a/drivers/media/video/tuner-core.c b/drivers/media/video/tuner-core.c
index 1e9c9755dec..88db8b33c86 100644
--- a/drivers/media/video/tuner-core.c
+++ b/drivers/media/video/tuner-core.c
@@ -103,7 +103,15 @@ static void fe_release(struct dvb_frontend *fe)
fe->ops.tuner_ops.release(fe);
fe->ops.analog_demod_ops = NULL;
- /* DO NOT kfree(t->fe.analog_demod_priv) */
+
+ /* DO NOT kfree(fe->analog_demod_priv)
+ *
+ * If we are in this function, analog_demod_priv contains a pointer
+ * to struct tuner *t. This will be kfree'd in tuner_detach().
+ *
+ * Otherwise, fe->ops.analog_demod_ops->release will
+ * handle the cleanup for analog demodulator modules.
+ */
fe->analog_demod_priv = NULL;
}