diff mbox

TT-budget S2-3200 cannot tune on HB13E DVBS2 transponder

Message ID 4DA63A66.1070300@gmx.net (mailing list archive)
State Rejected
Headers show

Commit Message

Lutz Sammer April 14, 2011, 12:05 a.m. UTC 
> On 05/04/11 21:07, Steffen Barszus wrote:
>> On Tue, 05 Apr 2011 13:00:14 +0200
>> "Issa Gorissen" <flop.m@xxxxxxx> wrote:
>>
>>> Hi,
>>>
>>> Eutelsat made a recent migration from DVB-S to DVB-S2 (since
>>> 31/3/2011) on two transponders on HB13E
>>>
>>> - HOT BIRD 6 13° Est TP 159 Freq 11,681 Ghz DVB-S2 FEC 3/4 27500
>>> Msymb/s 0.2 Pilot off Polar H
>>>
>>> - HOT BIRD 9 13° Est TP 99 Freq 12,692 Ghz DVB-S2 FEC 3/4 27500
>>> Msymb/s 0.2 Pilot off Polar H
>>>
>>>
>>> Before those changes, with my TT S2 3200, I was able to watch TV on
>>> those transponders. Now, I cannot even tune on those transponders. I
>>> have tried with scan-s2 and w_scan and the latest drivers from git.
>>> They both find the transponders but cannot tune onto it.
>>>
>>> Something noteworthy is that my other card, a DuoFlex S2 can tune
>>> fine on those transponders.
>>>
>>> My question is; can someone try this as well with a TT S2 3200 and
>>> post the results ?
>> i read something about it lately here (german!): 
>> http://www.vdr-portal.de/board16-video-disk-recorder/board85-hdtv-dvb-s2/p977938-stb0899-fec-3-4-tester-gesucht/#post977938
>>
>> It says in stb0899_drv.c function:
>> static void stb0899_set_iterations(struct stb0899_state *state) 
>>
>> This:
>> reg = STB0899_READ_S2REG(STB0899_S2DEMOD, MAX_ITER);
>> STB0899_SETFIELD_VAL(MAX_ITERATIONS, reg, iter_scale);
>> stb0899_write_s2reg(state, STB0899_S2DEMOD, STB0899_BASE_MAX_ITER, STB0899_OFF0_MAX_ITER, reg);
>>
>> should be replaced with this:
>>
>> reg = STB0899_READ_S2REG(STB0899_S2FEC, MAX_ITER);
>> STB0899_SETFIELD_VAL(MAX_ITERATIONS, reg, iter_scale);
>> stb0899_write_s2reg(state, STB0899_S2FEC, STB0899_BASE_MAX_ITER, STB0899_OFF0_MAX_ITER, reg);
>>
>> Basically replace STB0899_S2DEMOD with STB0899_S2FEC in this 2 lines
>> affected.
>>
>> Kind Regards 
>>
>> Steffen
> Hi Steffen,
> 
> Unfortunately, it does not help in my case. Thx anyway.

Try my locking fix. With above patch I can lock the
channels without problem.

Johns

Comments

Mauro Carvalho Chehab May 3, 2011, 11:16 p.m. UTC | #1 
Em 13-04-2011 21:05, Lutz Sammer escreveu:
>> On 05/04/11 21:07, Steffen Barszus wrote:
>>> On Tue, 05 Apr 2011 13:00:14 +0200
>>> "Issa Gorissen" <flop.m@xxxxxxx> wrote:
>>>
>>>> Hi,
>>>>
>>>> Eutelsat made a recent migration from DVB-S to DVB-S2 (since
>>>> 31/3/2011) on two transponders on HB13E
>>>>
>>>> - HOT BIRD 6 13° Est TP 159 Freq 11,681 Ghz DVB-S2 FEC 3/4 27500
>>>> Msymb/s 0.2 Pilot off Polar H
>>>>
>>>> - HOT BIRD 9 13° Est TP 99 Freq 12,692 Ghz DVB-S2 FEC 3/4 27500
>>>> Msymb/s 0.2 Pilot off Polar H
>>>>
>>>>
>>>> Before those changes, with my TT S2 3200, I was able to watch TV on
>>>> those transponders. Now, I cannot even tune on those transponders. I
>>>> have tried with scan-s2 and w_scan and the latest drivers from git.
>>>> They both find the transponders but cannot tune onto it.
>>>>
>>>> Something noteworthy is that my other card, a DuoFlex S2 can tune
>>>> fine on those transponders.
>>>>
>>>> My question is; can someone try this as well with a TT S2 3200 and
>>>> post the results ?
>>> i read something about it lately here (german!): 
>>> http://www.vdr-portal.de/board16-video-disk-recorder/board85-hdtv-dvb-s2/p977938-stb0899-fec-3-4-tester-gesucht/#post977938
>>>
>>> It says in stb0899_drv.c function:
>>> static void stb0899_set_iterations(struct stb0899_state *state) 
>>>
>>> This:
>>> reg = STB0899_READ_S2REG(STB0899_S2DEMOD, MAX_ITER);
>>> STB0899_SETFIELD_VAL(MAX_ITERATIONS, reg, iter_scale);
>>> stb0899_write_s2reg(state, STB0899_S2DEMOD, STB0899_BASE_MAX_ITER, STB0899_OFF0_MAX_ITER, reg);
>>>
>>> should be replaced with this:
>>>
>>> reg = STB0899_READ_S2REG(STB0899_S2FEC, MAX_ITER);
>>> STB0899_SETFIELD_VAL(MAX_ITERATIONS, reg, iter_scale);
>>> stb0899_write_s2reg(state, STB0899_S2FEC, STB0899_BASE_MAX_ITER, STB0899_OFF0_MAX_ITER, reg);
>>>
>>> Basically replace STB0899_S2DEMOD with STB0899_S2FEC in this 2 lines
>>> affected.
>>>
>>> Kind Regards 
>>>
>>> Steffen
>> Hi Steffen,
>>
>> Unfortunately, it does not help in my case. Thx anyway.
> 
> Try my locking fix. With above patch I can lock the
> channels without problem.

Can someone confirm that such patch would fix the issue? If so, please
forward it in a way that it could be applied (patch is currently line-wrapped),
and submit with some comments/description and your SOB.

As the patch is currently broken, I'm just marking it as rejected at patchwork.

Manu,

Please take a look on this trouble report.

Thanks!
Mauro.

> 
> Johns
> 
> diff --git a/drivers/media/dvb/frontends/stb0899_algo.c
> b/drivers/media/dvb/frontends/stb0899_algo.c
> index 2da55ec..55f0c4e 100644
> --- a/drivers/media/dvb/frontends/stb0899_algo.c
> +++ b/drivers/media/dvb/frontends/stb0899_algo.c
> @@ -338,36 +338,42 @@ static enum stb0899_status
> stb0899_check_data(struct stb0899_state *state)
>         int lock = 0, index = 0, dataTime = 500, loop;
>         u8 reg;
> 
> -       internal->status = NODATA;
> +       reg = stb0899_read_reg(state, STB0899_VSTATUS);
> +       lock = STB0899_GETFIELD(VSTATUS_LOCKEDVIT, reg);
> +       if ( !lock ) {
> 
> -       /* RESET FEC    */
> -       reg = stb0899_read_reg(state, STB0899_TSTRES);
> -       STB0899_SETFIELD_VAL(FRESACS, reg, 1);
> -       stb0899_write_reg(state, STB0899_TSTRES, reg);
> -       msleep(1);
> -       reg = stb0899_read_reg(state, STB0899_TSTRES);
> -       STB0899_SETFIELD_VAL(FRESACS, reg, 0);
> -       stb0899_write_reg(state, STB0899_TSTRES, reg);
> +               internal->status = NODATA;
> 
> -       if (params->srate <= 2000000)
> -               dataTime = 2000;
> -       else if (params->srate <= 5000000)
> -               dataTime = 1500;
> -       else if (params->srate <= 15000000)
> -               dataTime = 1000;
> -       else
> -               dataTime = 500;
> -
> -       stb0899_write_reg(state, STB0899_DSTATUS2, 0x00); /* force
> search loop  */
> -       while (1) {
> -               /* WARNING! VIT LOCKED has to be tested before
> VIT_END_LOOOP    */
> -               reg = stb0899_read_reg(state, STB0899_VSTATUS);
> -               lock = STB0899_GETFIELD(VSTATUS_LOCKEDVIT, reg);
> -               loop = STB0899_GETFIELD(VSTATUS_END_LOOPVIT, reg);
> +               /* RESET FEC    */
> +               reg = stb0899_read_reg(state, STB0899_TSTRES);
> +               STB0899_SETFIELD_VAL(FRESACS, reg, 1);
> +               stb0899_write_reg(state, STB0899_TSTRES, reg);
> +               msleep(1);
> +               reg = stb0899_read_reg(state, STB0899_TSTRES);
> +               STB0899_SETFIELD_VAL(FRESACS, reg, 0);
> +               stb0899_write_reg(state, STB0899_TSTRES, reg);
> 
> -               if (lock || loop || (index > dataTime))
> -                       break;
> -               index++;
> +                       msleep(1);
> +               }
>         }
> 
>         if (lock) {     /* DATA LOCK indicator  */
> 
> 
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Manu Abraham May 5, 2011, 4:25 a.m. UTC | #2 
On Wed, May 4, 2011 at 3:16 AM, Mauro Carvalho Chehab
<mchehab@redhat.com> wrote:
> Em 13-04-2011 21:05, Lutz Sammer escreveu:
>>> On 05/04/11 21:07, Steffen Barszus wrote:
>>>> On Tue, 05 Apr 2011 13:00:14 +0200
>>>> "Issa Gorissen" <flop.m@xxxxxxx> wrote:
>>>>
>>>>> Hi,
>>>>>
>>>>> Eutelsat made a recent migration from DVB-S to DVB-S2 (since
>>>>> 31/3/2011) on two transponders on HB13E
>>>>>
>>>>> - HOT BIRD 6 13° Est TP 159 Freq 11,681 Ghz DVB-S2 FEC 3/4 27500
>>>>> Msymb/s 0.2 Pilot off Polar H
>>>>>
>>>>> - HOT BIRD 9 13° Est TP 99 Freq 12,692 Ghz DVB-S2 FEC 3/4 27500
>>>>> Msymb/s 0.2 Pilot off Polar H
>>>>>
>>>>>
>>>>> Before those changes, with my TT S2 3200, I was able to watch TV on
>>>>> those transponders. Now, I cannot even tune on those transponders. I
>>>>> have tried with scan-s2 and w_scan and the latest drivers from git.
>>>>> They both find the transponders but cannot tune onto it.
>>>>>
>>>>> Something noteworthy is that my other card, a DuoFlex S2 can tune
>>>>> fine on those transponders.
>>>>>
>>>>> My question is; can someone try this as well with a TT S2 3200 and
>>>>> post the results ?
>>>> i read something about it lately here (german!):
>>>> http://www.vdr-portal.de/board16-video-disk-recorder/board85-hdtv-dvb-s2/p977938-stb0899-fec-3-4-tester-gesucht/#post977938
>>>>
>>>> It says in stb0899_drv.c function:
>>>> static void stb0899_set_iterations(struct stb0899_state *state)
>>>>
>>>> This:
>>>> reg = STB0899_READ_S2REG(STB0899_S2DEMOD, MAX_ITER);
>>>> STB0899_SETFIELD_VAL(MAX_ITERATIONS, reg, iter_scale);
>>>> stb0899_write_s2reg(state, STB0899_S2DEMOD, STB0899_BASE_MAX_ITER, STB0899_OFF0_MAX_ITER, reg);
>>>>
>>>> should be replaced with this:
>>>>
>>>> reg = STB0899_READ_S2REG(STB0899_S2FEC, MAX_ITER);
>>>> STB0899_SETFIELD_VAL(MAX_ITERATIONS, reg, iter_scale);
>>>> stb0899_write_s2reg(state, STB0899_S2FEC, STB0899_BASE_MAX_ITER, STB0899_OFF0_MAX_ITER, reg);
>>>>
>>>> Basically replace STB0899_S2DEMOD with STB0899_S2FEC in this 2 lines
>>>> affected.
>>>>
>>>> Kind Regards
>>>>
>>>> Steffen
>>> Hi Steffen,
>>>
>>> Unfortunately, it does not help in my case. Thx anyway.
>>
>> Try my locking fix. With above patch I can lock the
>> channels without problem.
>
> Can someone confirm that such patch would fix the issue? If so, please
> forward it in a way that it could be applied (patch is currently line-wrapped),
> and submit with some comments/description and your SOB.
>
> As the patch is currently broken, I'm just marking it as rejected at patchwork.
>
> Manu,
>
> Please take a look on this trouble report.
>
> Thanks!
> Mauro.


I am out of station currently. I will take a deeper look at it during
the weekend or next week.

Best Regards,
Manu
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Mauro Carvalho Chehab May 5, 2011, 11:05 a.m. UTC | #3 
Em 05-05-2011 01:25, Manu Abraham escreveu:
> On Wed, May 4, 2011 at 3:16 AM, Mauro Carvalho Chehab
> <mchehab@redhat.com> wrote:
>> Em 13-04-2011 21:05, Lutz Sammer escreveu:
>>>> On 05/04/11 21:07, Steffen Barszus wrote:
>>>>> On Tue, 05 Apr 2011 13:00:14 +0200
>>>>> "Issa Gorissen" <flop.m@xxxxxxx> wrote:
>>>>>
>>>>>> Hi,
>>>>>>
>>>>>> Eutelsat made a recent migration from DVB-S to DVB-S2 (since
>>>>>> 31/3/2011) on two transponders on HB13E
>>>>>>
>>>>>> - HOT BIRD 6 13° Est TP 159 Freq 11,681 Ghz DVB-S2 FEC 3/4 27500
>>>>>> Msymb/s 0.2 Pilot off Polar H
>>>>>>
>>>>>> - HOT BIRD 9 13° Est TP 99 Freq 12,692 Ghz DVB-S2 FEC 3/4 27500
>>>>>> Msymb/s 0.2 Pilot off Polar H
>>>>>>
>>>>>>
>>>>>> Before those changes, with my TT S2 3200, I was able to watch TV on
>>>>>> those transponders. Now, I cannot even tune on those transponders. I
>>>>>> have tried with scan-s2 and w_scan and the latest drivers from git.
>>>>>> They both find the transponders but cannot tune onto it.
>>>>>>
>>>>>> Something noteworthy is that my other card, a DuoFlex S2 can tune
>>>>>> fine on those transponders.
>>>>>>
>>>>>> My question is; can someone try this as well with a TT S2 3200 and
>>>>>> post the results ?
>>>>> i read something about it lately here (german!):
>>>>> http://www.vdr-portal.de/board16-video-disk-recorder/board85-hdtv-dvb-s2/p977938-stb0899-fec-3-4-tester-gesucht/#post977938
>>>>>
>>>>> It says in stb0899_drv.c function:
>>>>> static void stb0899_set_iterations(struct stb0899_state *state)
>>>>>
>>>>> This:
>>>>> reg = STB0899_READ_S2REG(STB0899_S2DEMOD, MAX_ITER);
>>>>> STB0899_SETFIELD_VAL(MAX_ITERATIONS, reg, iter_scale);
>>>>> stb0899_write_s2reg(state, STB0899_S2DEMOD, STB0899_BASE_MAX_ITER, STB0899_OFF0_MAX_ITER, reg);
>>>>>
>>>>> should be replaced with this:
>>>>>
>>>>> reg = STB0899_READ_S2REG(STB0899_S2FEC, MAX_ITER);
>>>>> STB0899_SETFIELD_VAL(MAX_ITERATIONS, reg, iter_scale);
>>>>> stb0899_write_s2reg(state, STB0899_S2FEC, STB0899_BASE_MAX_ITER, STB0899_OFF0_MAX_ITER, reg);
>>>>>
>>>>> Basically replace STB0899_S2DEMOD with STB0899_S2FEC in this 2 lines
>>>>> affected.
>>>>>
>>>>> Kind Regards
>>>>>
>>>>> Steffen
>>>> Hi Steffen,
>>>>
>>>> Unfortunately, it does not help in my case. Thx anyway.
>>>
>>> Try my locking fix. With above patch I can lock the
>>> channels without problem.
>>
>> Can someone confirm that such patch would fix the issue? If so, please
>> forward it in a way that it could be applied (patch is currently line-wrapped),
>> and submit with some comments/description and your SOB.
>>
>> As the patch is currently broken, I'm just marking it as rejected at patchwork.
>>
>> Manu,
>>
>> Please take a look on this trouble report.
>>
>> Thanks!
>> Mauro.
> 
> 
> I am out of station currently. I will take a deeper look at it during
> the weekend or next week.

Ok, thank you!

Mauro.
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/drivers/media/dvb/frontends/stb0899_algo.c
b/drivers/media/dvb/frontends/stb0899_algo.c
index 2da55ec..55f0c4e 100644
--- a/drivers/media/dvb/frontends/stb0899_algo.c
+++ b/drivers/media/dvb/frontends/stb0899_algo.c
@@ -338,36 +338,42 @@  static enum stb0899_status
stb0899_check_data(struct stb0899_state *state)
        int lock = 0, index = 0, dataTime = 500, loop;
        u8 reg;

-       internal->status = NODATA;
+       reg = stb0899_read_reg(state, STB0899_VSTATUS);
+       lock = STB0899_GETFIELD(VSTATUS_LOCKEDVIT, reg);
+       if ( !lock ) {

-       /* RESET FEC    */
-       reg = stb0899_read_reg(state, STB0899_TSTRES);
-       STB0899_SETFIELD_VAL(FRESACS, reg, 1);
-       stb0899_write_reg(state, STB0899_TSTRES, reg);
-       msleep(1);
-       reg = stb0899_read_reg(state, STB0899_TSTRES);
-       STB0899_SETFIELD_VAL(FRESACS, reg, 0);
-       stb0899_write_reg(state, STB0899_TSTRES, reg);
+               internal->status = NODATA;

-       if (params->srate <= 2000000)
-               dataTime = 2000;
-       else if (params->srate <= 5000000)
-               dataTime = 1500;
-       else if (params->srate <= 15000000)
-               dataTime = 1000;
-       else
-               dataTime = 500;
-
-       stb0899_write_reg(state, STB0899_DSTATUS2, 0x00); /* force
search loop  */
-       while (1) {
-               /* WARNING! VIT LOCKED has to be tested before
VIT_END_LOOOP    */
-               reg = stb0899_read_reg(state, STB0899_VSTATUS);
-               lock = STB0899_GETFIELD(VSTATUS_LOCKEDVIT, reg);
-               loop = STB0899_GETFIELD(VSTATUS_END_LOOPVIT, reg);
+               /* RESET FEC    */
+               reg = stb0899_read_reg(state, STB0899_TSTRES);
+               STB0899_SETFIELD_VAL(FRESACS, reg, 1);
+               stb0899_write_reg(state, STB0899_TSTRES, reg);
+               msleep(1);
+               reg = stb0899_read_reg(state, STB0899_TSTRES);
+               STB0899_SETFIELD_VAL(FRESACS, reg, 0);
+               stb0899_write_reg(state, STB0899_TSTRES, reg);

-               if (lock || loop || (index > dataTime))
-                       break;
-               index++;
+                       msleep(1);
+               }
        }

        if (lock) {     /* DATA LOCK indicator  */


--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html