* [PATCH] Fix for hanging si2168 in PCTV 292e, making the code match
@ 2017-09-19 12:13 Nigel Kettlewell
2017-12-14 14:48 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 4+ messages in thread
From: Nigel Kettlewell @ 2017-09-19 12:13 UTC (permalink / raw)
To: crope; +Cc: linux-media, linux-kernel
[re-sending as plain text]
Fix for hanging si2168 in PCTV 292e USB, making the code match the comment.
Using firmware v4.0.11 the 292e would work once and then hang on
subsequent attempts to view DVB channels, until physically unplugged and
plugged back in.
With this patch, the warm state is reset for v4.0.11 and it appears to
work both on the first attempt and on subsequent attempts.
(Patch basis Linux 4.11.9 f82a53b87594f460f2dd9983eeb851a5840e8df8)
---
drivers/media/dvb-frontends/si2168.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/media/dvb-frontends/si2168.c
b/drivers/media/dvb-frontends/si2168.c
index 680ba06..523acd1 100644
--- a/drivers/media/dvb-frontends/si2168.c
+++ b/drivers/media/dvb-frontends/si2168.c
@@ -582,7 +582,7 @@ static int si2168_sleep(struct dvb_frontend *fe)
dev->active = false;
/* Firmware B 4.0-11 or later loses warm state during sleep */
- if (dev->version > ('B' << 24 | 4 << 16 | 0 << 8 | 11 << 0))
+ if (dev->version >= ('B' << 24 | 4 << 16 | 0 << 8 | 11 << 0))
dev->warm = false;
memcpy(cmd.args, "\x13", 1);
--
2.9.4
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Fix for hanging si2168 in PCTV 292e, making the code match
2017-09-19 12:13 [PATCH] Fix for hanging si2168 in PCTV 292e, making the code match Nigel Kettlewell
@ 2017-12-14 14:48 ` Mauro Carvalho Chehab
2018-03-07 13:23 ` Antti Palosaari
0 siblings, 1 reply; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2017-12-14 14:48 UTC (permalink / raw)
To: Nigel Kettlewell; +Cc: crope, linux-media, linux-kernel
Em Tue, 19 Sep 2017 13:13:52 +0100
Nigel Kettlewell <nigel.kettlewell@googlemail.com> escreveu:
> [re-sending as plain text]
>
> Fix for hanging si2168 in PCTV 292e USB, making the code match the comment.
>
> Using firmware v4.0.11 the 292e would work once and then hang on
> subsequent attempts to view DVB channels, until physically unplugged and
> plugged back in.
>
> With this patch, the warm state is reset for v4.0.11 and it appears to
> work both on the first attempt and on subsequent attempts.
>
> (Patch basis Linux 4.11.9 f82a53b87594f460f2dd9983eeb851a5840e8df8)
Patch is missing a Signed-off-by. See:
https://elinux.org/Developer_Certificate_Of_Origin).
>
> ---
> drivers/media/dvb-frontends/si2168.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/dvb-frontends/si2168.c
> b/drivers/media/dvb-frontends/si2168.c
> index 680ba06..523acd1 100644
> --- a/drivers/media/dvb-frontends/si2168.c
> +++ b/drivers/media/dvb-frontends/si2168.c
> @@ -582,7 +582,7 @@ static int si2168_sleep(struct dvb_frontend *fe)
> dev->active = false;
>
> /* Firmware B 4.0-11 or later loses warm state during sleep */
> - if (dev->version > ('B' << 24 | 4 << 16 | 0 << 8 | 11 << 0))
> + if (dev->version >= ('B' << 24 | 4 << 16 | 0 << 8 | 11 << 0))
> dev->warm = false;
>
> memcpy(cmd.args, "\x13", 1);
> --
> 2.9.4
>
Thanks,
Mauro
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Fix for hanging si2168 in PCTV 292e, making the code match
2017-12-14 14:48 ` Mauro Carvalho Chehab
@ 2018-03-07 13:23 ` Antti Palosaari
2018-03-07 14:02 ` Mauro Carvalho Chehab
0 siblings, 1 reply; 4+ messages in thread
From: Antti Palosaari @ 2018-03-07 13:23 UTC (permalink / raw)
To: Mauro Carvalho Chehab, Nigel Kettlewell; +Cc: linux-media, linux-kernel
On 12/14/2017 04:48 PM, Mauro Carvalho Chehab wrote:
> Em Tue, 19 Sep 2017 13:13:52 +0100
> Nigel Kettlewell <nigel.kettlewell@googlemail.com> escreveu:
>
>> [re-sending as plain text]
>>
>> Fix for hanging si2168 in PCTV 292e USB, making the code match the comment.
>>
>> Using firmware v4.0.11 the 292e would work once and then hang on
>> subsequent attempts to view DVB channels, until physically unplugged and
>> plugged back in.
>>
>> With this patch, the warm state is reset for v4.0.11 and it appears to
>> work both on the first attempt and on subsequent attempts.
It is comment which is wrong. With firmware 4.0.11 it works well without
need of download it every time. But firmware 4.0.19 needs to be
downloaded every time after device is put to sleep.
Probably your issue is coming from some other reason.
>>
>> (Patch basis Linux 4.11.9 f82a53b87594f460f2dd9983eeb851a5840e8df8)
>
> Patch is missing a Signed-off-by. See:
> https://elinux.org/Developer_Certificate_Of_Origin).
>
>
>>
>> ---
>> drivers/media/dvb-frontends/si2168.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/media/dvb-frontends/si2168.c
>> b/drivers/media/dvb-frontends/si2168.c
>> index 680ba06..523acd1 100644
>> --- a/drivers/media/dvb-frontends/si2168.c
>> +++ b/drivers/media/dvb-frontends/si2168.c
>> @@ -582,7 +582,7 @@ static int si2168_sleep(struct dvb_frontend *fe)
>> dev->active = false;
>>
>> /* Firmware B 4.0-11 or later loses warm state during sleep */
>> - if (dev->version > ('B' << 24 | 4 << 16 | 0 << 8 | 11 << 0))
>> + if (dev->version >= ('B' << 24 | 4 << 16 | 0 << 8 | 11 << 0))
>> dev->warm = false;
>>
>> memcpy(cmd.args, "\x13", 1);
>> --
>> 2.9.4
>>
>
>
>
> Thanks,
> Mauro
>
regards
Antti
--
http://palosaari.fi/
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Fix for hanging si2168 in PCTV 292e, making the code match
2018-03-07 13:23 ` Antti Palosaari
@ 2018-03-07 14:02 ` Mauro Carvalho Chehab
0 siblings, 0 replies; 4+ messages in thread
From: Mauro Carvalho Chehab @ 2018-03-07 14:02 UTC (permalink / raw)
To: Antti Palosaari; +Cc: Nigel Kettlewell, linux-media, linux-kernel
Em Wed, 7 Mar 2018 15:23:52 +0200
Antti Palosaari <crope@iki.fi> escreveu:
> On 12/14/2017 04:48 PM, Mauro Carvalho Chehab wrote:
> > Em Tue, 19 Sep 2017 13:13:52 +0100
> > Nigel Kettlewell <nigel.kettlewell@googlemail.com> escreveu:
> >
> >> [re-sending as plain text]
> >>
> >> Fix for hanging si2168 in PCTV 292e USB, making the code match the comment.
> >>
> >> Using firmware v4.0.11 the 292e would work once and then hang on
> >> subsequent attempts to view DVB channels, until physically unplugged and
> >> plugged back in.
> >>
> >> With this patch, the warm state is reset for v4.0.11 and it appears to
> >> work both on the first attempt and on subsequent attempts.
>
> It is comment which is wrong. With firmware 4.0.11 it works well without
> need of download it every time. But firmware 4.0.19 needs to be
> downloaded every time after device is put to sleep.
Ok. Let's then apply the enclosed patch (or something similar to it).
> Probably your issue is coming from some other reason.
I've no idea why Nigel proposed this patch. From my side, all I
want is to update the status of this patch:
https://patchwork.linuxtv.org/patch/44304/
and keep cleaning up my patch queue ;-)
Thanks,
Mauro
[PATCH] media: si2168: fix a comment about firmware version
There's a comment there at s82168 that it is wrong.
With firmware 4.0.11, sleep/resume works well without
need of download it every time. But firmware 4.0.19
needs to be downloaded again after sleep.
Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com>
diff --git a/drivers/media/dvb-frontends/si2168.c b/drivers/media/dvb-frontends/si2168.c
index a91947784842..324493e05f9f 100644
--- a/drivers/media/dvb-frontends/si2168.c
+++ b/drivers/media/dvb-frontends/si2168.c
@@ -614,7 +614,7 @@ static int si2168_sleep(struct dvb_frontend *fe)
if (ret)
goto err;
- /* Firmware B 4.0-11 or later loses warm state during sleep */
+ /* Firmware later than B 4.0-11 loses warm state during sleep */
if (dev->version > ('B' << 24 | 4 << 16 | 0 << 8 | 11 << 0))
dev->warm = false;
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2018-03-07 14:02 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-09-19 12:13 [PATCH] Fix for hanging si2168 in PCTV 292e, making the code match Nigel Kettlewell
2017-12-14 14:48 ` Mauro Carvalho Chehab
2018-03-07 13:23 ` Antti Palosaari
2018-03-07 14:02 ` Mauro Carvalho Chehab
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).