]> git.baikalelectronics.ru Git - kernel.git/commit
[media] dvb_frontend: Don't call get_frontend() if idle
authorMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 17 Jan 2012 18:20:37 +0000 (16:20 -0200)
committerMauro Carvalho Chehab <mchehab@redhat.com>
Tue, 17 Jan 2012 18:20:37 +0000 (16:20 -0200)
commit1b518539aecf6ad54754e32047b5d8d5a11cfb44
tree0e428e66df6e83c90a5bb0129c15fc117f4ceb75
parent31afc5b9f7b2130699f2a0973fe54c273b852192
[media] dvb_frontend: Don't call get_frontend() if idle

If the frontend is in idle state, don't call get_frontend.

Calling get_frontend() when the device is not tuned may
result in wrong parameters to be returned to the
userspace.

I was tempted to not call get_frontend() at all, except
inside the dvb frontend thread, but this won't work for
all cases. The ISDB-T specs (ABNT NBR 15601 and ARIB
STD-B31) allow the broadcaster to dynamically change the
channel specs at runtime. That means that an ISDB-T optimized
application may want/need to monitor the TMCC tables, decoded
at the frontends via get_frontend call.

So, let's do the simpler change here.

Eventually, the logic could be changed to work only if
the device is tuned and has lock, but, even so, the
lock is also standard-dependent. For ISDB-T, the right
lock to wait is that the demod has TMCC lock. So, drivers
may need to implement some logic to detect if the get_frontend
info was retrieved or not.

Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
drivers/media/dvb/dvb-core/dvb_frontend.c