[vdr] Re: [linux-dvb] Re: Problems switching to dvb-kernel

[Klaus Schmidinger <Klaus.Schmidinger@cadsoft.de>]

Philippe Gramoullé wrote:
> 
> Hello Klaus,
> 
> On Sun, 05 Sep 2004 13:58:19 +0200
> Klaus Schmidinger <Klaus.Schmidinger@cadsoft.de> wrote:
> 
>   | However, I also have a problem with the kernel
>   | 2.4 branch of the dvb-kernel driver, because the picture and sound drop out
>   | every now and then. This is an old problem that I once have fixed in the old
>   | DVB driver, but maybe somebody dropped that fix when going to dvb-kernel... :-(
>   | It may only happen with the rev 1.3 Siemens DVB cards.
> 
> I can confirm that i have this exact same problem with a full featured TT 1.3 Rev board with
> almost any 2.6 driver version i can remember of, CVS or official kernels. (since at least 2.6.5, i'd say).
> 
> So, so far, i'm "stuck" with 2.4 linux-dvb drivers.
> 
> As my video directory is NFS mounted, i have *much better* performance when using 2.6 client <-> 2.6 NFS server
> than compared with 2.4 client <-> 2.6 NFS server, especially when editing some recordings.
> I've tried various NFS fixes and updates (NFS_ALL from Trond)without success.
> So, right now, i just decided to live with this sub-optimal setup as long as it is fairly stable.
> 
> So until the audio/video blackouts are solved (i'll happily test any patch :), i have no other option than to stay with
> 2003-08-11 linux-dvb driver.

This fixes it for me:

--- linux/drivers/media/dvb/frontends/ves1x93.c  2004-08-25 21:27:08.000000000 +0200
+++ linux/drivers/media/dvb/frontends/ves1x93.c  2004-09-11 15:03:09.000000000 +0200
@@ -452,6 +452,11 @@
        {
                fe_status_t *status = arg;
                u8 sync = ves1x93_readreg (i2c, 0x0e);
+               int maxtry = 10; // just for safety - let's not get stuck here
+               while ((sync & 0x03) != 0x03 && (sync & 0x0c) && maxtry--) {
+                       msleep(10);
+                       sync = ves1x93_readreg (i2c, 0x0e);
+                }
 
                *status = 0;


Klaus