Re: tda8290 regression fix

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Em Sat, 15 Sep 2012 19:39:31 +0200
Anders Thomson <aeriksson2@xxxxxxxxx> escreveu:

> On 2012-09-15 18:34, Mauro Carvalho Chehab wrote:
> > >  $ cat /TV_CARD.diff
> > >  diff --git a/drivers/media/common/tuners/tda8290.c
> > >  b/drivers/media/common/tuners/tda8290.c
> > >  index 064d14c..498cc7b 100644
> > >  --- a/drivers/media/common/tuners/tda8290.c
> > >  +++ b/drivers/media/common/tuners/tda8290.c
> > >  @@ -635,7 +635,11 @@ static int tda829x_find_tuner(struct dvb_frontend *fe)
> > >
> > >                   dvb_attach(tda827x_attach, fe, priv->tda827x_addr,
> > >                              priv->i2c_props.adap,&priv->cfg);
> > >  +               tuner_info("ANDERS: setting switch_addr. was 0x%02x, new
> > >  0x%02x\n",priv->cfg.switch_addr,priv->i2c_props.addr);
> > >                   priv->cfg.switch_addr = priv->i2c_props.addr;
> > >  +               priv->cfg.switch_addr = 0xc2 / 2;
> >
> > No, this is wrong. The I2C address is passed by the bridge driver or by
> > the tuner_core attachment, being stored at priv->i2c_props.addr.
> >
> > What's the driver and card you're using?
> >
> lspci -vv:
> 03:06.0 Multimedia controller: Philips Semiconductors 
> SAA7131/SAA7133/SAA7135 Video Broadcast Decoder (rev d1)
>          Subsystem: Pinnacle Systems Inc. Device 002f

There are lots of Pinnacle device supported by saa7134 driver. Without its
PCI ID that's not much we can do.

Also, please post the dmesg showing what happens without and with your patch.

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


[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux