SPTS-Fix für Avia500 Boxen

[PauleFoul]

Das kacksen ist immer wenn das Bild schwarz ist. Meist kurz bevor das Bild vom neuen Sender sichtbar ist.

Wenn ich "Mute" aktiv habe knackst nichts und bei Lautstärke 0 auch nichts...

PS: Wenn man über einen Sender zappt der nicht verfügbar ist hat man danach keinen Ton mehr. Einmal Mute / Unmute hilft...

[seife]

ok, und normalerweise (ohne den patch), knackst es da während oder eher nach dem zappen?

[PauleFoul]

ok, und normalerweise (ohne den patch), knackst es da während oder eher nach dem zappen?

Also immer wenn das Bild schwarz ist. Egal ob mit oder ohne Patch. Zeitpunkte ähnlich... Mit dem Patch tendenziell etwas weniger...

[Steffi]

So, ich konnte den zapit-Patch nun auch testen. Leider mit negativem Ergebnis. Ich kann keinen Unterschied feststellen.

Ich glaube auch nicht, dass es so trivial ist und mit einem Mute on/off behoben werden kann.

Es knackst nicht während des Umschaltens, sondern danach. Nach dem Umschalten gibt es für ca. 1 bis 2 Sekunden ein Schwarzbild und während dieser Zeit knackst es aus den Lautsprechern - mal mehr, mal weniger. Besonders schlimm ist es beim Umschalten auf verschlüsselte Sender, wahrscheinlich weil hier erst noch decodiert werden muss. Zieht man auf einem verschlüsselten Sender die Smartcard, so dass gar nicht entschlüsselt werden kann, knackst es ununterbrochen bis man wieder auf einen Sender schaltet, der decodiert werden kann.

Vor Mitte März konnte mit dem Patch der avia_av_core dieses Knacksen (und auch das mehr oder minder lange Schwarzbild) beim Umschalten komplett eliminiert werden. Mit anderen Worten: Mit dem Patch gab es nicht nur ein tonloses Umschalten, sondern auch ein viel flüssigeres Umschalten!

Ich habe keine Ahnung, ob das Schwarzbild-Problem auftritt, weil der Ton nicht schnell genug syncronisiert werden kann oder ob der Ton nicht hinterherkommt, weil das Bild nicht schnell genug syncronisiert wird. Ich bin mir aber ziemlich sicher, dass beide Probleme miteinander zusammenhängen.

Steffi

[mrvica]

teste mal die avia_av.o aus dem Archiv, vollständigketshalber habe ich auch avia_gt.o beigefügt, sind für Kernel 2.4.34, der momentane Kernel ist aber 2.4.37.5, patchen geht somit nicht, du brauchst die abgeänderte rcS wo die zwei Treiber mit insmod -f geladen werden, denke die neue rcS muss nach /var/etc, bin mir aber nicht sicher, die Treiber kommen nach /var/lib/modules, habe nicht getestet ob die Treiber mit dem neuen Kernel harmonieren, möglich ist dass die Box gar nicht oder ohne Bild staret
http://www.file-upload.net/download-187 ... t.zip.html

mrvica

[Steffi]

@mrvica: Leider negativ! Ich baue in der Regel jffs2-Images, deshalb konnte ich die beiden Treiber von dir an Ort und Stelle überschreiben. rcS habe ich bei den Treibern mit -f modifiziert und neu gebootet. Erwartungsgemäß gibt es keinen Unterschied. Wie auch? Ob ich jetzt einen vorcompilierten Treiber nehme oder einen selbst compilierten, der Source wird mit Sicherheit gleich sein.

Es gibt von dem avia_av_core-Patch imho 2 Varianten: einmal so wie ihn Striper weiter oben eingefügt hat und zweitens einen, bei dem der sptsfix-Parameter eingebaut ist, so dass man über die Datei .sptsfix in /var/etc und eine angepasste rcS bestimmen kann, ob der Fix beim Booten wirksam werden soll oder nicht.

Steffi

[mrvica]

denke dass sptsfix in dem obigen Treiber immer an ist, hier habe ich noch einen älteren 2.4.32, habe die avia_av im Hexeditor gehabt, da steht was von sptsfix, du musst es aber beim Laden von avia_av als Parameter übergeben
http://www.file-upload.net/download-187 ... v.zip.html

...........
if [ -e /var/etc/.sptsfix ]; then
AVOPTS="${AVOPTS}sptsfix=1 "
fi
$IM $MD/avia_av firmware=/var/tuxbox/ucodes ${AVOPTS}
............

mrvica

[seife]

Steffi: probier es mal mit dem spts-fix und dann in zapit.cpp folgendes:

in der Funktion startPlayBack() den Abschnitt "/* start video */" (Zeile 2732 bis 2736) vor den Abschnitt "/* select audio output and start audio */" (Zeile 2721) verschieben.

Diese Reihenfolge hat sich beim controld-zapit-merge geändert. Evtl. ist es ja genau das.

[gugu]

Hat schon jemand versucht ohne diesen patch http://cvs.tuxbox-cvs.sourceforge.net/l ... 00100.html ?
Bzw funktioniert das auch nicht mehr mit neuen Neutrino mit alten drivers ?

[Steffi]

seife you are the man!

Die Umstellung in der zapit.cpp hat es tatsächlich gebracht. Der alte Patch funktioniert damit wieder.

Vielen Dank! Damit hast du nicht nur mich glücklich gemacht.

Steffi
(die endlich wieder auf aktuellem Stand ist)

PS: Ein weiterer negativer Effekt, der jetzt wieder korrigiert ist: Nach dem Booten hatte ich bei aktuellen Images immer ein Standbild, das erst nach einem Umschalten behoben wurde. Auch das ist jetzt repariert!

[Striper]

Danke seife, dann kann ich den Fix auch wieder verwenden nun. Der Hammer wäre, wenn jetzt noch einer den SPTS Fix schaltbar machen würde in Neutrino.

[rhabarber1848]

Es gibt von dem avia_av_core-Patch imho 2 Varianten: einmal so wie ihn Striper weiter oben eingefügt hat und zweitens einen, bei dem der sptsfix-Parameter eingebaut ist, so dass man über die Datei .sptsfix in /var/etc und eine angepasste rcS bestimmen kann, ob der Fix beim Booten wirksam werden soll oder nicht.

Wenn es dazu Sourcecode gibt, dürfte der Einbau dieser
Funktion keine unüberwindliche Hürde darstellen.

[rhabarber1848]

Andere Frage, der Patch nutzt

Code: Alles auswählen

if (aviarev)

aviarev ist true, wenn ein Avia500-Chip gefunden wurde.
Ich teste besagten Patch gerade auf meiner Sagem Avia600-Box,
dort läuft er ohne Probleme, da der Avia500-Code dort nicht
ausgeführt wird. Spricht etwas dagegen, den sptsfix so ins CVS
zu committen, dass er auf allen Avia500-Boxen ausgeführt wird?

Hier der Patch, der auch den zapit-Patch beinhaltet: sptsfix.diff

[GetAway]

Meine 500er läuft auch ohne Patch.
Da keiner genau weiss was der Patch überhaupt macht,
und die meisten Boxen auch ohne Fix laufen, sollte er schaltbar
gemacht werden, für die die ihn benötigen.

Edit:
Ich teste gerade, scheint auch bei problemlosen Boxen keine
negativen Auswirkungen zu haben. Schaltbar könnte man es
auch später noch machen. Im Endeffekt kommen eh Rückmeldungen,
wenn es bei irgendeinem nicht hinhaut.

[rhabarber1848]

Die Umstellung in der zapit.cpp hat es tatsächlich gebracht. Der alte Patch funktioniert damit wieder.

Könntest Du meinen Patch nochmal testen?

[Steffi]

@rhabarber1848: Ich komme erst heute abend zum Testen.

Das Schaltbarmachen ist kein Problem. Ich habe es hier seit vielen Monaten schaltbar.

Hier ist ein Diff gegen avia_av_core.c im cvs (mit sptsfix-Parameter):

Code: Alles auswählen

--- avia_av_core.c	2007-09-08 12:13:12.000000000 +0200
+++ avia_av_core_patched.c	2009-09-07 10:50:09.000000000 +0200
@@ -51,6 +51,7 @@
 
 TUXBOX_INFO(dbox2_gt);
 
+static int sptsfix = 0;
 static int tv_standard;
 static int no_watchdog;
 static char *firmware;
@@ -1229,6 +1230,12 @@
 			avia_av_cmd(SelectStream, 0x00, (play_state_video == AVIA_AV_PLAY_STATE_PLAYING) ? pid_video : 0xFFFF);
 			avia_av_cmd(Play, 0x00, (play_state_video == AVIA_AV_PLAY_STATE_PLAYING) ? pid_video : 0xFFFF, pid_audio);
 		}
+		if (sptsfix == 1) {
+			if ((aviarev) && (play_state_audio != AVIA_AV_PLAY_STATE_PAUSED)) {
+                     	if (play_state_audio != AVIA_AV_PLAY_STATE_PLAYING)
+					avia_av_cmd(Play, 0x00, (play_state_video == AVIA_AV_PLAY_STATE_PLAYING) ? pid_video : 0xFFFF, pid_audio);
+			}
+		}
 		bypass_mode_changed = 0;
 		break;
 
@@ -1240,8 +1247,17 @@
 		dprintk("avia_av: stopping audio decoder\n");
 
 		if (play_state_video == AVIA_AV_PLAY_STATE_STOPPED) {
-			avia_av_dram_write(AV_SYNC_MODE, AVIA_AV_SYNC_MODE_NONE);
-			avia_av_cmd(NewChannel, 0x00, 0xFFFF, 0xFFFF);
+			if (sptsfix == 1) {
+				if (aviarev) {
+					avia_av_cmd(SelectStream, 0x03 - bypass_mode, 0xFFFF); // ???
+					avia_av_cmd(Abort, 0x00);
+				}
+			}
+
+ 			avia_av_dram_write(AV_SYNC_MODE, AVIA_AV_SYNC_MODE_NONE);
+
+			if (!sptsfix || !aviarev)
+				avia_av_cmd(NewChannel, 0x00, 0xFFFF, 0xFFFF);
 		}
 		else {
 			avia_av_cmd(SelectStream, 0x03 - bypass_mode, 0xFFFF);
@@ -1287,8 +1303,17 @@
 		dprintk("avia_av: stopping video decoder\n");
 
 		if (play_state_audio == AVIA_AV_PLAY_STATE_STOPPED) {
-			avia_av_dram_write(AV_SYNC_MODE, AVIA_AV_SYNC_MODE_NONE);
-			avia_av_cmd(NewChannel, 0x00, 0xFFFF, 0xFFFF);
+			if (sptsfix == 1) {
+				if (aviarev) {
+					avia_av_cmd(SelectStream, 0x00, 0xFFFF); // ???
+					avia_av_cmd(Abort, 0x00);
+				}
+			}
+
+ 			avia_av_dram_write(AV_SYNC_MODE, AVIA_AV_SYNC_MODE_NONE);
+
+			if (!sptsfix || !aviarev)
+				avia_av_cmd(NewChannel, 0x00, 0xFFFF, 0xFFFF);
 		}
 		else {
 			avia_av_cmd(SelectStream, 0x00, 0xFFFF);
@@ -1519,6 +1544,7 @@
 	int err;
 
 	printk(KERN_INFO "avia_av: $Id: avia_av_core.c,v 1.99 2006/01/08 21:36:22 carjay Exp $\n");
+	printk(KERN_INFO "avia_av: AVIA500 sptsfix is %s\n", sptsfix?"on":"off");
 
 	if ((tv_standard < AVIA_AV_VIDEO_SYSTEM_PAL) ||
 		(tv_standard > AVIA_AV_VIDEO_SYSTEM_NTSC))
@@ -1558,3 +1584,5 @@
 MODULE_PARM_DESC(tv_standard, "0: PAL, 1: NTSC");
 MODULE_PARM_DESC(no_watchdog, "0: wd enabled, 1: wd disabled");
 MODULE_PARM_DESC(firmware, "path to microcode");
+MODULE_PARM(sptsfix,"i");
+MODULE_PARM_DESC(sptsfix, "0: OFF, 1: ON");

Der Parameter ist abhängig von der Flag-Datei .sptsfix in /var/etc und wird in der rcS beim Booten abgefragt:

Code: Alles auswählen

AVOPTS=""
IM=/sbin/insmod
MD=/lib/modules/$(uname -r)/misc

if [ -e /var/etc/.no_watchdog ]; then
	AVOPTS="no_watchdog=1 "
fi
if [ -e /var/etc/.sptsfix ]; then
	AVOPTS="${AVOPTS}sptsfix=1 "
fi
$IM -f $MD/avia_av.o firmware=/var/tuxbox/ucodes $AVOPTS

Das eigentliche Schalten muss dann bei den Parametern in neutrino_menu.cpp (inkl. settings.h, locals.h und locals_intern.h) realisiert werden. Die Diffs erspare ich mir hier. Das könnt ihr viel besser als ich.

Steffi

[rhabarber1848]

gesammelte Werke: EDIT: Patch ist im CVS
- SPTS-Fix für avia_av_core.c, parametrisiert
- zapit-Umschalte-Fix
- Neutrino-Konfiguration der marker-Datei /var/etc/.sptsfix
- rcS-Unterstützung für marker-Datei /var/etc/.sptsfix

Patch aktualisiert:
- Fehler in Neutrino-Konfiguration behoben
- Kernel-Meldungen aufgeräumt

[Gaucho316]

Im JtG-Image ist der SPTS-Fix schon seit Ewigkeiten drin. Soweit ich mich erinnern kann, ist der nur bis jetzt nicht ins CVS gewandert, weil das ganze wohl ein ziemlich übler Hack sein soll, den einige nicht im CVS sehen wollen/wollten.

[rhabarber1848]

OK, dann werde ich den Patch trotz etwaiger positiver
Rückmeldungen erst committen, wenn das Thema
durchdiskutiert wurde. Ich habe dazu keine Meinung,
da ich keine Avia500-Box habe

[rhabarber1848]

Gibt es Probleme mit dem Movieplayer, wenn der SPTS-Fix
auf Avia500 aktiviert wurde?
http://forum.tuxbox-cvs.sourceforge.net ... 60#p348160

EDIT:
vom 13.03.2006: http://forum.tuxbox-cvs.sourceforge.net ... 66#p293366

Denn der einzige Nebeneffekt den ich festgestellt habe ist, dass der Movieplayer nun nach der Pause nicht mehr ohne Resync im Bild weiterläuft.

Nochmals bestätigt: http://forum.tuxbox-cvs.sourceforge.net ... 92#p301692
und nochmal: http://forum.tuxbox-cvs.sourceforge.net ... 26#p301626

der SPTS-Fix AVIA 500 auf EIN ist der Grund bei Pause für das "Standbild" mit funktionierenden Ton.

Alles Meldungen aus 2006, ist das noch aktuell?

[Striper]

Ist doch egal ob das ein übler Hack ist oder nicht. So lange es funktioniert passt doch alles. :)
Schaltbar ist es ja auch, also ich seh da keine Probleme. Ganz im Gegenteil, da werden sich viele freuen wenn das endlich ins CVS übernommen wird.

[Steffi]

Es kann schon sein, dass es beim Movieplayer bei aktiviertem sptsfix die genannten Probleme gibt. Aber die sind für die geplagten Besitzer von 500er Boxen mit Sicherheit einfacher zu ertragen, als dieses lästige Knacksen und trägere Umschaltverhalten.

Wie oft schaltet man um und wie oft drückt man im Vergleich dazu beim Movieplayer auf Pause/Restart?

Außerdem kann man das hängende Bild beim MP einfach durch Drücken von "0" wieder syncronisieren, während es für das Knackser-Problem außer diesem dirty Hack keine andere Abhilfe gibt.

Und wenn der Fix schaltbar und standardmäßig deaktiviert ist, sollten eigentlich alle zufrieden sein.

Beim JTG ist der Fix wie oben erwähnt schon seit Monaten enthalten und ich kann mich nicht erinnern, dass Riker von unzufriedenen Usern zum Entfernen aufgefordert wurde.

Steffi

[rhabarber1848]

gesammelte Werke: EDIT: Patch ist im CVS

Patch aktualisiert:
- Fehler in Neutrino-Konfiguration behoben
- Kernel-Meldungen aufgeräumt

Bitte Patch neu herunterladen und testen.
Hier funktioniert Neutrino noch

avia_av: $Id: avia_av_core.c,v 1.99 2006/01/08 21:36:22 carjay Exp $
avia_av: AVIA 600L found.

[seife]

Ich habe den zapit-Teil mal mit ausführlichem Kommentar (und "#ifndef TRIPLEDRAGON" ) committed, der ist unstrittig.

[rhabarber1848]

gesammelte Werke: EDIT: Patch ist im CVS

Patch aktualisiert
- zapit-Teil entfernt, da bereits im CVS