[linux-dvb] [PATCH] lgdt330x: fix QAM symbol_rate_min

[Michael Krufky]

On Monday 17 October 2005 09:00 am, Steve Malenfant wrote:

>Quick question, why is the symbol rate for the lgdt330x
>is : .symbol_rate_min = 10762000,
>.symbol_rate_max = 10762000,
>
>Shouldn't this be :
>.symbol_rate_min = 5056941, (QAM_64)
>.symbol_rate_max = 10762000, (VSB_8)
>
>I didn't know that VSB_8 was using half the bandwidth of
>the symbol, I need to look into this.
>
>Steve M.
>  
>
Mac Michaels wrote:

>Here is a patch:
>
Mac-

Your patch has only applied this change to lgdt3302_ops ...  I've 
revised your patch to apply the same change to lgdt3303_ops (attached)

Would you like me to include your sign-off when I apply this to cvs?  
Please provide it in your reply.

Thank you.

-- 

 lgdt330x.c |   10 ++++------
 1 file changed, 4 insertions(+), 6 deletions(-)

Signed-off-by: Michael Krufky <mkrufky at m1k.net>

-------------- next part --------------
Index: linux/drivers/media/dvb/frontends/lgdt330x.c
===================================================================
RCS file: /cvs/linuxtv/dvb-kernel/linux/drivers/media/dvb/frontends/lgdt330x.c,v
retrieving revision 1.11
diff -u -p -r1.11 lgdt330x.c
--- linux/drivers/media/dvb/frontends/lgdt330x.c	15 Oct 2005 17:48:57 -0000	1.11
+++ linux/drivers/media/dvb/frontends/lgdt330x.c	18 Oct 2005 00:04:43 -0000
@@ -772,9 +772,8 @@ static struct dvb_frontend_ops lgdt3302_
 		.frequency_min= 54000000,
 		.frequency_max= 858000000,
 		.frequency_stepsize= 62500,
-		/* Symbol rate is for all VSB modes need to check QAM */
-		.symbol_rate_min    = 10762000,
-		.symbol_rate_max    = 10762000,
+		.symbol_rate_min    = 5056941,	/* QAM 64 */
+		.symbol_rate_max    = 10762000,	/* VSB 8  */
 		.caps = FE_CAN_QAM_64 | FE_CAN_QAM_256 | FE_CAN_8VSB
 	},
 	.init                 = lgdt330x_init,
@@ -796,9 +795,8 @@ static struct dvb_frontend_ops lgdt3303_
 		.frequency_min= 54000000,
 		.frequency_max= 858000000,
 		.frequency_stepsize= 62500,
-		/* Symbol rate is for all VSB modes need to check QAM */
-		.symbol_rate_min    = 10762000,
-		.symbol_rate_max    = 10762000,
+		.symbol_rate_min    = 5056941,	/* QAM 64 */
+		.symbol_rate_max    = 10762000,	/* VSB 8  */
 		.caps = FE_CAN_QAM_64 | FE_CAN_QAM_256 | FE_CAN_8VSB
 	},
 	.init                 = lgdt330x_init,