diff options
| author | Hans Verkuil <hverkuil@xs4all.nl> | 2017-07-12 16:07:08 -0400 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@s-opensource.com> | 2017-07-26 05:46:43 -0400 |
| commit | bd34ca8758d0ee9b13c0c067a956b0e9eb247d18 (patch) | |
| tree | 0263b1e597a3004a91e8f115242d9cce0b859efc | |
| parent | f3aa68405d907cdec852078d73a200608d5aa473 (diff) | |
media: cec: cec_transmit_attempt_done: ignore CEC_TX_STATUS_MAX_RETRIES
The switch in cec_transmit_attempt_done() should ignore the
CEC_TX_STATUS_MAX_RETRIES status bit.
Calling this function with e.g. CEC_TX_STATUS_NACK | CEC_TX_STATUS_MAX_RETRIES
is perfectly legal and should not trigger the WARN(1).
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
| -rw-r--r-- | drivers/media/cec/cec-adap.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/cec/cec-adap.c b/drivers/media/cec/cec-adap.c index bf45977b2823..d596b601ff42 100644 --- a/drivers/media/cec/cec-adap.c +++ b/drivers/media/cec/cec-adap.c | |||
| @@ -559,7 +559,7 @@ EXPORT_SYMBOL_GPL(cec_transmit_done); | |||
| 559 | 559 | ||
| 560 | void cec_transmit_attempt_done(struct cec_adapter *adap, u8 status) | 560 | void cec_transmit_attempt_done(struct cec_adapter *adap, u8 status) |
| 561 | { | 561 | { |
| 562 | switch (status) { | 562 | switch (status & ~CEC_TX_STATUS_MAX_RETRIES) { |
| 563 | case CEC_TX_STATUS_OK: | 563 | case CEC_TX_STATUS_OK: |
| 564 | cec_transmit_done(adap, status, 0, 0, 0, 0); | 564 | cec_transmit_done(adap, status, 0, 0, 0, 0); |
| 565 | return; | 565 | return; |
