summaryrefslogtreecommitdiffstats
path: root/drivers/media
diff options
context:
space:
mode:
authorDavid Howells <dhowells@redhat.com>2015-04-02 08:03:11 -0300
committerMauro Carvalho Chehab <mchehab@osg.samsung.com>2015-06-10 11:00:06 -0300
commit80868c8e5c04bb725e411b078881b65ebbecd142 (patch)
tree129bc0207fc55814a2dd6c964f14ec887c32a2f4 /drivers/media
parentf158cbceb165f318a89a8bb831dc3ea27823b3f8 (diff)
downloadop-kernel-dev-80868c8e5c04bb725e411b078881b65ebbecd142.zip
op-kernel-dev-80868c8e5c04bb725e411b078881b65ebbecd142.tar.gz
[media] ts2020: Add a comment about lifetime of on-stack pdata in ts2020_attach()
ts2020_attach() allocates a variable pdata on the stack and then passes a pointer to it to i2c_new_device() which stashes the pointer in persistent structures. Add a comment to the effect that this isn't actually an error because the contents of the variable are only used in ts2020_probe() and this is only called ts2020_attach()'s stack frame exists. Signed-off-by: David Howells <dhowells@redhat.com> Signed-off-by: Antti Palosaari <crope@iki.fi> Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media')
-rw-r--r--drivers/media/dvb-frontends/ts2020.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/media/dvb-frontends/ts2020.c b/drivers/media/dvb-frontends/ts2020.c
index 797112b..f674717 100644
--- a/drivers/media/dvb-frontends/ts2020.c
+++ b/drivers/media/dvb-frontends/ts2020.c
@@ -363,6 +363,8 @@ struct dvb_frontend *ts2020_attach(struct dvb_frontend *fe,
{
struct i2c_client *client;
struct i2c_board_info board_info;
+
+ /* This is only used by ts2020_probe() so can be on the stack */
struct ts2020_config pdata;
memcpy(&pdata, config, sizeof(pdata));
OpenPOWER on IntegriCloud