Re: [PATCH-TESTERS-REQUIRED] Leadtek Winfast PxDVR 3200 H - DVB Only support

[stev391@xxxxxxxxx]

Steve,

That must have slipped in from my effort to get the analog support for the card.  I will remove that when I go through and redo the callback.

Thanks for your comments.

Regards,
Stephen.

----- Original Message -----
From: "Steven Toth"
To: stev391@xxxxxxxxx
Subject: Re: [PATCH-TESTERS-REQUIRED] Leadtek Winfast PxDVR 3200 H - DVB Only support
Date: Mon, 04 Aug 2008 10:35:20 -0400


> case CX23885_BOARD_HAUPPAUGE_HVR1800:
> case CX23885_BOARD_HAUPPAUGE_HVR1800lp:
> case CX23885_BOARD_HAUPPAUGE_HVR1700:
> + case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H:
> request_module("cx25840");
> break;
> }

Steve, thanks for look at this.

I took a quick look at your patch. Obviously the callback stuff
you're planning to re-work will be based Antons patch, which I plan
to push tonight after more testing.... So I'm ignoring this.

Minor nitpick... Don't request module cx25840 above unless you plan
to use it. If you are planning to add analog support, make this a
second patch after the digital stuff gets merged.

Other than that, it will be great to have another product supported
in the tree.

Regards,

- Steve

--

Be Yourself @ mail.com!
Choose From 200+ Email Addresses
Get a Free Account at www.mail.com!

_______________________________________________
linux-dvb mailing list
linux-dvb@xxxxxxxxxxx
http://www.linuxtv.org/cgi-bin/mailman/listinfo/linux-dvb