summaryrefslogtreecommitdiffstats
path: root/drivers/media/dvb/frontends/au8522_decoder.c
diff options
context:
space:
mode:
authorDevin Heitmueller <dheitmueller@kernellabs.com>2010-02-17 22:47:55 -0300
committerMauro Carvalho Chehab <mchehab@redhat.com>2010-05-18 00:46:21 -0300
commit7f2c983cf5978186ee2c379fd63d04316158fc9b (patch)
treee8db82b72129777469ea7ba3a313f4ce237447e3 /drivers/media/dvb/frontends/au8522_decoder.c
parent535653b1c22c29d4e8f554928efc87fe138f917d (diff)
downloadop-kernel-dev-7f2c983cf5978186ee2c379fd63d04316158fc9b.zip
op-kernel-dev-7f2c983cf5978186ee2c379fd63d04316158fc9b.tar.gz
V4L/DVB: au8522: fix race condition in switching from digital to analog mode
With applications like MythTV, switching inputs results in closing the digital side and then immediately opening the analog side. This exposes a race condition where the dvb_frontend kernel thread powers down the chip and closes the i2c gate even though we're in the middle of bringing up the analog part of the chip (since the shutdown of the dvb_frontend kernel thread occurs asychronously). Introduce a construct to keep track of what mode we're in, and drop requests to power down or management the gate if we've already switched to analog mode. Thanks to Zaphod Beeblebrox for reporting this issue. Signed-off-by: Devin Heitmueller <dheitmueller@kernellabs.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@redhat.com>
Diffstat (limited to 'drivers/media/dvb/frontends/au8522_decoder.c')
-rw-r--r--drivers/media/dvb/frontends/au8522_decoder.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/media/dvb/frontends/au8522_decoder.c b/drivers/media/dvb/frontends/au8522_decoder.c
index 24268ef..ef5fd49 100644
--- a/drivers/media/dvb/frontends/au8522_decoder.c
+++ b/drivers/media/dvb/frontends/au8522_decoder.c
@@ -664,6 +664,8 @@ static int au8522_reset(struct v4l2_subdev *sd, u32 val)
{
struct au8522_state *state = to_state(sd);
+ state->operational_mode = AU8522_ANALOG_MODE;
+
au8522_writereg(state, 0xa4, 1 << 5);
return 0;
OpenPOWER on IntegriCloud