Re: zr36067 no longer loads automatically (regression)

2009-01-10 Thread Jean Delvare
On Fri, 9 Jan 2009 21:28:31 +0100, Jean Delvare wrote: > On Fri, 9 Jan 2009 12:12:09 -0800 (PST), Trent Piepho wrote: > > I have some more patches at http://linuxtv.org/hg/~tap/zoran > > I'll take a look, thanks. As discussed on IRC: all patches tested, they worked fine for me, thanks for your ex

Re: zr36067 no longer loads automatically (regression)

2009-01-10 Thread Jean Delvare
Hi Trent, On Fri, 9 Jan 2009 12:20:54 -0800 (PST), Trent Piepho wrote: > On Fri, 9 Jan 2009, Trent Piepho wrote: > > Here is a new version against latest v4l-dvb sources. Jean, are you trying > > to apply against the kernel tree? These patches are against the v4l-dvb Hg > > repository which isn'

Re: zr36067 no longer loads automatically (regression)

2009-01-09 Thread Jean Delvare
On Fri, 9 Jan 2009 12:12:09 -0800 (PST), Trent Piepho wrote: > On Fri, 9 Jan 2009, Jean Delvare wrote: > > On Fri, 9 Jan 2009 09:20:18 -0200, Mauro Carvalho Chehab wrote: > > > Unfortunately, I don't have any Zoran card here to test. > > > > > > Jean, it is up to you to test Trent's patch. > > > >

Re: zr36067 no longer loads automatically (regression)

2009-01-09 Thread Trent Piepho
On Fri, 9 Jan 2009, Trent Piepho wrote: > Here is a new version against latest v4l-dvb sources. Jean, are you trying > to apply against the kernel tree? These patches are against the v4l-dvb Hg > repository which isn't quite the same as what's in the kernel. > > I have some more patches at http:/

Re: zr36067 no longer loads automatically (regression)

2009-01-09 Thread Trent Piepho
On Fri, 9 Jan 2009, Jean Delvare wrote: > On Fri, 9 Jan 2009 09:20:18 -0200, Mauro Carvalho Chehab wrote: > > On Thu, 8 Jan 2009 22:01:08 -0800 (PST) > > Trent Piepho wrote: > > > > > > Yuck, why don't we fix this instead? > > > > This will be much better. > > > > > Here's an initial test. I have

Re: zr36067 no longer loads automatically (regression)

2009-01-09 Thread Jean Delvare
On Fri, 9 Jan 2009 09:20:18 -0200, Mauro Carvalho Chehab wrote: > > On Thu, 8 Jan 2009 22:01:08 -0800 (PST) > Trent Piepho wrote: > > > On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > > On Thu, 8 Jan 2009 13:20:19 -0800 (PST) > > > Trent Piepho wrote: > > > > On Thu, 8 Jan 2009, Mauro Carv

Re: zr36067 no longer loads automatically (regression)

2009-01-09 Thread Mauro Carvalho Chehab
On Fri, 9 Jan 2009 01:13:59 -0800 (PST) Trent Piepho wrote: > > > Here's an initial test. I haven't yet found my dc10+ to test it with. > > > > I'm all for it, but what tree was your patch built against? It doesn't > > seem to apply to anything I have. > > The v4l-dvb main Hg from yesterday.

Re: zr36067 no longer loads automatically (regression)

2009-01-09 Thread Mauro Carvalho Chehab
On Thu, 8 Jan 2009 22:01:08 -0800 (PST) Trent Piepho wrote: > On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > On Thu, 8 Jan 2009 13:20:19 -0800 (PST) > > Trent Piepho wrote: > > > On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > > It doesn't seem like any other driver needs to protect

Re: zr36067 no longer loads automatically (regression)

2009-01-09 Thread Trent Piepho
On Fri, 9 Jan 2009, Jean Delvare wrote: > On Thu, 8 Jan 2009 22:01:08 -0800 (PST), Trent Piepho wrote: > > On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > > However, Zoran driver doesn't rely on pci_register_driver(). Instead, it > > > uses a > > > while() loop to probe for Zoran devices: > >

Re: zr36067 no longer loads automatically (regression)

2009-01-09 Thread Jean Delvare
Hi Trent, On Thu, 8 Jan 2009 22:01:08 -0800 (PST), Trent Piepho wrote: > On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > On Thu, 8 Jan 2009 13:20:19 -0800 (PST) > > Trent Piepho wrote: > > > On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > > It doesn't seem like any other driver needs to

Re: zr36067 no longer loads automatically (regression)

2009-01-08 Thread Trent Piepho
On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > On Thu, 8 Jan 2009 13:20:19 -0800 (PST) > Trent Piepho wrote: > > On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > It doesn't seem like any other driver needs to protect the module device > > table with an ifdef. > > However, Zoran driver does

Re: zr36067 no longer loads automatically (regression)

2009-01-08 Thread Mauro Carvalho Chehab
On Thu, 8 Jan 2009 22:45:20 +0100 Jean Delvare wrote: > Hi Mauro, > > On Thu, 8 Jan 2009 17:56:27 -0200, Mauro Carvalho Chehab wrote: > > > Hi Mauro, > > > > > > This commit of yours: > > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=60b4bde48b36c0315ef41fd3

Re: zr36067 no longer loads automatically (regression)

2009-01-08 Thread Jean Delvare
Hi Mauro, On Thu, 8 Jan 2009 17:56:27 -0200, Mauro Carvalho Chehab wrote: > > Hi Mauro, > > > > This commit of yours: > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=60b4bde48b36c0315ef41fd38c339b9c7e68c46f > > causes the zr36067 driver to no longer load automa

Re: zr36067 no longer loads automatically (regression)

2009-01-08 Thread Mauro Carvalho Chehab
On Thu, 8 Jan 2009 13:20:19 -0800 (PST) Trent Piepho wrote: > On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=60b4bde48b36c0315ef41fd38c339b9c7e68c46f > > > causes the zr36067 driver to no longer load automati

Re: zr36067 no longer loads automatically (regression)

2009-01-08 Thread Trent Piepho
On Thu, 8 Jan 2009, Mauro Carvalho Chehab wrote: > > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=60b4bde48b36c0315ef41fd38c339b9c7e68c46f > > causes the zr36067 driver to no longer load automatically on systems > > with a Zoran-based adapter. The comment says tha

Re: zr36067 no longer loads automatically (regression)

2009-01-08 Thread Mauro Carvalho Chehab
> Hi Mauro, > > This commit of yours: > http://git.kernel.org/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commitdiff;h=60b4bde48b36c0315ef41fd38c339b9c7e68c46f > causes the zr36067 driver to no longer load automatically on systems > with a Zoran-based adapter. The comment says that zr36067_pci_tb