Tuner Signalwerte via neuem API ermitteln

3PO · April 7, 2017 at 8:32 PM

Scheint wohl am Treiber zu liegen.

Mit einem anderen Treiber gibt dvb-fe-tool folgendes aus:

Code

vdr01_64 ~ # dvb-fe-tool -a1 -m -c5 -v
WARNING  Ignoring device /dev/dvb/adapter0/ci0
WARNING  Ignoring device /dev/dvb/adapter0/ci1
Device STV0910 (/dev/dvb/adapter1/frontend0) capabilities:
     CAN_2G_MODULATION
     CAN_FEC_AUTO
     CAN_INVERSION_AUTO
     CAN_QPSK
DVB API Version 5.10, Current v5 delivery system: DVBS2
Supported delivery systems:
     DVBS
    [DVBS2]
     DSS
Got parameters for DVBS2:
FREQUENCY = 4293575296
INVERSION = AUTO
SYMBOL_RATE = 27500000
INNER_FEC = 9/10
MODULATION = QPSK
PILOT = OFF
ROLLOFF = 35
POLARIZATION = OFF
STREAM_ID = 0
DELIVERY_SYSTEM = DVBS2
Lock   (0x1f) Signal= -9,30dBm C/N= 14,40dB postBER= 0
Lock   (0x1f) Signal= -9,30dBm C/N= 14,40dB postBER= 0
Lock   (0x1f) Signal= -9,30dBm C/N= 14,40dB postBER= 0
Lock   (0x1f) Signal= -9,30dBm C/N= 14,40dB postBER= 0
Lock   (0x1f) Signal= -9,29dBm C/N= 14,40dB postBER= 0
SEC: set voltage to OFF
ERROR    FE_SET_VOLTAGE: Die Operation ist nicht erlaubt
vdr01_64 ~ #

Display More

BTW:
Was ist das für eine Frequenz?

Code

FREQUENCY = 4293575296

3PO · April 7, 2017 at 10:13 PM

Quote from kls

Ich hab das jetzt mal eingebaut (siehe Patch).
Für Debug-Ausgaben können die Zeilen

//#define DEBUG_SIGNALSTRENGTH
//#define DEBUG_SIGNALQUALITY

aktiviert werden.

....

Wohin gehen die Debug-Ausgaben? Ich habe es mal aktiviert, aber im Syslog sehe ich keinen Unterschied.

kls · April 7, 2017 at 10:35 PM

Die Debug-Ausgaben gehen nach stderr.

Bis der Bug im Treiber gefixt ist kannst du in cDvbTuner::GetSignalStrength() vor Zeile 614 ("return Signal;") folgendes einfügen:

if (Signal > 0) Signal = -Signal;

Klaus

jasminj · April 8, 2017 at 12:16 AM

Quote from kls

Bis der Bug im Treiber gefixt ist

Hast du Ralph schon informiert?
Oder mag nst aktiv werden?
Oder soll ich was tun (obwohl ich mich da wieder erst einarbeiten müsste)?

LG,
Jasmin

3PO · April 8, 2017 at 9:28 AM

Quote from kls

Die Debug-Ausgaben gehen nach stderr.

Bis der Bug im Treiber gefixt ist kannst du in cDvbTuner::GetSignalStrength() vor Zeile 614 ("return Signal;") folgendes einfügen:

if (Signal > 0) Signal = -Signal;

Klaus

Damit sieht es dann so aus:

[Blocked Image: http://imagizer.imageshack.us/v2/280x200q90/924/GA6BRw.jpg]

Code

vdr01_64 ~ # dvb-fe-tool -a1 -m -c5
WARNING  Ignoring device /dev/dvb/adapter0/ci0
WARNING  Ignoring device /dev/dvb/adapter0/ci1
Lock   (0x1f) Signal= -9,32dBm C/N= 14,60dB postBER= 0
Lock   (0x1f) Signal= -9,32dBm C/N= 14,50dB postBER= 0
Lock   (0x1f) Signal= -9,32dBm C/N= 14,50dB postBER= 0
Lock   (0x1f) Signal= -9,32dBm C/N= 14,50dB postBER= 0
Lock   (0x1f) Signal= -9,32dBm C/N= 14,60dB postBER= 0
ERROR    FE_SET_VOLTAGE: Die Operation ist nicht erlaubt
vdr01_64 ~ #

wirbel · April 8, 2017 at 9:53 AM

der workaround müßte in dB1000toPercent() oder vor den Aufruf von db1000toPercent, z.B. so
Am Ende vor dem return fügt das sonst einen neuen Bug ein, falls scale == FE_SCALE_RELATIVE.

+int dB1000toPercent(int dB1000, int Low, int High)
+{
+ if (dB1000 > 0)
+ dB1000 = - dB1000;
+
+ // Convert the given value, which is in 1/1000 dBm, to a percentage in the
+ // range 0..100. Anything below Low is considered 0%, and anything above
+ // High counts as 100%.
+ if (dB1000 < Low)

Und der Treiber gibt einen 64bit signed anstelle von int (32bit) zurück.
Wäre sicherer wenn diese Funktion mit int64_t arbeitet.

domml · April 8, 2017 at 11:18 AM

Sollte es in dem Patch von Klaus nicht "if (Signal < 0)" heißen. Oder gleich Signal = abs(Signal) ?

domml · April 8, 2017 at 11:22 AM

Ok, der Wert scheint negativ zu sein, also Signal = - abs(Signal)

kls · April 8, 2017 at 12:22 PM

Man sollte einfach nicht unter Alkoholeinfluß Code schreiben ;-).

Was ich hier geschrieben hatte war natürlich Quatsch, denn damit wird ja der Prozentwert invertiert.

In dB1000toPercent macht es aber auch keinen Sinn, da das für CNR auch mit positiven Werten aufgerufen wird.

Das hier sollte ein gangbarer Workaround sein:

Diff

--- dvbdevice.c 2017-04-06 19:02:35.000000000 +0200
+++ dvbdevice.c 2017-04-08 12:20:46.395950048 +0200
@@ -600,6 +600,8 @@
       if (Props[0].u.st.len > 0) {
          switch (Props[0].u.st.stat[0].scale) {
            case FE_SCALE_DECIBEL:  Signal = dB1000toPercent(Props[0].u.st.stat[0].svalue, -95000, -20000); // TODO use different values for DVB-S, -T, -C?
+                                   if (Signal > 0)
+                                      Signal = -Signal;
                                    break;
            case FE_SCALE_RELATIVE: Signal = 100 * Props[0].u.st.stat[0].uvalue / 0xFFFF;
                                    break;

Display More

Klaus

kls · April 8, 2017 at 12:24 PM

Quote from jasminj

Hast du Ralph schon informiert?

Nein, ist der Treiber denn von ihm?

Quote

Oder mag nst aktiv werden?

Die Interna des Treibers sind nicht so mein Gebiet...

Klaus

seahawk1986 · April 8, 2017 at 12:28 PM

Quote from kls

Nur um Verwirrung zu vermeiden: der Nutzer nst ist seit einiger Zeit daran die Treiber für die aktuellen DD Karten in den Kernel zu bringen: Treiber der Cine-CTv6/DDBridge/CI in den Kernel integrieren

nst · April 8, 2017 at 2:02 PM

Quote from jasminj

Hast du Ralph schon informiert?
Oder mag nst aktiv werden?

Ralph wird passende Register-Doku (dürfte am interessantesten sein), passende Hardware und 'ne DVB-S(2) Empfangsanlage haben, und ist obendrein Autor des Codes, das sollte von ihm gefixt werden.

Ich kann hier nicht viel machen, ausser Probleme mit der Code-Logik zu flicken, nehme aber gerne Patches in Form von Pullrequests via GitHub in die Kernel-Sourcen mit (das einsortieren in alle betroffenen Branches mache ich dann schon). Als Tipp - Wer das Thema verfolgen mag:
drivers/media/dvb-frontends/stv0910.c: static int read_signal_strength(...) ermittelt und liefert die SIgnalstärke. Da gibts aktuell noch ein Vorzeichenproblem, welches dieser Commit fixen könnte, das müsste aber jemand anderes mit passender Hardware verifizieren.

wirbel · April 8, 2017 at 7:48 PM

>> Das hier sollte ein gangbarer Workaround sein:

Du müsstest das erste Argument von dB1000toPercent invertieren, nicht das Ergebnis, ansonsten laufen diese Vergleiche ins Leere:

+ if (dB1000 < Low)
+ return 0;
+ if (dB1000 > High)
+ return 100;

kls · April 8, 2017 at 8:09 PM

O Mann, manchmal ist das Brett vorm Kopf einfach zu dick ;-).

Diff

--- dvbdevice.c 2017-04-06 19:02:35.000000000 +0200
+++ dvbdevice.c 2017-04-08 20:07:02.971746204 +0200
@@ -599,7 +599,7 @@
       int Signal = 0;;
       if (Props[0].u.st.len > 0) {
          switch (Props[0].u.st.stat[0].scale) {
-           case FE_SCALE_DECIBEL:  Signal = dB1000toPercent(Props[0].u.st.stat[0].svalue, -95000, -20000); // TODO use different values for DVB-S, -T, -C?
+           case FE_SCALE_DECIBEL:  Signal = dB1000toPercent(-(abs(Props[0].u.st.stat[0].svalue), -95000, -20000); // TODO use different values for DVB-S, -T, -C?
                                    break;
            case FE_SCALE_RELATIVE: Signal = 100 * Props[0].u.st.stat[0].uvalue / 0xFFFF;
                                    break;

Display More

Klaus

wirbel · April 8, 2017 at 8:17 PM

Das ist 1:1 die Lösung die ich vorgeschlagen hätte..

3PO · April 8, 2017 at 8:31 PM

Nur bauen lässt sich der VDR mit dem letzten Patch nicht mehr.

Code

vdr01_64 vdr-2.3.3 # make
g++ -Werror=overloaded-virtual -Wno-parentheses -march=core-avx2 -O2 -pipe -g -ggdb -O0 -D__STDC_CONSTANT_MACROS -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -fPIC -fPIC -c -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -D_LARGEFILE64_SOURCE -DREMOTE_KBD -DLIRC_DEVICE="/var/run/lirc/lircd" -DVIDEODIR="/video" -DCONFDIR="/etc/vdr" -DARGSDIR="/etc/vdr/conf.d" -DCACHEDIR="/var/cache/vdr" -DRESDIR="/etc/vdr" -DPLUGINDIR="/usr/lib/vdr/plugins" -DLOCDIR="/etc/vdr/locale" -I/usr/include/freetype2 -I/usr/include/linux/dvb -o dvbdevice.o dvbdevice.c
dvbdevice.c: In Elementfunktion »int cDvbTuner::GetSignalStrength() const«:
dvbdevice.c:602:113: Fehler: expected »)« before »;« token
            case FE_SCALE_DECIBEL:  Signal = dB1000toPercent(-(abs(Props[0].u.st.stat[0].svalue), -95000, -20000); // TODO use different values for DVB-S, -T, -C?
                                                                                                                 ^
make: *** [Makefile:124: dvbdevice.o] Fehler 1
vdr01_64 vdr-2.3.3 #

CafeDelMar · April 8, 2017 at 8:34 PM

Quote from 3PO

Nur bauen lässt sich der VDR mit dem letzten Patch nicht mehr.

Ich vermute, hinter .svalue fehlt eine zweite "runde Klammer zu".

CafeDelMar

EDIT: Also so:

Code

--- dvbdevice.c 2017-04-06 19:02:35.000000000 +0200+++ dvbdevice.c 2017-04-08 20:07:02.971746204 +0200
@@ -599,7 +599,7 @@
       int Signal = 0;;
       if (Props[0].u.st.len > 0) {
          switch (Props[0].u.st.stat[0].scale) {
-           case FE_SCALE_DECIBEL:  Signal = dB1000toPercent(Props[0].u.st.stat[0].svalue, -95000, -20000); // TODO use different values for DVB-S, -T, -C?
+           case FE_SCALE_DECIBEL:  Signal = dB1000toPercent(-(abs(Props[0].u.st.stat[0].svalue)), -95000, -20000); // TODO use different values for DVB-S, -T, -C?
                                    break;
            case FE_SCALE_RELATIVE: Signal = 100 * Props[0].u.st.stat[0].uvalue / 0xFFFF;                                     break;

3PO · April 8, 2017 at 8:39 PM

Nun, dass eine schließende Klammer fehlt, sagt ja schon die Error Message, nur bin ich mir nicht sicher, ob sie hinter ".svalue" gehört.

3PO · April 8, 2017 at 8:54 PM

Zu funktionieren schneit es aber:

[Blocked Image: http://imagizer.imageshack.us/v2/280x200q90/922/YAMfdu.jpg]

dvb-fe-tool -v -a1 -f0 -m -c5

Code

vdr01_64 ~ # dvb-fe-tool -v -a1 -f0 -m -c5
WARNING  Ignoring device /dev/dvb/adapter0/ci0
WARNING  Ignoring device /dev/dvb/adapter0/ci1
Device STV0910 (/dev/dvb/adapter1/frontend0) capabilities:
     CAN_2G_MODULATION
     CAN_FEC_AUTO
     CAN_INVERSION_AUTO
     CAN_QPSK
DVB API Version 5.10, Current v5 delivery system: DVBS2
Supported delivery systems:
     DVBS
    [DVBS2]
     DSS
Got parameters for DVBS2:
FREQUENCY = 4293653296
INVERSION = AUTO
SYMBOL_RATE = 27500000
INNER_FEC = 9/10
MODULATION = QPSK
PILOT = OFF
ROLLOFF = 35
POLARIZATION = OFF
STREAM_ID = 0
DELIVERY_SYSTEM = DVBS2
Lock   (0x1f) Signal= -31,27dBm C/N= 15,70dB preBER= 0
Lock   (0x1f) Signal= -31,28dBm C/N= 15,60dB preBER= 0
Lock   (0x1f) Signal= -31,34dBm C/N= 15,60dB preBER= 0
Lock   (0x1f) Signal= -31,32dBm C/N= 15,50dB preBER= 0
Lock   (0x1f) Signal= -31,27dBm C/N= 15,60dB preBER= 0
SEC: set voltage to OFF
ERROR    FE_SET_VOLTAGE: Die Operation ist nicht erlaubt
vdr01_64 ~ #

Display More

kls · April 8, 2017 at 9:02 PM

Streng genommen ist die öffnende Klammer vor dem 'abs' eine zuviel.
Diesmal aber auch hier testweise übersetzt:

Diff

--- dvbdevice.c 2017/04/06 17:02:35     4.5
+++ dvbdevice.c 2017/04/08 18:59:19
@@ -599,7 +599,7 @@
       int Signal = 0;;
       if (Props[0].u.st.len > 0) {
          switch (Props[0].u.st.stat[0].scale) {
-           case FE_SCALE_DECIBEL:  Signal = dB1000toPercent(Props[0].u.st.stat[0].svalue, -95000, -20000); // TODO use different values for DVB-S, -T, -C?
+           case FE_SCALE_DECIBEL:  Signal = dB1000toPercent(-abs(Props[0].u.st.stat[0].svalue), -95000, -20000); // TODO use different values for DVB-S, -T, -C?
                                    break;
            case FE_SCALE_RELATIVE: Signal = 100 * Props[0].u.st.stat[0].uvalue / 0xFFFF;
                                    break;

Display More

Sorry, hatte das nur nebenbei mit einem Auge schnell hingekritzelt und nicht selber ausprobiert - sollte man nicht machen...

Klaus

Participate now!