Commit 1b4fd9de authored by French, Nicholas A's avatar French, Nicholas A Committed by Mauro Carvalho Chehab

media: lgdt330x: fix lock status reporting

A typo in code cleanup commit db9c1007 ("media: lgdt330x: do
some cleanups at status logic") broke the FE_HAS_LOCK reporting
for 3303 chips by inadvertently modifying the register mask.

The broken lock status is critial as it prevents video capture
cards from reporting signal strength, scanning for channels,
and capturing video.

Fix regression by reverting mask change.

Cc: stable@vger.kernel.org # Kernel 4.17+
Fixes: db9c1007 ("media: lgdt330x: do some cleanups at status logic")
Signed-off-by: default avatarNick French <naf@ou.edu>
Reviewed-by: default avatarLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Tested-by: default avatarAdam Stylinski <kungfujesus06@gmail.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+samsung@kernel.org>
parent bb03641f
...@@ -783,7 +783,7 @@ static int lgdt3303_read_status(struct dvb_frontend *fe, ...@@ -783,7 +783,7 @@ static int lgdt3303_read_status(struct dvb_frontend *fe,
if ((buf[0] & 0x02) == 0x00) if ((buf[0] & 0x02) == 0x00)
*status |= FE_HAS_SYNC; *status |= FE_HAS_SYNC;
if ((buf[0] & 0xfd) == 0x01) if ((buf[0] & 0x01) == 0x01)
*status |= FE_HAS_VITERBI | FE_HAS_LOCK; *status |= FE_HAS_VITERBI | FE_HAS_LOCK;
break; break;
default: default:
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment