Video Treiber für Odroid-N2+ (softhdodroid)

Quote from jojo61

PS: das interne grabben braucht dann nicht deinen Kernel wenn man mit den Bildfehlern leben kann. Z.B. für live ist das unerheblich. da braucht es dann nur den Kernel mit dem capture device.

Der crasht aber leider bei 4K UHD irgendwann. Für Full HD funktioniert er, hat aber ab und an Bildfehler, die man in den LEDs nicht sieht.

Edit: Kann es sein, dass die unterschiedlichen Verhalten wegen unterschiedlicher EDIDs/Auflösungen da sind? Ich habe noch einen Receiver dazwischen und dann einen 4K TV...

Quote from jojo61

So vdrboblight läuft nun und verbindet sich auch mit dem boblightd. Dem boblightd habe ich eine config gegeben mit seriellem output.

Soweit läuft das auch. Ich sehe nun das er regelmässig 64x64 Bildchen grabt. Und das läuft in HD und UHD ohne absturz.

Ich hänge meine video.c mal hier an. Da ist ein Debug drin der das syslog zumüllt, aber jeden grab anzeigt. Würde mich mal interessieren wie das bei euch aussieht.

Ich lasse den boblightd gar nicht laufen, sondern versuche mich zum remote boblight-server zu verbinden imdem ich die IP des Rechners in den plugin Einstellungen mitgebe.

jojo61: Prima. Und das grab funktioniert nach wie vor? Dann wäre das doch die Lösung....

Falls das funktioniert, müsste man den Kernel dann noch "sauber" machen, also die Warnungen entfernen. Er ist nämlich mit der heißen Nadel gestrickt...

Man könnte dann auch noch die CoreElec-Patches einbauen...

Welches Kernel-Modul ist für PIP zuständig?

Ich habe mir einen Odroid-C4 zugelegt und nach Post 1 installiert. Bild sieht gut aus, Ton ist da, nur OSD fehlt.

Damit läufts, danke. Die nächste Frage wäre gewesen, wie ich auf 50Hz umstelle

Schön wäre, wenn man den VDR über die Tastatur steuern könnte. Evtl. geht das über plugin-remote, das kann ich mal Testen.

jojo61: Ich möchte mich an dieser Stelle noch einmal bei Dir bedanken für das Plugin und auch Deine Geduld, hyperion zum Laufen zu bekommen. Wenn Du möchtest, kannst Du die beiden Kernel gerne irgendwo hosten.

Mit meinem neuen Kernel und Deinem aktuellen Github-Stand habe ich keine Probleme. Wenn das bei anderen auch problemlos läuft, kann ich gerne noch die Warnungen entfernen und mich um die CoreElec-Patches bemühen.

Was braucht es im Kernel, damit PIP läuft? Dann versuche ich noch, das einzubauen.

Also nochmal: Vielen herzlichen Dank. So macht der Odroid N2+ echt Spaß!

jojo61 : Eins ist mir noch aufgefallen: Mein Skin (Nopacity) ruft mittels GetVideoSize() die Größe des aktuellen Videos ab, um sie im OSD als SD / HD / UHD anzeigen zu können. Leider ist das im Plugin noch nicht implementiert. Ich hab mal versucht, das einzubauen.

diff --git a/softhddev.c b/softhddev.c
index 459d303..90a9057 100644
--- a/softhddev.c
+++ b/softhddev.c
@@ -2350,9 +2350,27 @@ int PlayVideo3(VideoStream * stream, const uint8_t * data, int size)
 **
 **  @todo FIXME: combine the 5 ifs at start of the function
 */
+
+static uint32_t _width  = 0;
+static uint32_t _height = 0;
+
 int PlayVideo(const uint8_t * data, int size)
 {
-    return PlayVideo3(MyVideoStream, data, size);
+    int iRet = PlayVideo3(MyVideoStream, data, size);
+
+    int fd = open("/sys/class/vdec/vdec_status", O_RDONLY);
+    if (fd > 0) {
+       char vdec_status[512];
+       if (read(fd, vdec_status, sizeof(vdec_status)) > 0) {
+           if (!strstr(vdec_status,"No vdec")) {
+               sscanf(strstr(vdec_status,"width"),"width : %d"  ,&_width);
+               sscanf(strstr(vdec_status,"height"),"height : %d",&_height);
+           }
+       }
+       close(fd);
+    }
+
+    return iRet;
 }

 /// call VDR support function
@@ -2526,6 +2544,10 @@ void GetVideoSize(int *width, int *height, double *aspect)
     int aspect_num;
     int aspect_den;

+    *width  = _width;
+    *height = _height;
+    *aspect = 16. / 9;
+#if    0
     if (MyVideoStream->HwDecoder) {
         VideoGetVideoSize(MyVideoStream->HwDecoder, width, height, &aspect_num, &aspect_den);
         *aspect = (double)aspect_num / (double)aspect_den;
@@ -2542,6 +2564,7 @@ void GetVideoSize(int *width, int *height, double *aspect)
         done_height = *height;
     }
 #endif
+#endif
 }

 /**

Vermutlich geht das eleganter und ist besser an anderer Stelle eingebunden, aber so klappt es bei mir.

Quote from beta

jojo61: Ich möchte mich an dieser Stelle noch einmal bei Dir bedanken für das Plugin und auch Deine Geduld, hyperion zum Laufen zu bekommen. Wenn Du möchtest, kannst Du die beiden Kernel gerne irgendwo hosten.

Mit meinem neuen Kernel und Deinem aktuellen Github-Stand habe ich keine Probleme. Wenn das bei anderen auch problemlos läuft, kann ich gerne noch die Warnungen entfernen und mich um die CoreElec-Patches bemühen.

Was braucht es im Kernel, damit PIP läuft? Dann versuche ich noch, das einzubauen.

Also nochmal: Vielen herzlichen Dank. So macht der Odroid N2+ echt Spaß!

Ich bedanke mich auch.

Ganz besonders auch an beta der mit aufs Pferd geholfen hat.

Bei mir läuft es nun auch mit hyperion lokal mit amlogic-grabber und weiterleitung an hyperion server.

Quote from jojo61

Wenn ich das richtig verstehe dann hattest du die kompletten Media Module aus dem corelec Kernel da eingebaut. Stimmt das so ?

Ja, das stimmt so. Ich habe den gesamten "media"- und media_modules"-Zweig eingebaut.

Ich hatte gestern allerdings einen Hänger mit diesem Kernel, den ich noch nicht zuordnen kann. Ich shaue mir das noch einmal an.

Außerdem habe ich noch ein Phänomen beobachtet, das ich nicht zuordnen kann. Wenn mein TV aus ist und ich eine config für Hpyerion erstelle, kann ich die nutzen und die funktioniert auch. Wenn der TV an, funktioniert das nicht mehr (Crash UHD). Da stimmt auch noch was nicht.

Ich wünsche Dir erst einmal einen tollen Urlaub, schöne Weihnachten und erholsame Tage. Das wünsche ich auch allen, die hier mitlesen.

Liebe Grüße

beta

Ich bin einen kleinen Schritt weitergekommen. Ich habe mir 2 unterschiedliche hyperion.db mit DB Browser for SQLite angeschaut. Die funktionierende DB hat den Eintrag (in der settings-Tabelle, "framegrabber":

{"available_devices":"AmLogic","cropBottom":0,"cropLeft":0,"cropRight":0,"cropTop":0,"device":"amlogic","device_inputs":"0","enable":true,"fps":5,"framerates":"5","height":1080,"input":0,"pixelDecimation":8,"resolutions":"0","width":1920}

Die nicht-funktionierende hat bei height 2159 und bei width 3839 stehen. Wenn ich die Werte ändere, kann ich auch die config nehmen, die mit Hyperion erstellt worden ist und die funktioniert dann auch. Trotz dieser Werte (1920x1080) kann ich aber 4K grabben. Ich weiß nicht, woran das liegt. Ich werde noch einen Patch für das hyperion-config-webinterface machen, dass hier immer "nur" 1920x1080 abgespeichert wird. Damit sollten dann auch eigene configs möglich sein.

Vielleicht hat jojo61 noch eine Idee nach seinem Urlaub. Mit diesen Infos sollte aber jeder erst einmal mit dem neuen Kernel, den ich gepostet habe und einem Git-Hyperion mit dem o.g. Trick eine laufende config hinbekommen.

Mit der Änderung aus dem letzten Commit gibt's einen Segfault, wenn die Auflösung nicht gelesen werden kann.

Ich habe die Funktionen jetzt was fehlersicherer gemacht, damit das nicht mehr passiert.

Der eigentliche Fehler ist in Zeile 67 im Patch.

diff --git a/video.c b/video.c
index 5e4f108..af48ce0 100644
--- a/video.c
+++ b/video.c
@@ -491,7 +491,7 @@ uint8_t *OdroidVideoGrab(int *ret_size, int *ret_width, int *ret_height, int mit
         return NULL;

        char vdec_status[512];
-       amlGetString("/sys/class/vdec/vdec_status",vdec_status);
+       amlGetString("/sys/class/vdec/vdec_status",vdec_status,sizeof(vdec_status));

        if(strstr(vdec_status,"No vdec")) {
                return NULL;
@@ -708,12 +708,11 @@ uint8_t *VideoGrab(int *size, int *width, int *height, int write_header)
 void VideoGetVideoSize(VideoHwDecoder *i, int *width, int *height, int *aspect_num, int *aspect_den) {
        char vdec_status[512];

-       amlGetString("/sys/class/vdec/vdec_status",vdec_status);
-
-       if(strstr(vdec_status,"No vdec")) {
+       if (amlGetString("/sys/class/vdec/vdec_status",vdec_status,sizeof(vdec_status)) < 0 ||
+           !*vdec_status || strstr(vdec_status,"No vdec")) {
                *width = *height = 0;
                *aspect_num = *aspect_den = 1;
-               return ;
+               return;
        }
        char r;
        sscanf(strstr(vdec_status,"width"),"width : %d",width);
@@ -1388,7 +1387,7 @@ bool getResolution(char *mode) {
        }


-       amlGetString("/sys/class/display/mode",mode);
+       amlGetString("/sys/class/display/mode",mode,sizeof(mode));
        getResolution(mode);
        ClearDisplay();

@@ -2494,26 +2493,22 @@ int amlSetString(char *path, char *valstr)
   return ret;
 }

-int amlGetString(char *path, char *valstr)
+int amlGetString(char *path, char *valstr, size_t size)
 {
   int len;
-  char buf[256] = {0};

   int fd = open(path, O_RDONLY);
   if (fd >= 0)
   {
-    *valstr = 0;
-    while ((len = read(fd, buf, 256)) > 0)
-      strncat(valstr,buf,len);
+    len = read(fd, valstr, size);
     close(fd);
-
-    //StringUtils::Trim(valstr);
-
-    return 0;
+    if (len > 0) {
+       return 0;
+    }
   }
-
   Debug(3, "%s: error reading %s",__FUNCTION__, path);
-  valstr = "fail";
+  if (valstr)
+     *valstr = 0;
   return -1;
 }

Ansonsten danke und schöne Feiertage!

Lothar