Vorzeichen in stb0899_read_snr()

kls · 18. Februar 2013

Nachdem mit meinen TT-budget S2-3200 Karten bei DVB-S2-Kanälen immer deine deutlich niedrigere Signalqualität angezeigt wird als bei DVB-S, habe ich mir mal die Funktion stb0899_read_snr() im Treiber (linux/drivers/media/dvb-frontends/stb0899_drv.c) näher angeschaut.

Code

static int stb0899_read_snr(struct dvb_frontend *fe, u16 *snr)
{
        ...
        unsigned int val, quant, quantn = -1, est, estn = -1;
        ...
        switch (state->delsys) {
        ...
        case SYS_DVBS2:
                if (internal->lock) {
                        reg = STB0899_READ_S2REG(STB0899_S2DEMOD, UWP_CNTRL1);
                        quant = STB0899_GETFIELD(UWP_ESN0_QUANT, reg);
                        reg = STB0899_READ_S2REG(STB0899_S2DEMOD, UWP_STAT2);
                        est = STB0899_GETFIELD(ESN0_EST, reg);
                        if (est == 1)
                                val = 301; /* C/N = 30.1 dB */
                        else if (est == 2)
                                val = 270; /* C/N = 27.0 dB */
                        else {
                                /* quantn = 100 * log(quant^2) */
                                quantn = stb0899_table_lookup(stb0899_quant_tab, ARRAY_SIZE(stb0899_quant_tab) - 1, quant * 100);
                                /* estn = 100 * log(est) */
                                estn = stb0899_table_lookup(stb0899_est_tab, ARRAY_SIZE(stb0899_est_tab) - 1, est);
                                /* snr(dBm/10) = -10*(log(est)-log(quant^2)) => snr(dBm/10) = (100*log(quant^2)-100*log(est))/10 */
                                val = (quantn - estn) / 10;
                        }
                        *snr = val;
                        ...
                }
                break;
        ...
        }
        return 0;
}

Alles anzeigen

Ohne jetzt im Einzelnen zu wissen, was "quant" und "est" genau ist, mache ich mal folgende Überlegungen:
- quantn über die Tabelle stb0899_quant_tab[] ermittelt und kann Werte zwischen 0 und 4810 annehmen.
- estn wird über stb0899_est_tab[] ermittelt und liegt zwischen 0 und 5721.
- val wird aus quantn und estn berechnet und kann in den 4 möglichen Extremfällen folgende Werte annehmen:
(0 - 0) / 10 = 0
(0 - 5721) / 10 = -572.1
(4810 - 0) / 10 = 481.0
(4810 - 5721) / 10 = -91.1

Wie man sieht kann das Ergebnis gerne mal negativ sein. Dennoch wird es einer "unsigned"-Variablen und dann auch einem "unsigned"-Rückgabeparameter zugewiesen.

Ist hier irgendwas fundamental im Argen, oder sehe ich da was nicht?

Klaus

Emma53 · 18. Februar 2013

Hi KLS
Sei froh das Du ueberhaupt was im femon angezeigt bekommst
Mit meiner Satelco EasyWatch S2 ebenfalls mit STB0899 Chipsatz
zeigt mir das femon ueberhaupt nichts an und steigt beim Kanalwechsel oft total aus.
Es gab mal 2 Patches ftp://ftp.tvdr.de/vdr/Developer/Driver-Patches/
Index von ftp://ftp.tvdr.de/vdr/Developer/Driver-Patches/

[Blockierte Grafik: moz-icon://.diff?size=16]02a-stb0899_signal_flag.diff
1 KB
23.04.2012
00:00:00

[Blockierte Grafik: moz-icon://.diff?size=16]04-stb0899-ber_no_msleep.diff
2 KB
23.04.2012
00:00:00

aber die sind im neuen yavdr nicht mehr anwendbar.
Konnte keine Lösung bisher für das leidige Problem finden..

mfg emma53

kls · 18. Februar 2013

Zitat von Emma53

Hi KLS
Sei froh das Du ueberhaupt was im femon angezeigt bekommst

Ich benutze nicht femon, sondern die Anzeige in der LCARS Skin.

Zitat

Mit meiner Satelco EasyWatch S2 ebenfalls mit STB0899 Chipsatz
zeigt mir das femon ueberhaupt nichts an und steigt beim Kanalwechsel oft total aus.
Es gab mal 2 Patches ftp://ftp.tvdr.de/vdr/Developer/Driver-Patches/
Index von ftp://ftp.tvdr.de/vdr/Developer/Driver-Patches/

[Blockierte Grafik: moz-icon://.diff?size=16]02a-stb0899_signal_flag.diff
1 KB
23.04.2012
00:00:00

[Blockierte Grafik: moz-icon://.diff?size=16]04-stb0899-ber_no_msleep.diff
2 KB
23.04.2012
00:00:00

aber die sind im neuen yavdr nicht mehr anwendbar.

Alles anzeigen

Zumindest in dem Treiber den ich verwende (http://linuxtv.org/hg/~endriss/media_build_experimental) sind diese Patches bereits drin.

Inzwischen bin ich auf das hier gestoßen: https://github.com/OpenELEC/OpenELEC.tv/pull/1733/files.
Sieht so aus als würde der stb0899 im DVB-S und -S2 Fall unterschiedliche Maximalwerte liefern. Muß ich mir noch näher anschauen.
Mann ist das alles verkorkst... :-/

Klaus

UFO · 20. Februar 2013

Zitat von kls

Nachdem mit meinen TT-budget S2-3200 Karten bei DVB-S2-Kanälen immer deine deutlich niedrigere Signalqualität angezeigt wird als bei DVB-S, habe ich mir mal die Funktion stb0899_read_snr() im Treiber (linux/drivers/media/dvb-frontends/stb0899_drv.c) näher angeschaut.

Code

static int stb0899_read_snr(struct dvb_frontend *fe, u16 *snr)
{
        ...
        unsigned int val, quant, quantn = -1, est, estn = -1;
        ...
        switch (state->delsys) {
        ...
        case SYS_DVBS2:
                if (internal->lock) {
                        reg = STB0899_READ_S2REG(STB0899_S2DEMOD, UWP_CNTRL1);
                        quant = STB0899_GETFIELD(UWP_ESN0_QUANT, reg);
                        reg = STB0899_READ_S2REG(STB0899_S2DEMOD, UWP_STAT2);
                        est = STB0899_GETFIELD(ESN0_EST, reg);
                        if (est == 1)
                                val = 301; /* C/N = 30.1 dB */
                        else if (est == 2)
                                val = 270; /* C/N = 27.0 dB */
                        else {
                                /* quantn = 100 * log(quant^2) */
                                quantn = stb0899_table_lookup(stb0899_quant_tab, ARRAY_SIZE(stb0899_quant_tab) - 1, quant * 100);
                                /* estn = 100 * log(est) */
                                estn = stb0899_table_lookup(stb0899_est_tab, ARRAY_SIZE(stb0899_est_tab) - 1, est);
                                /* snr(dBm/10) = -10*(log(est)-log(quant^2)) => snr(dBm/10) = (100*log(quant^2)-100*log(est))/10 */
                                val = (quantn - estn) / 10;
                        }
                        *snr = val;
                        ...
                }
                break;
        ...
        }
        return 0;
}

Alles anzeigen

Ohne jetzt im Einzelnen zu wissen, was "quant" und "est" genau ist, mache ich mal folgende Überlegungen:
- quantn über die Tabelle stb0899_quant_tab[] ermittelt und kann Werte zwischen 0 und 4810 annehmen.
- estn wird über stb0899_est_tab[] ermittelt und liegt zwischen 0 und 5721.
- val wird aus quantn und estn berechnet und kann in den 4 möglichen Extremfällen folgende Werte annehmen:
(0 - 0) / 10 = 0
(0 - 5721) / 10 = -572.1
(4810 - 0) / 10 = 481.0
(4810 - 5721) / 10 = -91.1

Wie man sieht kann das Ergebnis gerne mal negativ sein. Dennoch wird es einer "unsigned"-Variablen und dann auch einem "unsigned"-Rückgabeparameter zugewiesen.

Ist hier irgendwas fundamental im Argen, oder sehe ich da was nicht?

Alles anzeigen

Ist halt die Frage, ob die beiden Werte nicht evtl. zusammenhängen, so daß ein solcher Fall gar nicht auftreten kann.
Habe leider kein Datenblatt. Frage doch einfach den Treiberautor (Manu Abraham).

Btw, daß bei DVB-S ein anderes SNR herauskommt als bei DVB-S2, überrascht mich nicht. Es sind schließlich zwei verschiedene Modulationsverfahren, die - salopp gesagt - den Datenkanal unterschiedlich ausnutzen...

CU
Oliver

kls · 20. Februar 2013

Zitat von UFO

Ist halt die Frage, ob die beiden Werte nicht evtl. zusammenhängen, so daß ein solcher Fall gar nicht auftreten kann.

Das kann natürlich sein.

Zitat

Habe leider kein Datenblatt. Frage doch einfach den Treiberautor (Manu Abraham).

Btw, daß bei DVB-S ein anderes SNR herauskommt als bei DVB-S2, überrascht mich nicht. Es sind schließlich zwei verschiedene Modulationsverfahren, die - salopp gesagt - den Datenkanal unterschiedlich ausnutzen...

Ich habe inzwischen mit den Werten von hier die Auswertung in cDvbTuner::GetSignalQuality() angepasst:

Diff

--- dvbdevice.c 2013/02/19 10:24:39     2.81
+++ dvbdevice.c 2013/02/19 11:55:00     2.82
@@ -621,15 +621,22 @@
               break;
               }
            }
+     uint16_t MinSnr = 0x0000;
      uint16_t MaxSnr = 0xFFFF; // Let's assume the default is using the entire range.
      // Use the subsystemId to identify individual devices in case they need
      // special treatment to map their Snr value into the range 0...0xFFFF.
      switch (subsystemId) {
-       case 0x13C21019: MaxSnr = 200; break; // TT-budget S2-3200 (DVB-S/DVB-S2)
+       case 0x13C21019: if (frontendType == SYS_DVBS2) { // TT-budget S2-3200 (DVB-S/DVB-S2)
+                           MinSnr = 10;
+                           MaxSnr = 70;
+                           }
+                        else
+                           MaxSnr = 200;
+                        break;
        case 0x20130245:                      // PCTV Systems PCTV 73ESE
        case 0x2013024F: MaxSnr = 255; break; // PCTV Systems nanoStick T2 290e
        }
-     int a = int(Snr) * 100 / MaxSnr;
+     int a = int(constrain(Snr, MinSnr, MaxSnr)) * 100 / (MaxSnr - MinSnr);
      int b = 100 - (Unc * 10 + (Ber / 256) * 5);
      if (b < 0)
         b = 0;

Alles anzeigen

Damit sieht es jetzt schon besser aus.

Klaus

Vorzeichen in stb0899_read_snr()

Jetzt mitmachen!

Teilen