public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Steven Toth <stoth@linuxtv.org>
To: stev391@email.com
Cc: linux-dvb@linuxtv.org
Subject: Re: [linux-dvb] [PATCH-TESTERS-REQUIRED] Leadtek Winfast PxDVR 3200 H - DVB Only support
Date: Mon, 04 Aug 2008 10:35:20 -0400	[thread overview]
Message-ID: <489713A8.9080007@linuxtv.org> (raw)
In-Reply-To: <20080804113406.44F511BF28D@ws1-1.us4.outblaze.com>

>      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

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

  reply	other threads:[~2008-08-04 14:35 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-08-04 11:34 [linux-dvb] [PATCH-TESTERS-REQUIRED] Leadtek Winfast PxDVR 3200 H - DVB Only support stev391
2008-08-04 14:35 ` Steven Toth [this message]
  -- strict thread matches above, loose matches on Subject: below --
2008-08-04 21:15 stev391
     [not found] <1292178.61217937203262.JavaMail.mark@trogdor.carbonaro.org>
2008-08-05 11:34 ` Mark Carbonaro
2008-08-05 12:21   ` Jonathan Hummel
2008-08-05 11:47 stev391
2008-08-05 14:30 ` Steven Toth
     [not found] <24526361.241217944963449.JavaMail.mark@trogdor.carbonaro.org>
2008-08-05 13:47 ` Mark Carbonaro
2008-08-05 21:33 stev391
2008-08-05 21:37 ` Steven Toth
2008-08-06 11:27 ` Mark Carbonaro
     [not found] <1946053.01217974303703.JavaMail.mark@trogdor.carbonaro.org>
2008-08-05 21:50 ` Mark Carbonaro
2008-08-10  1:42 stev391
2008-08-11 13:36 ` Jonathan Hummel
2008-08-10  5:31 stev391
2008-08-10 13:26 ` Steven Toth
2008-08-11 23:59 stev391
2008-08-13 12:46 ` Jonathan Hummel
2008-08-12  4:22 stev391
2008-08-12 16:33 ` Steven Toth
2008-08-13  8:02 stev391
2008-08-13 21:49 stev391
2008-08-14 11:18 ` Jonathan Hummel
2008-08-14 23:36 stev391
2008-08-15 13:49 ` Jonathan Hummel
2008-08-16  1:35 stev391
2008-08-16  4:14 ` Robert Golding
2008-08-16 10:46   ` Jonathan Hummel
2008-08-17  4:01     ` Robert Golding
2008-08-16  5:14 stev391
2008-08-18  0:00 stev391
2008-08-18  4:25 ` Robert Golding
2008-08-18 21:24 stev391
2008-08-26  6:57 stev391
2008-08-26 14:24 ` Steven Toth

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=489713A8.9080007@linuxtv.org \
    --to=stoth@linuxtv.org \
    --cc=linux-dvb@linuxtv.org \
    --cc=stev391@email.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox