Re: [PATCH 02/12] [media] dvb-pll: Add support for THOMSON DTT7546X tuner.

Peter Griffin
2015-07-30T09:46:19Z

Hi Mauro,

On Wed, 22 Jul 2015, Mauro Carvalho Chehab wrote:

> Em Wed, 24 Jun 2015 16:11:00 +0100
> Peter Griffin <peter.griffin@linaro.org> escreveu:
>
> > This is used in conjunction with the STV0367 demodulator on
> > the STV0367-NIM-V1.0 NIM card which can be used with the STi
> > STB SoC's.
> >
> > This tuner has a fifth register, so some changes have been made
> > to accommodate this.
> >
> > Signed-off-by: Peter Griffin <peter.griffin@linaro.org>
> > ---
> > drivers/media/dvb-frontends/dvb-pll.c | 74 +++++++++++++++++++++++++++++------
> > drivers/media/dvb-frontends/dvb-pll.h | 1 +
> > 2 files changed, 64 insertions(+), 11 deletions(-)
> >
> > diff --git a/drivers/media/dvb-frontends/dvb-pll.c b/drivers/media/dvb-frontends/dvb-pll.c
> > index 6d8fe88..f7381c7 100644
> > --- a/drivers/media/dvb-frontends/dvb-pll.c
> > +++ b/drivers/media/dvb-frontends/dvb-pll.c
> > @@ -141,6 +141,35 @@ static struct dvb_pll_desc dvb_pll_thomson_dtt7520x = {
> > },
> > };
> >
> > +static void thomson_dtt7546x_bw(struct dvb_frontend *fe, u8 *buf)
> > +{
> > + /* set CB2 reg - set ATC, XTO */
> > + buf[4] = 0xc3;
> > +}
> > +
> > +static struct dvb_pll_desc dvb_pll_thomson_dtt7546x = {
> > + .name = "Thomson dtt7546x",
> > + .min = 44250000,
> > + .max = 863250000,
> > + .set = thomson_dtt7546x_bw,
> > + .iffreq= 36166667,
>
> Whitespace is missing. Please check the patchs with scripts/checkpatch.pl.

Will fix in V2.

regards,

Peter.