On 05.04.2010 12:43, Teemu Rantanen wrote: > Hi, > > I tried also without delete cDeviceHook but it still crashed... Please try creating the cReddoDeviceHook in cPluginDisableReddoQAM256::Initialize(), as suggested in PLUGINS.html. > Well, the fixes are basically available in the plugin, as I copied the > probe method (and modified slightly to suit the plugin). Those are: > - The sysfs filenames are idVendor (subsystem_vendor) and idProduct > (subsystem_device) This is what I get here: root@video:/home/kls/vdr/VDR > ls -l /sys/class/dvb/dvb0.frontend0/device/subsystem_* -r--r--r-- 1 root root 4096 2010-04-05 16:25 /sys/class/dvb/dvb0.frontend0/device/subsystem_device -r--r--r-- 1 root root 4096 2010-04-05 16:25 /sys/class/dvb/dvb0.frontend0/device/subsystem_vendor root@video:/home/kls/vdr/VDR > ls -l /sys/class/dvb/dvb0.frontend0/device/id* ls: cannot access /sys/class/dvb/dvb0.frontend0/device/id*: No such file or directory Maybe a bug in the driver? > - You need to set strtoul() base as 16 as the files don't have 0x in > front of the hex root@video:/home/kls/vdr/VDR > cat /sys/class/dvb/dvb0.frontend0/device/subsystem_device 0x0000 root@video:/home/kls/vdr/VDR > cat /sys/class/dvb/dvb0.frontend0/device/subsystem_vendor 0x13c2 Maybe the "reddo" driver behaves differently than the FuSi driver? > What about making this an utility method in VDR? Give device file name > and return id... Will do. Have you tried whether it works with my patch? In that case you wouldn't even need the whole plugin. Klaus > 2010/4/5 Klaus Schmidinger <Klaus.Schmidinger@xxxxxxx > <mailto:Klaus.Schmidinger@xxxxxxx>> > > From PLUGINS.html: > > A plugin that creates a derived cDeviceHook ... > shall not delete this object. It will be automatically deleted when the > program ends > > I have added a similar note to device.h now. > > > ... > > - The sysfs probe code in full feature card plugin is buggy :-) > > Would you mind posting a patch that fixes this? > > Klaus _______________________________________________ vdr mailing list vdr@xxxxxxxxxxx http://www.linuxtv.org/cgi-bin/mailman/listinfo/vdr
