From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from mail-in-07.arcor-online.net ([151.189.21.47]:42506 "EHLO mail-in-07.arcor-online.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755984Ab0DAAcP (ORCPT ); Wed, 31 Mar 2010 20:32:15 -0400 Subject: Re: [PATCH] Fix default state Beholder H6 tuner. From: hermann pitton To: Dmitri Belimov Cc: linux-media@vger.kernel.org, Mauro Carvalho Chehab In-Reply-To: <20100331131407.741b7822@glory.loctelecom.ru> References: <20100330160217.52e26a33@glory.loctelecom.ru> <1269942855.3361.7.camel@pc07.localdom.local> <20100331131407.741b7822@glory.loctelecom.ru> Content-Type: text/plain Date: Thu, 01 Apr 2010 02:31:55 +0200 Message-Id: <1270081915.3227.8.camel@pc07.localdom.local> Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi Dimitry, Am Mittwoch, den 31.03.2010, 13:14 +1000 schrieb Dmitri Belimov: > Hi Hermann > > > Hi, > > > > Am Dienstag, den 30.03.2010, 16:02 +1000 schrieb Dmitri Belimov: > > > Hi > > > > > > The hybrid tuner FMD1216MEX_MK3 after cold start has disabled IF. > > > This tuner has internal I2C switch. This switch switch I2C bus > > > between DVB-T and IF part. Default state is DVB-T. When module > > > saa7134 is load it can't find IF tda9887 and disable analog TV mode. > > > > > > This patch set internal I2C switch of the tuner to IF by send > > > special value to the tuner as for receive analog TV from low band. > > > It can be usefule for other cards. > > > > > > I didn't set configure a tuner by a tuner model because this tuner > > > can has different I2C address. May be we can do it later after > > > discuss for more robust support a tuners. > > > > just as a reminder. It is the same for the FMD1216ME hybrid MK3. > > After every boot, analog mode fails with missing tda9887. > > > > Currently, after tuner modules are not independent anymore, one has to > > reload the saa7134 driver once. > > > > Relevant code in tuner.core.c. > > > > case TUNER_PHILIPS_FMD1216ME_MK3: > > buffer[0] = 0x0b; > > buffer[1] = 0xdc; > > buffer[2] = 0x9c; > > buffer[3] = 0x60; > > i2c_master_send(c, buffer, 4); > > mdelay(1); > > buffer[2] = 0x86; > > buffer[3] = 0x54; > > i2c_master_send(c, buffer, 4); > > if (!dvb_attach(simple_tuner_attach, &t->fe, > > t->i2c->adapter, t->i2c->addr, > > t->type)) goto attach_failed; > > break; > > That is good. I'll try add case TUNER_PHILIPS_FMD1216MEX_MK3 here and test. > This is much better. it wont work for any what I can tell. We were forced into such an universal looking solution, but it was broken only a short time later. I for sure don't say that this time, late 2005, it was in anyway perfect, too much random on module load orders and also duplicate address stuff around meanwhile. But, however, it seems to be blocked for a global attempt within the current schemes too. Cheers, Hermann > With my best regards, Dmitry. > > > Hermann > > > > > diff -r 1ef0265456c8 > > > linux/drivers/media/video/saa7134/saa7134-cards.c --- > > > a/linux/drivers/media/video/saa7134/saa7134-cards.c Fri Mar > > > 26 00:54:18 2010 -0300 +++ > > > b/linux/drivers/media/video/saa7134/saa7134-cards.c Sun Mar > > > 28 08:21:10 2010 -0400 @@ -7450,6 +7450,21 @@ } break; > > > } > > > + case SAA7134_BOARD_BEHOLD_H6: > > > + { > > > + u8 data[] = { 0x09, 0x9f, 0x86, 0x11}; > > > + struct i2c_msg msg = {.addr = 0x61, .flags = > > > 0, .buf = data, > > > + .len = > > > sizeof(data)}; + > > > + /* The tuner TUNER_PHILIPS_FMD1216MEX_MK3 after > > > hardware */ > > > + /* start has disabled IF and enabled DVB-T. When > > > saa7134 */ > > > + /* scan I2C devices it not detect IF tda9887 and > > > can`t */ > > > + /* watch TV without software reboot. For solve > > > this problem */ > > > + /* switch the tuner to analog TV mode > > > manually. */ > > > + if (i2c_transfer(&dev->i2c_adap, &msg, 1) != 1) > > > + printk(KERN_WARNING > > > + "%s: Unable to enable IF of > > > the tuner.\n", > > > + dev->name); > > > + break; > > > + } > > > } /* switch() */ > > > > > > /* initialize tuner */ > > > > > > Signed-off-by: Beholder Intl. Ltd. Dmitry Belimov > > > > > > > > > With my best regards, Dmitry. > >