From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-lb0-f174.google.com ([209.85.217.174]:64247 "EHLO mail-lb0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752079Ab2IORjf (ORCPT ); Sat, 15 Sep 2012 13:39:35 -0400 Received: by lbbgj3 with SMTP id gj3so3418938lbb.19 for ; Sat, 15 Sep 2012 10:39:34 -0700 (PDT) Message-ID: <5054BD53.7060109@gmail.com> Date: Sat, 15 Sep 2012 19:39:31 +0200 From: Anders Thomson MIME-Version: 1.0 To: Mauro Carvalho Chehab CC: "linux-media@vger.kernel.org" Subject: Re: tda8290 regression fix References: <503F4E19.1050700@gmail.com> <20120915133417.27cb82a1@redhat.com> In-Reply-To: <20120915133417.27cb82a1@redhat.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: 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 Control: I/O- Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B- DisINTx- Status: Cap+ 66MHz- UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- SERR-