Re: tda8290 regression fix

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

 



On 2012-09-15 19:58, Mauro Carvalho Chehab wrote:
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.
That here, right?
lspci -nvv:
03:06.0 0480: 1131:7133 (rev d1)
        Subsystem: 11bd:002f
Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR- <PERR- INTx-
        Latency: 64 (21000ns min, 8000ns max)
        Interrupt: pin A routed to IRQ 21
        Region 0: Memory at fdeff000 (32-bit, non-prefetchable) [size=2K]
        Capabilities: [40] Power Management version 2
Flags: PMEClk- DSI- D1+ D2+ AuxCurrent=0mA PME(D0-,D1-,D2-,D3hot-,D3cold-)
                Status: D0 NoSoftRst- PME-Enable- DSel=0 DScale=1 PME-
        Kernel driver in use: saa7134
        Kernel modules: saa7134




Also, please post the dmesg showing what happens without and with your patch.
Coming. Hold on...
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