diff options
| author | Hans Verkuil <hverkuil@xs4all.nl> | 2017-12-03 18:03:11 +0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-12-08 18:39:17 +0300 |
| commit | 979d33d3a7ea6ae76e6fecd457fd6be8c57052c7 (patch) | |
| tree | d8b10724af70ce1beed9ac96176e328c402cc09d /drivers/media/i2c/adv7604.c | |
| parent | 678856299c0db53c47bfcbedda7921d878a78be5 (diff) | |
| download | linux-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.c | 1 |
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; |