<div dir="ltr">Mijhail Moreyra wrote:<br>
&gt; Steven Toth wrote:<br>
&gt;&gt; Mijhail Moreyra wrote:<br>
&gt;&gt;&gt; Steven Toth wrote:<br>
&gt;&gt;&gt;&gt; Mijhail,<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; <a href="http://linuxtv.org/hg/%7Estoth/cx23885-audio" target="_blank">http://linuxtv.org/hg/~stoth/cx23885-audio</a><br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; This tree contains your patch with some minor whitespace cleanups<br>
&gt;&gt;&gt;&gt; and fixes for HUNK related merge issues due to the patch wrapping at<br>
&gt;&gt;&gt;&gt; 80 cols.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Please build this tree and retest in your environment to ensure I<br>
&gt;&gt;&gt;&gt; did not break anything. Does this tree still work OK for you?<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; After this I will apply some other minor cleanups then invite a few<br>
&gt;&gt;&gt;&gt; other HVR1500 owners to begin testing.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Thanks again.<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Regards,<br>
&gt;&gt;&gt;&gt;<br>
&gt;&gt;&gt;&gt; Steve<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Hi, sorry for the delay.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; I&#39;ve tested the <a href="http://linuxtv.org/hg/%7Estoth/cx23885-audio" target="_blank">http://linuxtv.org/hg/~stoth/cx23885-audio</a> tree and<br>
&gt;&gt;&gt; it doesn&#39;t work well.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; You seem to have removed a piece from my patch that avoids some register<br>
&gt;&gt;&gt; modification in cx25840-core.c:cx23885_<div id=":ak" class="ArwC7c ckChnd">initialize()<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; - &nbsp; &nbsp; &nbsp; cx25840_write(client, 0x2, 0x76);<br>
&gt;&gt;&gt; + &nbsp; &nbsp; &nbsp; if (state-&gt;rev != 0x0000) /* FIXME: How to detect the bridge<br>
&gt;&gt;&gt; type ??? */<br>
&gt;&gt;&gt; + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; /* This causes image distortion on a true cx23885<br>
&gt;&gt;&gt; board */<br>
&gt;&gt;&gt; + &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; cx25840_write(client, 0x2, 0x76);<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; As the patch says that register write causes a horrible image distortion<br>
&gt;&gt;&gt; on my HVR-1500 which has a real cx23885 (not 23887, 23888, etc) board.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; I don&#39;t know if it&#39;s really required for any bridge as everything seems<br>
&gt;&gt;&gt; to be auto-configured by default, maybe it can be simply dropped.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Other than that the cx23885-audio tree works well.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; WRT the whitespaces, 80 cols, etc; most are also in the sources I took<br>
&gt;&gt;&gt; as basis, so I didn&#39;t think they were a problem.<br>
&gt;&gt;<br>
&gt;&gt; That&#39;s a mistake, I&#39;ll add that later tonight, thanks for finding<br>
&gt;&gt; this. I must of missed it when I had to tear apart your email because<br>
&gt;&gt; of HUNK issues caused by patch line wrapping.<br>
&gt;&gt;<br>
&gt;&gt; Apart from this, is everything working as you expect?<br>
&gt;&gt;<br>
&gt;&gt; Regards,<br>
&gt;&gt;<br>
&gt;&gt; Steve<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;<br>
&gt; OK.<br>
&gt;<br>
&gt; And sorry about the patch, I didn&#39;t know it was going to be broken that<br>
&gt; way by being sent by email.<br>
&gt;<br>
&gt; &nbsp;&gt;&gt; Other than that the cx23885-audio tree works well.<br>
&gt;<br>
<br>
&gt; Great, thanks for confirming.<br>
<br>
&gt; Regards,<br>
<br>
&gt; Steve</div><br>I&#39;ll try asking again since my replies in gmail were not including the correct subject heading.<br>Can this code for cx23885 analog support be adapted for the DViCO Fusion HDTV7 Dual Express which also uses the cx23885?&nbsp; Currently the driver for that card is digital only and I am stuck with a free antiquated large satellite system that is analog only in my apartment. I am willing to put in the work if someone can point me in the right direction.&nbsp; Thank you,<br clear="all">
<br> --Tim<br>
</div>