Re: av7110 and budget_av are broken! (was: Re: changeset 14351:2eda2bcc8d6f)

2010-03-20 Thread Oliver Endriss
ges > > > was: It wasn't > > > possible to unload the module for the TT-C2300. > > > > Guys, when will you finally apply this fix? > > Thanks for reminding me, I frankly forgot about this. > > Hartmut, is the problem with unloading the module something th

Re: av7110 and budget_av are broken! (was: Re: changeset 14351:2eda2bcc8d6f)

2010-03-20 Thread Hans Verkuil
C2300. > > Guys, when will you finally apply this fix? Thanks for reminding me, I frankly forgot about this. Hartmut, is the problem with unloading the module something that my patch caused? Or was that there as well before changeset 14351:2eda2bcc8d6f? Are there any kernel messages indicating wh

av7110 and budget_av are broken! (was: Re: changeset 14351:2eda2bcc8d6f)

2010-03-20 Thread Oliver Endriss
ns, > > it works with my TT-C2300 perfectly. The main problem of your changes was: It > wasn't > possible to unload the module for the TT-C2300. Guys, when will you finally apply this fix? As Hartmut pointed out, changeset 14351:2eda2bcc8d6f broke the av7110 driver (and also bud

Re: changeset 14351:2eda2bcc8d6f

2010-03-16 Thread e9hack
Am 13.3.2010 17:27, schrieb Hans Verkuil: > If there are no further comments, then I'll post a pull request in a few days. > > Tested with the mxb board. It would be nice if you can verify this with the > av7110. Hi hans, it works with my TT-C2300 perfectly. The main problem of your changes was:

Re: changeset 14351:2eda2bcc8d6f

2010-03-13 Thread Hans Verkuil
dev); int saa7146_vv_init(struct saa7146_dev* dev, struct saa7146_ext_vv *ext_vv); int saa7146_vv_release(struct saa7146_dev* dev); On Wednesday 03 March 2010 12:39:27 e9hack wrote: > Hi, > > changeset 14351:2eda2bcc8d6f is incomplete. If the init function is split in > two

changeset 14351:2eda2bcc8d6f

2010-03-03 Thread e9hack
Hi, changeset 14351:2eda2bcc8d6f is incomplete. If the init function is split in two function, the deinit function shall consider this. The changes shall be apply also to av7110_init_v4l(). diff -r 58ae12f18e80 linux/drivers/media/common/saa7146_fops.c --- a/linux/drivers/media/common