Mauro Carvalho Chehab wrote: > On Sun, 15 Jun 2008 21:23:36 +0800 > timf <timf@xxxxxxxxxxxx> wrote: > > >> Invisible whitespaces! Grrr! >> > > Happens all the time :) > > "make whitespace" clears it. > > >> timf@ubuntu:~/1/try2$ diff -upr v4l-dvb v4l-dvb-tim > tim.patch >> timf@ubuntu:~/1/try2$ cd v4l-dvb-tim >> timf@ubuntu:~/1/try2/v4l-dvb-tim$ make checkpatch >> make -C /home/timf/1/try2/v4l-dvb-tim/v4l checkpatch >> make[1]: Entering directory `/home/timf/1/try2/v4l-dvb-tim/v4l' >> scripts/check.pl -c >> # WARNING: /lib/modules/`uname -r`/build/scripts/checkpatch.pl version >> 0.12 is >> # older than scripts/checkpatch.pl --no-tree version 0.16. >> # Using in-tree one. >> # >> make[1]: Leaving directory `/home/timf/1/try2/v4l-dvb-tim/v4l' >> timf@ubuntu:~/1/try2/v4l-dvb-tim$ >> >> My confidence is shot to pieces, you realise! >> >> Attached - tim.patch >> >> Please, please work! >> > > Worked ;) > > Patch applied, thanks. > > Now, all we need to do is to make tda1004x more stable. The weird thing is that > it works fine with my Intel based notebook. It just fails on my dual core AMD, > with a higher clock. I suspect that this is due to a timeout issue, but not > 100% sure. > > > Cheers, > Mauro > > Fantastic! I have an old box which I'm pretty sure has a cpu about 1.6, non-dual core. Tomorrow, (too much noise at this hour of night here!) I will try either/or Kworld 210, Pinnacle 310i, Kworld 220. I will try both current v4l-dvb and that patch of yours in tda10046x.c, and see what happens. Is it worth trying different msleep values? All I can think of is this is a relatively recent event with this "revision FF" business, but I can't be exact as to how long ago I noticed it. Regards, Timf _______________________________________________ linux-dvb mailing list linux-dvb@xxxxxxxxxxx http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb