On Sun, Dec 09, 2018 at 07:11:18AM +0000, French, Nicholas A. wrote:
> A typo in code cleanup commit db9c1007bc07 ("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.
>
> Signed-off-by: Nick French <naf@xxxxxx>
> ---
> drivers/media/dvb-frontends/lgdt330x.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/dvb-frontends/lgdt330x.c b/drivers/media/dvb-frontends/lgdt330x.c
> index 96807e134886..8abb1a510a81 100644
> --- a/drivers/media/dvb-frontends/lgdt330x.c
> +++ b/drivers/media/dvb-frontends/lgdt330x.c
> @@ -783,7 +783,7 @@ static int lgdt3303_read_status(struct dvb_frontend *fe,
>
> if ((buf[0] & 0x02) == 0x00)
> *status |= FE_HAS_SYNC;
> - if ((buf[0] & 0xfd) == 0x01)
> + if ((buf[0] & 0x01) == 0x01)
> *status |= FE_HAS_VITERBI | FE_HAS_LOCK;
> break;
> default:
> --
> 2.19.2
>
Patch worked for me.
Tested-by: Adam Stylinski <kungfujesus06@xxxxxxxxx>
Attachment:
signature.asc
Description: Digital signature
