On Mon, Feb 15, 2010 at 12:37 PM, <[email protected]> wrote: > From: Stefan Ringel <[email protected]> > > Signed-off-by: Stefan Ringel <[email protected]> > > diff --git a/drivers/media/common/tuners/tuner-xc2028.c > b/drivers/media/common/tuners/tuner-xc2028.c > index ed50168..e051caa 100644 > --- a/drivers/media/common/tuners/tuner-xc2028.c > +++ b/drivers/media/common/tuners/tuner-xc2028.c > @@ -1114,7 +1114,12 @@ static int xc2028_set_params(struct dvb_frontend *fe, > > /* All S-code tables need a 200kHz shift */ > if (priv->ctrl.demod) { > - demod = priv->ctrl.demod + 200; > + if ((priv->firm_version == 0x0306) && > + (priv->ctrl.demod == XC3028_FE_ZARLINK456) && > + ((type & DTV78) || (type & DTV8))) > + demod = priv->ctrl.demod; > + else > + demod = priv->ctrl.demod + 200; > /* > * The DTV7 S-code table needs a 700 kHz shift. > * Thanks to Terry Wu <[email protected]> for reporting > this
I would still like to better understand the origin of this change. Was the tm6000 board not locking without it? Was this change based on any documented source? What basis are you using when deciding this issue is specific only to the zl10353 and not all boards using the xc3028L? We've got a number of boards already supported which use the xc3028L, so we need to ensure there is no regression introduced in those boards just to get yours working. Devin -- Devin J. Heitmueller - Kernel Labs http://www.kernellabs.com -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
