<div>


<div>
Steve,<br><br>I have reworked the tuner callback now against your branch at:<span id="obmessage"><br>http://linuxtv.org/hg/~stoth/v4l-dvb<br><br>The new Patch (to add support for this card) is attached inline below for testing (this is a hint Mark &amp; Jon), I have not provided a signed-off note on purpose as I want to solve the issue mentioned in the next paragraph first.<br><br>Regarding the cx25840 module; the card doesn't seem to initialise properly (no DVB output and DMA errors in log) unless I have this requested.&nbsp; Once the card is up and running I can unload all drivers, recompile without the cx25840 and load and it will work again until I power off the computer and back on again (This has been tedious trying to work out which setting I had missed).&nbsp; Is there some initialisation work being performed in the cx25840 module that I can incorporate into my patch to remove this dependency? Or should I leave it as is?<br><br>Anyway nearly bedtime here.<br><br>Regards,<br>Stephen.<br></span>
<br><br>--------Patch------<br>diff -Naur v4l-dvb/linux/Documentation/video4linux/CARDLIST.cx23885 v4l-dvb2/linux/Documentation/video4linux/CARDLIST.cx23885<br>--- v4l-dvb/linux/Documentation/video4linux/CARDLIST.cx23885&nbsp;&nbsp;&nbsp; 2008-08-05 16:48:13.000000000 +1000<br>+++ v4l-dvb2/linux/Documentation/video4linux/CARDLIST.cx23885&nbsp;&nbsp;&nbsp; 2008-08-05 20:07:16.000000000 +1000<br>@@ -10,3 +10,4 @@<br>&nbsp;&nbsp; 9 -&gt; Hauppauge WinTV-HVR1400&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; [0070:8010]<br>&nbsp; 10 -&gt; DViCO FusionHDTV7 Dual Express&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; [18ac:d618]<br>&nbsp; 11 -&gt; DViCO FusionHDTV DVB-T Dual Express&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; [18ac:db78]<br>+ 12 -&gt; Leadtek Winfast PxDVR3200 H&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; [107d:6681]<br>diff -Naur v4l-dvb/linux/drivers/media/video/cx23885/cx23885-cards.c v4l-dvb2/linux/drivers/media/video/cx23885/cx23885-cards.c<br>--- v4l-dvb/linux/drivers/media/video/cx23885/cx23885-cards.c&nbsp;&nbsp;&nbsp; 2008-08-05 16:48:14.000000000 +1000<br>+++ v4l-dvb2/linux/drivers/media/video/cx23885/cx23885-cards.c&nbsp;&nbsp;&nbsp; 2008-08-05 21:29:59.000000000 +1000<br>@@ -155,6 +155,10 @@<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .portb&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; = CX23885_MPEG_DVB,<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .portc&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; = CX23885_MPEG_DVB,<br>&nbsp;&nbsp;&nbsp;&nbsp; },<br>+&nbsp;&nbsp;&nbsp; [CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H] = {<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .name&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; = "Leadtek Winfast PxDVR3200 H",<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .portc&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; = CX23885_MPEG_DVB,<br>+&nbsp;&nbsp;&nbsp; },<br>&nbsp;};<br>&nbsp;const unsigned int cx23885_bcount = ARRAY_SIZE(cx23885_boards);<br>&nbsp;<br>@@ -230,7 +234,11 @@<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .subvendor = 0x18ac,<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .subdevice = 0xdb78,<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .card&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; = CX23885_BOARD_DVICO_FUSIONHDTV_DVB_T_DUAL_EXP,<br>-&nbsp;&nbsp;&nbsp; },<br>+&nbsp;&nbsp;&nbsp; },{<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .subvendor = 0x107d,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .subdevice = 0x6681,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .card&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; = CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H,<br>+&nbsp; &nbsp;&nbsp;&nbsp; },<br>&nbsp;};<br>&nbsp;const unsigned int cx23885_idcount = ARRAY_SIZE(cx23885_subids);<br>&nbsp;<br>@@ -353,6 +361,10 @@<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if (command == 0)<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; bitmask = 0x04;<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; break;<br>+&nbsp;&nbsp;&nbsp; case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H:<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; /* Tuner Reset Command */<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; bitmask = 0x00070404;<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; break;<br>&nbsp;&nbsp;&nbsp;&nbsp; case CX23885_BOARD_DVICO_FUSIONHDTV_7_DUAL_EXP:<br>&nbsp;&nbsp;&nbsp;&nbsp; case CX23885_BOARD_DVICO_FUSIONHDTV_DVB_T_DUAL_EXP:<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if (command == 0) {<br>@@ -492,6 +504,15 @@<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; mdelay(20);<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; cx_set(GP0_IO, 0x000f000f);<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; break;<br>+&nbsp;&nbsp;&nbsp; case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H:<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; /* GPIO-2&nbsp; xc3028 tuner reset */<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; /* Put the parts into reset and back */<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; cx_set(GP0_IO, 0x00040000);<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; mdelay(20);<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; cx_clear(GP0_IO, 0x00000004);<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; mdelay(20);<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; cx_set(GP0_IO, 0x00040004);<br>+&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; break;<br>&nbsp;&nbsp;&nbsp;&nbsp; }<br>&nbsp;}<br>&nbsp;<br>@@ -579,6 +600,7 @@<br>&nbsp;&nbsp;&nbsp;&nbsp; case CX23885_BOARD_HAUPPAUGE_HVR1200:<br>&nbsp;&nbsp;&nbsp;&nbsp; case CX23885_BOARD_HAUPPAUGE_HVR1700:<br>&nbsp;&nbsp;&nbsp;&nbsp; case CX23885_BOARD_HAUPPAUGE_HVR1400:<br>+&nbsp;&nbsp;&nbsp; case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H:<br>&nbsp;&nbsp;&nbsp;&nbsp; default:<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; ts2-&gt;gen_ctrl_val&nbsp; = 0xc; /* Serial bus + punctured clock */<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; ts2-&gt;ts_clk_en_val = 0x1; /* Enable TS_CLK */<br>@@ -592,6 +614,7 @@<br>&nbsp;&nbsp;&nbsp;&nbsp; case CX23885_BOARD_HAUPPAUGE_HVR1800:<br>&nbsp;&nbsp;&nbsp;&nbsp; case CX23885_BOARD_HAUPPAUGE_HVR1800lp:<br>&nbsp;&nbsp;&nbsp;&nbsp; case CX23885_BOARD_HAUPPAUGE_HVR1700:<br>+&nbsp;&nbsp;&nbsp; case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H:<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; request_module("cx25840");<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; break;<br>&nbsp;&nbsp;&nbsp;&nbsp; }<br>diff -Naur v4l-dvb/linux/drivers/media/video/cx23885/cx23885-dvb.c v4l-dvb2/linux/drivers/media/video/cx23885/cx23885-dvb.c<br>--- v4l-dvb/linux/drivers/media/video/cx23885/cx23885-dvb.c&nbsp;&nbsp;&nbsp; 2008-08-05 16:48:14.000000000 +1000<br>+++ v4l-dvb2/linux/drivers/media/video/cx23885/cx23885-dvb.c&nbsp;&nbsp;&nbsp; 2008-08-05 21:29:00.000000000 +1000<br>@@ -37,6 +37,7 @@<br>&nbsp;#include "tda8290.h"<br>&nbsp;#include "tda18271.h"<br>&nbsp;#include "lgdt330x.h"<br>+#include "zl10353.h"<br>&nbsp;#include "xc5000.h"<br>&nbsp;#include "tda10048.h"<br>&nbsp;#include "tuner-xc2028.h"<br>@@ -502,6 +503,32 @@<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; }<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; break;<br>&nbsp;&nbsp;&nbsp;&nbsp; }<br>+ &nbsp;&nbsp;&nbsp; case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H:<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; i2c_bus = &amp;dev-&gt;i2c_bus[0];<br>+ <br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; port-&gt;dvb.frontend = dvb_attach(zl10353_attach,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &amp;dvico_fusionhdtv_xc3028,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; &amp;i2c_bus-&gt;i2c_adap);<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if (port-&gt;dvb.frontend != NULL) {<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; struct dvb_frontend&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; *fe;<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; struct xc2028_config&nbsp;&nbsp;&nbsp; &nbsp; cfg = {<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .i2c_adap&nbsp; = &amp;dev-&gt;i2c_bus[1].i2c_adap,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .i2c_addr&nbsp; = 0x61,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .video_dev = port,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .callback&nbsp; = cx23885_tuner_callback,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; };<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; static struct xc2028_ctrl ctl = {<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .fname&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; = "xc3028-v27.fw",<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .max_len&nbsp;&nbsp;&nbsp;&nbsp; = 64,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; .demod&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; = XC3028_FE_ZARLINK456,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; };<br>+ <br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; fe = dvb_attach(xc2028_attach, port-&gt;dvb.frontend,<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &amp;cfg);<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if (fe != NULL &amp;&amp; fe-&gt;ops.tuner_ops.set_config != NULL)<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; fe-&gt;ops.tuner_ops.set_config(fe, &amp;ctl);<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; }<br>+ &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; break;<br>&nbsp;&nbsp;&nbsp;&nbsp; default:<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; printk("%s: The frontend of your DVB/ATSC card isn't supported yet\n",<br>&nbsp;&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; dev-&gt;name);<br>diff -Naur v4l-dvb/linux/drivers/media/video/cx23885/cx23885.h v4l-dvb2/linux/drivers/media/video/cx23885/cx23885.h<br>--- v4l-dvb/linux/drivers/media/video/cx23885/cx23885.h&nbsp;&nbsp;&nbsp; 2008-08-05 16:48:14.000000000 +1000<br>+++ v4l-dvb2/linux/drivers/media/video/cx23885/cx23885.h&nbsp;&nbsp;&nbsp; 2008-08-05 20:10:57.000000000 +1000<br>@@ -66,6 +66,7 @@<br>&nbsp;#define CX23885_BOARD_HAUPPAUGE_HVR1400&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 9<br>&nbsp;#define CX23885_BOARD_DVICO_FUSIONHDTV_7_DUAL_EXP 10<br>&nbsp;#define CX23885_BOARD_DVICO_FUSIONHDTV_DVB_T_DUAL_EXP 11<br>+#define CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H 12<br>&nbsp;<br>&nbsp;/* Currently unsupported by the driver: PAL/H, NTSC/Kr, SECAM B/G/H/LC */<br>&nbsp;#define CX23885_NORMS (\<br>diff -Naur v4l-dvb/linux/drivers/media/video/cx23885/Kconfig v4l-dvb2/linux/drivers/media/video/cx23885/Kconfig<br>--- v4l-dvb/linux/drivers/media/video/cx23885/Kconfig&nbsp;&nbsp;&nbsp; 2008-08-05 16:48:14.000000000 +1000<br>+++ v4l-dvb2/linux/drivers/media/video/cx23885/Kconfig&nbsp;&nbsp;&nbsp; 2008-08-05 20:06:06.000000000 +1000<br>@@ -15,6 +15,7 @@<br>&nbsp;&nbsp;&nbsp;&nbsp; select DVB_S5H1409 if !DVB_FE_CUSTOMISE<br>&nbsp;&nbsp;&nbsp;&nbsp; select DVB_S5H1411 if !DVB_FE_CUSTOMISE<br>&nbsp;&nbsp;&nbsp;&nbsp; select DVB_LGDT330X if !DVB_FE_CUSTOMISE<br>+ &nbsp;&nbsp;&nbsp; select DVB_ZL10353 if !DVB_FE_CUSTOMISE<br>&nbsp;&nbsp;&nbsp;&nbsp; select MEDIA_TUNER_XC2028 if !DVB_FE_CUSTOMIZE<br>&nbsp;&nbsp;&nbsp;&nbsp; select MEDIA_TUNER_TDA8290 if !DVB_FE_CUSTOMIZE<br>&nbsp;&nbsp;&nbsp;&nbsp; select MEDIA_TUNER_TDA18271 if !DVB_FE_CUSTOMIZE<br><br><br><br>-------End Patch-------<br>
<blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px;">----- Original Message -----<br>
From: "Steven Toth" <stoth@linuxtv.org><br>
To: stev391@email.com<br>
Subject: Re: [linux-dvb] [PATCH-TESTERS-REQUIRED] Leadtek Winfast PxDVR 3200 H - DVB Only support<br>
Date: Mon, 04 Aug 2008 10:35:20 -0400<br>
<br>

<br>
&gt;      case CX23885_BOARD_HAUPPAUGE_HVR1800:<br>
&gt;      case CX23885_BOARD_HAUPPAUGE_HVR1800lp:<br>
&gt;      case CX23885_BOARD_HAUPPAUGE_HVR1700:<br>
&gt; +    case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H:<br>
&gt;          request_module("cx25840");<br>
&gt;          break;<br>
&gt;      }<br>
<br>
Steve, thanks for look at this.<br>
<br>
I took a quick look at your patch. Obviously the callback stuff <br>
you're planning to re-work will be based Antons patch, which I plan <br>
to push tonight after more testing.... So I'm ignoring this.<br>
<br>
Minor nitpick... Don't request module cx25840 above unless you plan <br>
to use it. If you are planning to add analog support, make this a <br>
second patch after the digital stuff gets merged.<br>
<br>
Other than that, it will be great to have another product supported <br>
in the tree.<br>
<br>
Regards,<br>
<br>
- Steve<br>
</stoth@linuxtv.org></blockquote>
</div>


</div>
<BR>

-- 
<div> Be Yourself @ mail.com!<br>
Choose From 200+ Email Addresses<br>
Get a <b>Free</b> Account at <a href="http://www.mail.com/Product.aspx" target="_blank">www.mail.com</a>!</div>