I just tested the latest changes and they work great!!<br><br>Albert<br><br><div class="gmail_quote">2008/5/28 Albert Comerma <<a href="mailto:albert.comerma@gmail.com">albert.comerma@gmail.com</a>>:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
I can't test it right now (I'm out for work). But I've looked the changes and they seem great for me, I just compiled and installed, without any problem, there's just the test left, for next weekend.<br><br>
Thanks!<br><br>Albert<br><br><div class="gmail_quote">2008/5/27 Mauro Carvalho Chehab <<a href="mailto:mchehab@infradead.org" target="_blank">mchehab@infradead.org</a>>:<div><div></div><div class="Wj3C7c"><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi Albert,<br>
<div><br>
On Tue, 27 May 2008 17:23:50 +0200<br>
"Albert Comerma" <<a href="mailto:albert.comerma@gmail.com" target="_blank">albert.comerma@gmail.com</a>> wrote:<br>
<br>
> There seems to be a problem with the last changes on xc2028 code, please try<br>
> this;<br>
><br>
> In linux/drivers/media/common/tuners/tuner-xc2028.c file, on xc2028_attach,<br>
> video_dev must be = cfg->video_dev;<br>
> and on the current source it's = cfg->i2c_adap->algo_data; which completely<br>
> breaks the module when loaded.<br>
><br>
> It was already suggested that this should be changed, but nobody said why<br>
> this modification was done or why it was kept.<br>
><br>
> Mauro could you trace when and why this modification was done? or at least<br>
> give it back to original state?<br>
><br>
<br>
</div>It seems that I forgot to merge the fix for this. I've just committed it. Could<br>
you please test ? It is already available at mercurial tree.<br>
<br>
Cheers,<br>
<font color="#888888">Mauro<br>
</font></blockquote></div></div></div><br>
</blockquote></div><br>