From 13158eca8802518b8b4426eb20f86aae1b512461 Mon Sep 17 00:00:00 2001 From: Michael Krufky Date: Mon, 22 Oct 2007 17:15:39 -0400 Subject: tuner-core: improve comments inside function fe_release() From: Michael Krufky Explain who is responsible for freeing analog_demod_priv Signed-off-by: Michael Krufky Signed-off-by: Mauro Carvalho Chehab --- linux/drivers/media/video/tuner-core.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) --- linux/drivers/media/video/tuner-core.c | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) (limited to 'linux/drivers/media') diff --git a/linux/drivers/media/video/tuner-core.c b/linux/drivers/media/video/tuner-core.c index f04c53631..9e7cf77e0 100644 --- a/linux/drivers/media/video/tuner-core.c +++ b/linux/drivers/media/video/tuner-core.c @@ -128,7 +128,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; } -- cgit v1.2.3