Re: szap sets FE_DISEQC_SEND_BURST incorrectly

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Reinhard Nissl wrote:
> Hi,
> 
> I'm not sure whether this is the right place to report an issue
> regarding szap, so please be so kind and point me to the proper place in
> case I'm wrong.
> 
> Please have a look into szap.c, function diseqc(). There is this
> function call:
> 
>    diseqc_send_msg(secfd, pol_vert ? SEC_VOLTAGE_13 : SEC_VOLTAGE_18,
>                    &cmd, hi_band ? SEC_TONE_ON : SEC_TONE_OFF,
>                    (sat_no / 4) % 2 ? SEC_MINI_B : SEC_MINI_A);
> 
> I think it should read:
> 
>    diseqc_send_msg(secfd, pol_vert ? SEC_VOLTAGE_13 : SEC_VOLTAGE_18,
>                    &cmd, hi_band ? SEC_TONE_ON : SEC_TONE_OFF,
>                    sat_no % 2 ? SEC_MINI_B : SEC_MINI_A);
> 
> As sat_no will typically be in the range from 0 to 3, the original
> statement will never use SEC_MINI_B for sat_no 1 respectively 3.

Thanks, committed.

Oliver

-- 
--------------------------------------------------------
VDR Remote Plugin 0.3.9 available at
http://www.escape-edv.de/endriss/vdr/
--------------------------------------------------------


_______________________________________________
linux-dvb mailing list
linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb

[Index of Archives]     [Linux Media]     [Video 4 Linux]     [Asterisk]     [Samba]     [Xorg]     [Xfree86]     [Linux USB]

  Powered by Linux