Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 )
> you are using the older patch use the attached one.
> The fix-s2api-dvbs is not needed with this one.
I have some comment from these patches as I tried to investigate the code
to find out why it does not work with my hvr-1300 and hvr-4000 in
vdr-1.7.1 and xineliboutput. With vdr-1.7.0 version of these patches, the
hvr-4000 is able to scan dvb-s channels ok but hvr-1300 fails to scan
dvb-t channels.
One thing that looks suspicious is that the code in
dvbdevice.c function cDvbDevice::cDvbDevice(int n).
I have hvr-1300 (dvb-t) as a first device and hvr-4000
(dvb-t+dvb-s/s2) as a second device.
For hvr-1300 this method reports incorrect transport modes "DVBS DSS",
when it should report "DVBT". For hvr-4000 it is reporting
the "DVBS2", so it is halfly correct. I think it should report "DVBS2
DVBT" as I am using s2-mfe drivers and thus 2 frontends created under
/dev/dvb/adapter1.
Oct 18 00:00:23 tinka vdr: [28627] device 1 provides: DVBS DSS
...
Oct 18 00:00:23 tinka vdr: [28627] device 2 provides: DVBS2
I think there are couple of bugs in the code.
1) frontend.h in dvb-b drivers defines following types:
typedef enum fe_delivery_system {
SYS_UNDEFINED,
SYS_DVBC_ANNEX_AC,
SYS_DVBC_ANNEX_B,
SYS_DVBT,
SYS_DVBS,
SYS_DVBS2,
SYS_DVBH,
SYS_ISDBT,
SYS_ISDBS,
SYS_ISDBC,
SYS_ATSC,
SYS_ATSCMH,
SYS_DMBTH,
SYS_CMMB,
SYS_DAB,
} fe_delivery_system_t;
But in vdr-code those are compared to stings defined in following array:
const char *DeliverySystems[] = {
"DVBS",
"DSS",
"DVBS2",
"DVBC",
"DVBT",
"DVBH",
"ATSC",
NULL
};
So both the order and amount of strings compared is different and I
do not understand how this "if (frontendType & (1u << i))" is planned to
work.
If I have understood correctly this bug does not however explain why
my hvr-1300 does not work, as in the end this for loop is only used for
trying to detect whether the frontend type is SYS_DVBS2. For other types
it will just printout informative debug.
2) Second problem that comes to my mind is that the code seems to assume
that there is only 1 frontend for every adapter available, so it will not
detect if card supports for example both dvb-t and dvb-s. I do not know
yet, which would be the best place for making changes to support both.
___
vdr mailing list
vdr@linuxtv.org
http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 )
Hi, enable permanent timeshift in vdr. cu Edgar (gimli) Hucek > Hello, > > thanks for the quick response! Everything compiled fine now. > But still something seems to be broken there. Watching TV with mplayer > works fine but there is a problem with vdr/xine. I attached my syslog > and the output of vdr-sxfe. I hope someone has an idea whats causing this. > > Many thanks in advance! > Martin Schlosser > > > Hi, > > > > you are using the older patch use the attached one. > > The fix-s2api-dvbs is not needed with this one. > > > > cu > > > > Edgar (gimli) Hucek > > > > > > Hi All, > > > > > > > > this patches are for Steve Toth's repository at : > > > > http://linuxtv.org/hg/~stoth/s2/ . > > > > In theory they should work. I did them offline from my VDR. > > > > > > > > cu > > > > > > > > Edgar (gimli) Hucek > > > > > > Hi, > > > > > > I just can't compile my vdr. I applied the following patches to > vanilla > > > vdr-1.7.1: > > > > > > - vdr-1.7.1_extensions.diff > > > - vdr-1.7.1-ext_h264.diff > > > - vdr-1.7.1-s2api-ext64.patch > > > - fix-s2api-dvbs-tuning.patch > > > > > > Everything worked fine until here. Now I downloaded and installed > S2API > > > (from http://linuxtv.org/hg/~stoth/s2-mfe right?). > > > > > > After setting the paths etc in teh make configs I tried to compile > vdr, > > > but this is what happens then: > > > > > > root at HTPC:/usr/local/src/vdr-1.7.1# make > > > g++ -g -O2 -Wall -Woverloaded-virtual -Wno-parentheses -c > > > -DUSE_CMDRECCMDI18N -DUSE_CMDSUBMENU -DUSE_CUTTIME -DUSE_DDEPGENTRY > > > -DUSE_DOLBYINREC -DUSE_JUMPPLAY -DUSE_LIEMIKUUTIO -DUSE_PLUGINMISSING > > > -DUSE_SETTIME -DUSE_WAREAGLEICON -DREMOTE_KBD -DVDR_USER=\"vdr\" > > > -DLIRC_DEVICE=\"/dev/lircd\" -DRCU_DEVICE=\"/dev/ttyS1\" > -D_GNU_SOURCE > > > -DVIDEODIR=\"/daten/Video/vdr/video.00\" > > > -DCONFDIR=\"/daten/Video/vdr/video.00\" -DPLUGINDIR=\"./PLUGINS/lib\" > > > -DLOCDIR=\"./locale\" -DUSE_PLUGINAPI -I/usr/include/freetype2 > > > -I../s2-mfe/linux/include channels.c > > > channels.c:70: error: 'NBC_QPSK' was not declared in this scope > > > channels.c:71: error: '_8PSK' was not declared in this scope > > > channels.c:72: error: '_16APSK' was not declared in this scope > > > channels.c: In member function 'void > > > cChannel::SetLinkChannels(cLinkChannels*)': > > > channels.c:690: warning: format not a string literal and no format > > > arguments > > > make: *** [channels.o] Fehler 1 > > > > > > > > > Any ideas? > > > > > > Regards > > > Martin Schlosser > ___ > vdr mailing list > vdr@linuxtv.org > http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr > ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 )
> [EMAIL PROTECTED]:/usr/local/src/vdr-1.7.1# make > g++ -g -O2 -Wall -Woverloaded-virtual -Wno-parentheses -c > -DUSE_CMDRECCMDI18N -DUSE_CMDSUBMENU -DUSE_CUTTIME -DUSE_DDEPGENTRY > -DUSE_DOLBYINREC -DUSE_JUMPPLAY -DUSE_LIEMIKUUTIO -DUSE_PLUGINMISSING > -DUSE_SETTIME -DUSE_WAREAGLEICON -DREMOTE_KBD -DVDR_USER=\"vdr\" > -DLIRC_DEVICE=\"/dev/lircd\" -DRCU_DEVICE=\"/dev/ttyS1\" -D_GNU_SOURCE > -DVIDEODIR=\"/daten/Video/vdr/video.00\" > -DCONFDIR=\"/daten/Video/vdr/video.00\" -DPLUGINDIR=\"./PLUGINS/lib\" > -DLOCDIR=\"./locale\" -DUSE_PLUGINAPI -I/usr/include/freetype2 > -I../s2-mfe/linux/include channels.c > channels.c:70: error: 'NBC_QPSK' was not declared in this scope > channels.c:71: error: '_8PSK' was not declared in this scope > channels.c:72: error: '_16APSK' was not declared in this scope > channels.c: In member function 'void > cChannel::SetLinkChannels(cLinkChannels*)': > channels.c:690: warning: format not a string literal and no format arguments > make: *** [channels.o] Fehler 1 > > > Any ideas? There has been renamings in the driver about 7 days ago which break the vdr patch. For example _8PSK --> PSK_8 See following patch for details: http://linuxtv.org/hg/~stoth/s2-mfe/rev/d1c25e26799f Mika ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 )
Hi, you are using the older patch use the attached one. The fix-s2api-dvbs is not needed with this one. cu Edgar (gimli) Hucek > > Hi All, > > > > this patches are for Steve Toth's repository at : > > http://linuxtv.org/hg/~stoth/s2/ . > > In theory they should work. I did them offline from my VDR. > > > > cu > > > > Edgar (gimli) Hucek > > Hi, > > I just can't compile my vdr. I applied the following patches to vanilla > vdr-1.7.1: > > - vdr-1.7.1_extensions.diff > - vdr-1.7.1-ext_h264.diff > - vdr-1.7.1-s2api-ext64.patch > - fix-s2api-dvbs-tuning.patch > > Everything worked fine until here. Now I downloaded and installed S2API > (from http://linuxtv.org/hg/~stoth/s2-mfe right?). > > After setting the paths etc in teh make configs I tried to compile vdr, > but this is what happens then: > > [EMAIL PROTECTED]:/usr/local/src/vdr-1.7.1# make > g++ -g -O2 -Wall -Woverloaded-virtual -Wno-parentheses -c > -DUSE_CMDRECCMDI18N -DUSE_CMDSUBMENU -DUSE_CUTTIME -DUSE_DDEPGENTRY > -DUSE_DOLBYINREC -DUSE_JUMPPLAY -DUSE_LIEMIKUUTIO -DUSE_PLUGINMISSING > -DUSE_SETTIME -DUSE_WAREAGLEICON -DREMOTE_KBD -DVDR_USER=\"vdr\" > -DLIRC_DEVICE=\"/dev/lircd\" -DRCU_DEVICE=\"/dev/ttyS1\" -D_GNU_SOURCE > -DVIDEODIR=\"/daten/Video/vdr/video.00\" > -DCONFDIR=\"/daten/Video/vdr/video.00\" -DPLUGINDIR=\"./PLUGINS/lib\" > -DLOCDIR=\"./locale\" -DUSE_PLUGINAPI -I/usr/include/freetype2 > -I../s2-mfe/linux/include channels.c > channels.c:70: error: 'NBC_QPSK' was not declared in this scope > channels.c:71: error: '_8PSK' was not declared in this scope > channels.c:72: error: '_16APSK' was not declared in this scope > channels.c: In member function 'void > cChannel::SetLinkChannels(cLinkChannels*)': > channels.c:690: warning: format not a string literal and no format > arguments > make: *** [channels.o] Fehler 1 > > > Any ideas? > > Regards > Martin Schlosser > > ___ > vdr mailing list > vdr@linuxtv.org > http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr > > > vdr-1.7.1-s2api-v2-07102008-ext64.patch.gz Description: application/gzip ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 )
Klaus Schmidinger <[EMAIL PROTECTED]> wrote: > So as long as these cards are not > supported, VDR will not go anywhere near S2API (and I certainly will > not support two DVB APIs in parallel). once again a strong reason to splitt off dvb support into plugins. clemens ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 )
-Original message- From: Klaus Schmidinger <[EMAIL PROTECTED]> Sent: Tue 07-10-2008 13:44 To: vdr@linuxtv.org; Subject: Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 ) > On 10/07/08 12:46, gimli wrote: > > Hi All, > > > > this patches are for Steve Toth's repository at : > > http://linuxtv.org/hg/~stoth/s2/ . > > In theory they should work. I did them offline from my VDR. > > Just so nobody wonders why I'm not currently adopting this switch > to S2API: I've tested a recent version of the S2API driver and it > apparently doesn't support the TT-Budget S2-3200 cards yet (which > IIRC the "multiproto" driver does). I understand this completely. Even besides the fact that the card (still?) has locking problems with multiproto, I hope that the current drivers in multiproto and/or mantis will be be ported to S2API soon. I have high hopes that this will be the case in a short time. It's been two weeks after the S2API announcement and they announced that in 4 weeks after the announcement the TT-3200 should be included in S2API. So have hope! :) > So as long as these cards are not > supported, VDR will not go anywhere near S2API (and I certainly will > not support two DVB APIs in parallel). > The reason why I started (read: continued Igor's patch ;) with the S2API patch, is because S2API has been voted to be the next enhancement in the v4l drivers and I wanted to test S2API. Since I have a WinTV-NOVA-HD-S2, I didn't had much difficulty to test S2API in combination with VDR (since it's supported since the beginning, ghe). The plus of this all is, is that people can use VDR *now* with their S2API supported DVB-cards for DVB-T, DVB-S, DVB-S2 and possibly DVB-C. But other can use VDR as well, as soon when it becomes a part of v4l and/or part of the next kernel release :) Currently only Kaffeine (untested, but it should be working) and VDR 1.7.x have support for S2API. So VDR is atleast ahead of MythTV on this point (I haven't seen any S2API patches floating around for MythTV). And I'll be making patches for S2API-support, as long as long as you can't test/support it :) > Klaus > Regards, Niels Wagenaar ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 )
On 10/07/08 12:46, gimli wrote: > Hi All, > > this patches are for Steve Toth's repository at : > http://linuxtv.org/hg/~stoth/s2/ . > In theory they should work. I did them offline from my VDR. Just so nobody wonders why I'm not currently adopting this switch to S2API: I've tested a recent version of the S2API driver and it apparently doesn't support the TT-Budget S2-3200 cards yet (which IIRC the "multiproto" driver does). So as long as these cards are not supported, VDR will not go anywhere near S2API (and I certainly will not support two DVB APIs in parallel). Klaus ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 )
-Original message- From: gimli <[EMAIL PROTECTED]> Sent: Tue 07-10-2008 12:46 To: VDR Mailing List ; CC: VDR Mailing List ; Subject: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64 ( 071020008 ) > Hi All, > > this patches are for Steve Toth's repository at : > http://linuxtv.org/hg/~stoth/s2/ . > In theory they should work. I did them offline from my VDR. > Nice :) I saw the announcment/message from Steve on the linux-dvb mailinglist. Since this morning, many enhancements were added to the frontend.h for DVB-T. Also, QPSK/8PSK modulation with DVB-S2 should be fixed (no need for the NBC_QPSK hack). Since I'm at work, I'm not able to do anything (I can program but I cannot test). As soon as I'm home, I'll make the patch for 1.7.0 and make sure that everything works correctly with DVB-T, DVB-S and DVB-S2. > cu > > Edgar (gimli) Hucek Regards, Niels Wagenaar ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64
Приветствую, Mika > > here are the vdr S2API patches for vdr 1.7.1 + extensions 64 patch > > and also for the vanilla vdr 1.7.1. > > What is the purpose of this extension 64 patch? here you can read about it http://www.zulu-entertainment.de/download.php ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64
Приветствую, Mika > > i forgott write that the ext64 patch sits on top off : > > > > vdr 1.7.1 + extensions 64 + the h264 patch. > > With h264 patch, do you mean that > vdr-1.7.0-h264-syncearly-framespersec-audioindexer-fielddetection-speedup.diff > has been ported somewhere for vdr-1.7.1? no, h264 patch is vdr-1.7.1-ext_h264.diff , which included in VDR Extensions Patch revision 64 from Zulu http://www.zulu-entertainment.de/download.php Goga ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64
> i forgott write that the ext64 patch sits on top off : > > vdr 1.7.1 + extensions 64 + the h264 patch. With h264 patch, do you mean that vdr-1.7.0-h264-syncearly-framespersec-audioindexer-fielddetection-speedup.diff has been ported somewhere for vdr-1.7.1? Mika ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64
Hi, i forgott write that the ext64 patch sits on top off : vdr 1.7.1 + extensions 64 + the h264 patch. cu Edgar (gimli) Hucek >> here are the vdr S2API patches for vdr 1.7.1 + extensions 64 patch >> and also for the vanilla vdr 1.7.1. > > > could you release s2api patch for vdr 171 already patched with > vdr-1.7.1-ext_h264.diff > > ___ > vdr mailing list > vdr@linuxtv.org > http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr > > > ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64
> here are the vdr S2API patches for vdr 1.7.1 + extensions 64 patch > and also for the vanilla vdr 1.7.1. could you release s2api patch for vdr 171 already patched with vdr-1.7.1-ext_h264.diff ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64
> here are the vdr S2API patches for vdr 1.7.1 + extensions 64 patch > and also for the vanilla vdr 1.7.1. What is the purpose of this extension 64 patch? Mika ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64
Yes >> here are the vdr S2API patches for vdr 1.7.1 + extensions 64 patch >> and also for the vanilla vdr 1.7.1. > > does these patches include the DVB-S, DVB-S2, DVB-T and DVB-C support ? > > Goga > > ___ > vdr mailing list > vdr@linuxtv.org > http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr > > > ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
Re: [vdr] [PATCH] S2API for vdr-1.7.1 vanilla and extensions patch 64
> here are the vdr S2API patches for vdr 1.7.1 + extensions 64 patch > and also for the vanilla vdr 1.7.1. does these patches include the DVB-S, DVB-S2, DVB-T and DVB-C support ? Goga ___ vdr mailing list vdr@linuxtv.org http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
