<div>
Steve,<br><br>That must have slipped in from my effort to get the analog support for the card.&nbsp; I will remove that when I go through and redo the callback.<br><br>Thanks for your comments.<br><br>Regards,<br>Stephen.<br>
<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>
<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>