diff mbox

[Resend] cxusb, d680 dmbth use unified lgs8gxx code instead of lgs8gl5

Message ID 15ed362e0906101016g13b81df6h1282e3bd410928b2@mail.gmail.com (mailing list archive)
State Accepted
Delegated to: Douglas Landgraf
Headers show

Commit Message

David Wong June 10, 2009, 5:16 p.m. UTC
Use unified lgs8gxx frontend instead of reverse engineered lgs8gl5 frontend.
After this patch, lgs8gl5 frontend could be mark as deprecated.
Future development should base on unified lgs8gxx frontend.

Signed-off-by: David T.L. Wong <davidtlwong <at> gmail.com>

Comments

Mauro Carvalho Chehab June 16, 2009, 7:05 p.m. UTC | #1
Hi David,

Em Thu, 11 Jun 2009 01:16:13 +0800
David Wong <davidtlwong@gmail.com> escreveu:

> Use unified lgs8gxx frontend instead of reverse engineered lgs8gl5 frontend.
> After this patch, lgs8gl5 frontend could be mark as deprecated.
> Future development should base on unified lgs8gxx frontend.
> 
> Signed-off-by: David T.L. Wong <davidtlwong <at> gmail.com>

Your patch makes sense. Have you tested it with the Magic-Pro DMB-TH usb stick? 

Michael and Timothy,

Can you check if the new frontend module works with the currently supported
devices?



Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Timothy Lee June 24, 2009, 2:29 a.m. UTC | #2
On 06/17/2009 03:05 AM, Mauro Carvalho Chehab wrote:
> Hi David,
>
> Em Thu, 11 Jun 2009 01:16:13 +0800
> David Wong<davidtlwong@gmail.com>  escreveu:
>    
>> Use unified lgs8gxx frontend instead of reverse engineered lgs8gl5 frontend.
>> After this patch, lgs8gl5 frontend could be mark as deprecated.
>> Future development should base on unified lgs8gxx frontend.
>>
>> Signed-off-by: David T.L. Wong<davidtlwong<at>  gmail.com>
>>      
> Your patch makes sense. Have you tested it with the Magic-Pro DMB-TH usb stick?
>
> Michael and Timothy,
>
> Can you check if the new frontend module works with the currently supported
> devices?
>    
Dear all,

It's Timothy here -- I'm the original author of the lgs8gl5 module.  
Sorry for the late reply, but since my office's relocation, I've been 
having a hard time getting good TV reception, and had not been able to 
exercise the patch.

However, I've been in close contact with David, and did try an earlier 
version of his patch prior to moving, so I'm optimistic that it should work.


Regards,
Timothy Lee
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

changeset:   11831:2fe3ef06a8e2
tag:         tip
user:        marvel@DHM200
date:        Mon May 18 17:03:36 2009 +0800
summary:     cxusb: d680 switch from lgs8gl5 to unified lgs8gxx frontend driver

diff -r 32e66a2bd568 -r 2fe3ef06a8e2 linux/drivers/media/dvb/dvb-usb/cxusb.c
--- a/linux/drivers/media/dvb/dvb-usb/cxusb.c	Mon May 18 16:01:15 2009 +0800
+++ b/linux/drivers/media/dvb/dvb-usb/cxusb.c	Mon May 18 17:03:36 2009 +0800
@@ -38,7 +38,7 @@ 
 #include "mxl5005s.h"
 #include "dib7000p.h"
 #include "dib0070.h"
-#include "lgs8gl5.h"
+#include "lgs8gxx.h"
 
 /* debug */
 static int dvb_usb_cxusb_debug;
@@ -1097,8 +1097,18 @@ 
 	return -EIO;
 }
 
-static struct lgs8gl5_config lgs8gl5_cfg = {
+static struct lgs8gxx_config d680_lgs8gl5_cfg = {
+	.prod = LGS8GXX_PROD_LGS8GL5,
 	.demod_address = 0x19,
+	.serial_ts = 0,
+	.ts_clk_pol = 0,
+	.ts_clk_gated = 1,
+	.if_clk_freq = 30400, /* 30.4 MHz */
+	.if_freq = 5725, /* 5.725 MHz */
+	.if_neg_center = 0,
+	.ext_adc = 0,
+	.adc_signed = 0,
+	.if_neg_edge = 0,
 };
 
 static int cxusb_d680_dmb_frontend_attach(struct dvb_usb_adapter *adap)
@@ -1138,7 +1148,7 @@ 
 	msleep(100);
 
 	/* Attach frontend */
-	adap->fe = dvb_attach(lgs8gl5_attach, &lgs8gl5_cfg, &d->i2c_adap);
+	adap->fe = dvb_attach(lgs8gxx_attach, &d680_lgs8gl5_cfg, &d->i2c_adap);
 	if (adap->fe == NULL)
 		return -EIO;