Tuner Signalwerte via neuem API ermitteln

3PO · 7. April 2017

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 ~ #

Alles anzeigen

BTW:
Was ist das für eine Frequenz?

Code

FREQUENCY = 4293575296

3PO · 7. April 2017

Zitat von 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.

....

Alles anzeigen

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

kls · 7. April 2017

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 · 8. April 2017

Zitat von 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 · 8. April 2017

Zitat von 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

Alles anzeigen

Damit sieht es dann so aus:

[Blockierte Grafik: 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 · 8. April 2017

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 · 8. April 2017

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

domml · 8. April 2017

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

kls · 8. April 2017

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;

Alles anzeigen

Klaus

kls · 8. April 2017

Zitat von jasminj

Hast du Ralph schon informiert?

Nein, ist der Treiber denn von ihm?

Zitat

Oder mag nst aktiv werden?

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

Klaus

seahawk1986 · 8. April 2017

Zitat von 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 · 8. April 2017

Zitat von 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 · 8. April 2017

>> 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 · 8. April 2017

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;

Alles anzeigen

Klaus

wirbel · 8. April 2017

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

3PO · 8. April 2017

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 · 8. April 2017

Zitat von 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 · 8. April 2017

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 · 8. April 2017

Zu funktionieren schneit es aber:

[Blockierte Grafik: http://imagizer.imageshack.us/v2/280x200q90/922/YAMfdu.jpg]

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

kls · 8. April 2017

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;

Alles anzeigen

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

Klaus

Tuner Signalwerte via neuem API ermitteln

Jetzt mitmachen!

Teilen