public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mark Carbonaro <mark@carbonaro.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: Wed, 6 Aug 2008 07:50:15 +1000 (EST)	[thread overview]
Message-ID: <29520898.21217974381956.JavaMail.mark@trogdor.carbonaro.org> (raw)
In-Reply-To: <1946053.01217974303703.JavaMail.mark@trogdor.carbonaro.org>


[-- Attachment #1.1: Type: text/plain, Size: 9897 bytes --]

Stephen, 

I used the second patch you send through against http://linuxtv.org/hg/~stoth/v4l-dvb/archive/tip.tar.gz. 

The card came up first time and the drivers were loaded automatically by udev from a cold start, here is my output from dmesg 

CORE cx23885[0]: subsystem: 107d:6681, board: Leadtek Winfast PxDVR3200 H [card=12,autodetected] 
cx23885[0]: i2c bus 0 registered 
cx23885[0]: i2c bus 1 registered 
cx23885[0]: i2c bus 2 registered 
cx25840' 2-0044: cx25 0-21 found @ 0x88 (cx23885[0]) 
cx23885[0]: cx23885 based dvb card 
xc2028 1-0061: creating new instance 
xc2028 1-0061: type set to XCeive xc2028/xc3028 tuner 
DVB: registering new adapter (cx23885[0]) 
DVB: registering frontend 0 (Zarlink ZL10353 DVB-T)... 
cx23885_dev_checkrevision() Hardware revision = 0xb0 
cx23885[0]/0: found at 0000:03:00.0, rev: 2, irq: 16, latency: 0, mmio: 0xd3000000 

I have to head to work now, but I will so some testing this evening now that the driver seems to be working fine. 

Mark 

----- Original Message ----- 
From: stev391@email.com 
To: "Mark Carbonaro" <mark@carbonaro.org>, "Jonathan Hummel" <jhhummel@bigpond.com> 
Cc: stev391@email.com, linux-dvb@linuxtv.org 
Sent: Wednesday, 6 August, 2008 7:33:49 AM (GMT+1000) Auto-Detected 
Subject: Re: [PATCH-TESTERS-REQUIRED] Leadtek Winfast PxDVR 3200 H - DVB Only support 


Mark, Jon, 

The patches I made were not against the v4l-dvb tip that is referenced in Mark's email below. I did this on purpose because there is a small amount of refactoring (recoding to make it better) being performed by Steven Toth and others. 

To get the version I used for the patch download (This is for the first initial patch [you can tell it is this one as the patch file mentions cx23885-sram in the path]): 
http://linuxtv.org/hg/~stoth/cx23885-sram/archive/tip.tar.gz 

For the second patch that emailed less then 12 hours ago download this version of drivers: 
http://linuxtv.org/hg/~stoth/v4l-dvb/archive/tip.tar.gz 
and then apply my patch (this patch mentions v4l-dvb). This version is a cleanup of the previous and uses the generic callback function. 

Other then that you are heading in the correct direction... 

Do either of you have the same issue I have that when the computer is first turned on the autodetect card feature doesn't work due to subvendor sub product ids of 0000? Or is just a faulty card that I have? 

Regards, 

Stephen. 


----- Original Message ----- 
From: "Mark Carbonaro" 
To: "Jonathan Hummel" 
Subject: Re: [PATCH-TESTERS-REQUIRED] Leadtek Winfast PxDVR 3200 H - DVB Only support 
Date: Tue, 5 Aug 2008 23:47:46 +1000 (EST) 


Hi Mark, 

Forgive my ignorance/ newbie-ness, but what do I do with that patch code 
below? is there a tutorial or howto or something somewhere that will 
introduce me to this. I have done some programming, but nothing of this 
level. 

cheers 

Jon 

----- Original Message ----- 
From: "Jonathan Hummel" 
To: "Mark Carbonaro" 
Cc: stev391@email.com, linux-dvb@linuxtv.org 
Sent: Tuesday, 5 August, 2008 10:21:11 PM (GMT+1000) Auto-Detected 
Subject: Re: [PATCH-TESTERS-REQUIRED] Leadtek Winfast PxDVR 3200 H 
- DVB Only support 

Hi Jon, 

Not a problem at all, I'm new to this myself, below is what went 
through and I may not be doing it the right way either. So if 
anyone would like to point out what I am doing wrong I would really 
appreciate it. 

The file that I downloaded was called v4l-dvb-2bade2ed7ac8.tar.bz2 
which I downloaded from 
http://linuxtv.org/hg/~stoth/v4l-dvb/archive/tip.tar.bz2, I also 
saved the patch to the same location as the download. 

The patch didn't apply for me, so I manually patched applied the 
patches and created a new diff that should hopefully work for you 
also (attached and inline below). From what I could see the 
offsets in Stephens patch were a little off for this code snapshot 
but otherwise it is all good. 

I ran the following using the attached diff... 

tar -xjf v4l-dvb-2bade2ed7ac8.tar.bz2 
cd v4l-dvb-2bade2ed7ac8 
patch -p1 < ../Leadtek.Winfast.PxDVR.3200.H.2.diff 

Once the patch was applied I was then able to build and install the 
modules as per the instructions in the INSTALL file. I ran the 
following... 

make all 
sudo make install 

>From there I could load the modules and start testing. 

I hope this helps you get started. 

Regards, 
Mark 






diff -Naur 
v4l-dvb-2bade2ed7ac8/linux/Documentation/video4linux/CARDLIST.cx23885 
v4l-dvb2/linux/Documentation/video4linux/CARDLIST.cx23885 
--- 
v4l-dvb-2bade2ed7ac8/linux/Documentation/video4linux/CARDLIST.cx23885 2008-08-05 11:18:19.000000000 
+1000 
+++ 
v4l-dvb2/linux/Documentation/video4linux/CARDLIST.cx23885 2008-08-05 
23:27:32.000000000 +1000 
@@ -10,3 +10,4 @@ 
9 -> Hauppauge WinTV-HVR1400 [0070:8010] 
10 -> DViCO FusionHDTV7 Dual Express [18ac:d618] 
11 -> DViCO FusionHDTV DVB-T Dual Express [18ac:db78] 
+ 12 -> Leadtek Winfast PxDVR3200 H [107d:6681] 
diff -Naur 
v4l-dvb-2bade2ed7ac8/linux/drivers/media/video/cx23885/Kconfig 
v4l-dvb2/linux/drivers/media/video/cx23885/Kconfig 
--- 
v4l-dvb-2bade2ed7ac8/linux/drivers/media/video/cx23885/Kconfig 2008-08-05 
11:18:19.000000000 +1000 
+++ v4l-dvb2/linux/drivers/media/video/cx23885/Kconfig 2008-08-05 
23:37:51.000000000 +1000 
@@ -15,6 +15,7 @@ 
select DVB_S5H1409 if !DVB_FE_CUSTOMISE 
select DVB_S5H1411 if !DVB_FE_CUSTOMISE 
select DVB_LGDT330X if !DVB_FE_CUSTOMISE 
+ select DVB_ZL10353 if !DVB_FE_CUSTOMISE 
select MEDIA_TUNER_XC2028 if !DVB_FE_CUSTOMIZE 
select MEDIA_TUNER_TDA8290 if !DVB_FE_CUSTOMIZE 
select MEDIA_TUNER_TDA18271 if !DVB_FE_CUSTOMIZE 
diff -Naur 
v4l-dvb-2bade2ed7ac8/linux/drivers/media/video/cx23885/cx23885-cards.c 
v4l-dvb2/linux/drivers/media/video/cx23885/cx23885-cards.c 
--- 
v4l-dvb-2bade2ed7ac8/linux/drivers/media/video/cx23885/cx23885-cards.c 2008-08-05 11:18:19.000000000 
+1000 
+++ 
v4l-dvb2/linux/drivers/media/video/cx23885/cx23885-cards.c 2008-08-05 
23:41:40.000000000 +1000 
@@ -155,6 +155,10 @@ 
.portb = CX23885_MPEG_DVB, 
.portc = CX23885_MPEG_DVB, 
}, 
+ [CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H] = { 
+ .name = "Leadtek Winfast PxDVR3200 H", 
+ .portc = CX23885_MPEG_DVB, 
+ }, 
}; 
const unsigned int cx23885_bcount = ARRAY_SIZE(cx23885_boards); 

@@ -230,6 +234,10 @@ 
.subvendor = 0x18ac, 
.subdevice = 0xdb78, 
.card = CX23885_BOARD_DVICO_FUSIONHDTV_DVB_T_DUAL_EXP, 
+ },{ 
+ .subvendor = 0x107d, 
+ .subdevice = 0x6681, 
+ .card = CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H, 
}, 
}; 
const unsigned int cx23885_idcount = ARRAY_SIZE(cx23885_subids); 
@@ -353,6 +361,10 @@ 
if (command == 0) 
bitmask = 0x04; 
break; 
+ case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H: 
+ /* Tuner Reset Command */ 
+ bitmask = 0x00070404; 
+ break; 
case CX23885_BOARD_DVICO_FUSIONHDTV_7_DUAL_EXP: 
case CX23885_BOARD_DVICO_FUSIONHDTV_DVB_T_DUAL_EXP: 
if (command == 0) { 
@@ -492,6 +504,15 @@ 
mdelay(20); 
cx_set(GP0_IO, 0x000f000f); 
break; 
+ case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H: 
+ /* GPIO-2 xc3028 tuner reset */ 
+ /* Put the parts into reset and back */ 
+ cx_set(GP0_IO, 0x00040000); 
+ mdelay(20); 
+ cx_clear(GP0_IO, 0x00000004); 
+ mdelay(20); 
+ cx_set(GP0_IO, 0x00040004); 
+ break; 
} 
} 

@@ -579,6 +600,7 @@ 
case CX23885_BOARD_HAUPPAUGE_HVR1200: 
case CX23885_BOARD_HAUPPAUGE_HVR1700: 
case CX23885_BOARD_HAUPPAUGE_HVR1400: 
+ case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H: 
default: 
ts2->gen_ctrl_val = 0xc; /* Serial bus + punctured clock */ 
ts2->ts_clk_en_val = 0x1; /* Enable TS_CLK */ 
@@ -592,6 +614,7 @@ 
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; 
} 
diff -Naur 
v4l-dvb-2bade2ed7ac8/linux/drivers/media/video/cx23885/cx23885-dvb.c 
v4l-dvb2/linux/drivers/media/video/cx23885/cx23885-dvb.c 
--- 
v4l-dvb-2bade2ed7ac8/linux/drivers/media/video/cx23885/cx23885-dvb.c 2008-08-05 11:18:19.000000000 
+1000 
+++ 
v4l-dvb2/linux/drivers/media/video/cx23885/cx23885-dvb.c 2008-08-05 
23:37:03.000000000 +1000 
@@ -37,6 +37,7 @@ 
#include "tda8290.h" 
#include "tda18271.h" 
#include "lgdt330x.h" 
+#include "zl10353.h" 
#include "xc5000.h" 
#include "tda10048.h" 
#include "tuner-xc2028.h" 
@@ -502,6 +503,32 @@ 
} 
break; 
} 
+ case CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H: 
+ i2c_bus = &dev->i2c_bus[0]; 
+ 
+ port->dvb.frontend = dvb_attach(zl10353_attach, 
+ &dvico_fusionhdtv_xc3028, 
+ &i2c_bus->i2c_adap); 
+ if (port->dvb.frontend != NULL) { 
+ struct dvb_frontend *fe; 
+ struct xc2028_config cfg = { 
+ .i2c_adap = &dev->i2c_bus[1].i2c_adap, 
+ .i2c_addr = 0x61, 
+ .video_dev = port, 
+ .callback = cx23885_tuner_callback, 
+ }; 
+ static struct xc2028_ctrl ctl = { 
+ .fname = "xc3028-v27.fw", 
+ .max_len = 64, 
+ .demod = XC3028_FE_ZARLINK456, 
+ }; 
+ 
+ fe = dvb_attach(xc2028_attach, port->dvb.frontend, 
+ &cfg); 
+ if (fe != NULL && fe->ops.tuner_ops.set_config != NULL) 
+ fe->ops.tuner_ops.set_config(fe, &ctl); 
+ } 
+ break; 
default: 
printk("%s: The frontend of your DVB/ATSC card isn't supported yet\n", 
dev->name); 
diff -Naur 
v4l-dvb-2bade2ed7ac8/linux/drivers/media/video/cx23885/cx23885.h 
v4l-dvb2/linux/drivers/media/video/cx23885/cx23885.h 
--- 
v4l-dvb-2bade2ed7ac8/linux/drivers/media/video/cx23885/cx23885.h 2008-08-05 
11:18:19.000000000 +1000 
+++ v4l-dvb2/linux/drivers/media/video/cx23885/cx23885.h 2008-08-05 
23:37:33.000000000 +1000 
@@ -66,6 +66,7 @@ 
#define CX23885_BOARD_HAUPPAUGE_HVR1400 9 
#define CX23885_BOARD_DVICO_FUSIONHDTV_7_DUAL_EXP 10 
#define CX23885_BOARD_DVICO_FUSIONHDTV_DVB_T_DUAL_EXP 11 
+#define CX23885_BOARD_LEADTEK_WINFAST_PXDVR3200_H 12 

/* Currently unsupported by the driver: PAL/H, NTSC/Kr, SECAM B/G/H/LC */ 
#define CX23885_NORMS (\ 
<< Leadtek.Winfast.PxDVR.3200.H.2.diff >> 

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

[-- Attachment #1.2: Type: text/html, Size: 11974 bytes --]

[-- Attachment #2: Type: text/plain, Size: 150 bytes --]

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

       reply	other threads:[~2008-08-05 22:13 UTC|newest]

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