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@xxxxxxx> -------------- 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,
