summaryrefslogtreecommitdiff
path: root/drivers/media/i2c/adv7604.c
diff options
context:
space:
mode:
authorHans Verkuil <hverkuil@xs4all.nl>2017-12-03 18:03:11 +0300
committerMauro Carvalho Chehab <mchehab@s-opensource.com>2017-12-08 18:39:17 +0300
commit979d33d3a7ea6ae76e6fecd457fd6be8c57052c7 (patch)
treed8b10724af70ce1beed9ac96176e328c402cc09d /drivers/media/i2c/adv7604.c
parent678856299c0db53c47bfcbedda7921d878a78be5 (diff)
downloadlinux-979d33d3a7ea6ae76e6fecd457fd6be8c57052c7.tar.xz
media: adv7604.c: add missing return
A 'return' was missing when detecting Arbitration Lost and calling transmit_done. With the return transmit_done could be called a second time, confusing the CEC framework. Luckily the Arbitration Lost condition is very rare. Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
Diffstat (limited to 'drivers/media/i2c/adv7604.c')
-rw-r--r--drivers/media/i2c/adv7604.c1
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/media/i2c/adv7604.c b/drivers/media/i2c/adv7604.c
index c786cd125417..1544920ec52d 100644
--- a/drivers/media/i2c/adv7604.c
+++ b/drivers/media/i2c/adv7604.c
@@ -1982,6 +1982,7 @@ static void adv76xx_cec_tx_raw_status(struct v4l2_subdev *sd, u8 tx_raw_status)
__func__);
cec_transmit_done(state->cec_adap, CEC_TX_STATUS_ARB_LOST,
1, 0, 0, 0);
+ return;
}
if (tx_raw_status & 0x04) {
u8 status;