[mythtv-users] Tuning problems with HD5000

[MythTV User]

> -----Original Message-----
> From: mythtv-users-bounces at mythtv.org 
> [mailto:mythtv-users-bounces at mythtv.org] On Behalf Of MythTV User
> Sent: Tuesday, September 18, 2007 2:43 PM
> To: 'Discussion about mythtv'
> Subject: Re: [mythtv-users] Tuning problems with HD5000
> 
> >>-----Original Message-----
> >>From: mythtv-users-bounces at mythtv.org
> [mailto:mythtv-users-bounces at mythtv.org] On Behalf Of 
> Boleslaw Ciesielski
> >>Sent: Tuesday, September 18, 2007 1:27 PM
> >>To: Discussion about mythtv
> >>Subject: Re: [mythtv-users] Tuning problems with HD5000 MythTV User 
> >>wrote:
> >>> 4. re-run the nvidia binary NVIDIA-Linux-x86-100.14.11-pkg1.run
> 
> >>You can just as well install nvidia rpms from atrpms.
> 
> >>Bolek
> 
> Thanks. Will consider this too.
> 
> I found this note on kernel 2.6.22.5-49
> http://www.redhat.com/archives/fedora-package-announce/2007-Se
ptember/msg002
> 19.html
> 
> - Fix Airstar hd5000 HDTV tuner
> - V4L/DVB: fix airstar hd5000 tuner
> 
> Can someone please verify if this has the fix for the tuning problem? 
> 
> 
> 

I managed to download the src RPM kernel-2.6.22.5-49.fc6.src.rpm and from
it, I've verified that the patch is included:


$ rpm -qipl kernel-2.6.22.5-49.fc6.src.rpm | grep v4l

linux-2.6-v4l-dvb-fix-airstar-hd5000-tuner.patch

Now, if only the lirc kernel module be available at ATRPMs then I will be
all set to upgrade my box. Paging, Axel, please help build
lirc-kmdl-2.6.22.5-49.fc6-xxxxxxxxxxx.rpm



$ cat linux-2.6-v4l-dvb-fix-airstar-hd5000-tuner.patch


From: Trent Piepho <xyzzy at speakeasy.org>
Date: Sun, 19 Aug 2007 08:05:54 +0000 (-0300)
Subject: V4L/DVB (6042): b2c2-flexcop: fix Airstar HD5000 tuning regression
X-Git-Url:
http://git.kernel.org/?p=linux%2Fkernel%2Fgit%2Fmchehab%2Fv4l-dvb.git;a=comm
itdiff_plain;h=6175e487e314385e37f06448847e4c46c20edb44

V4L/DVB (6042): b2c2-flexcop: fix Airstar HD5000 tuning regression

Git changeset 6bdcc6e6dbab8daffd05e5026486f34ba41a6c72 dropped the
stand-alone lgh06xf module, whose functionality was absorbed into the
dvb-pll module. However, there was a minor difference between the code
in lgh06xf and dvb-pll, which caused a regression in b2c2-flexcop
devices using the LG-H06xF NIM.

dvb-pll will probe for the presence of an i2c pll chip by performing a
single byte read, the lgh06xf driver did not do this. Unfortunately, the
code in flexcop-i2c.c does not currently support 1 byte or 0 byte reads
as a probe.  Such probes with the current code will always fail.

In order to work around this problem, and restore proper functionality
of the Airstar HD5000 device, this hack was created to make the probe
appear to succeed.  The single byte read in dvb_pll_attach is the only
place where such a probe would ever occur, so this change is safe, and
will not affect any other devices.

Of course, if one knew how to actually perform the read operation, it
would be better to go that route.  In the meantime, however, we must
apply this workaround, in order to prevent the regression that causes
tuning to fail on the Airstar HD5000 ATSC device.

Thanks to Jarod Wilson, who had originally reported this regression, and
to Geoffrey Hausheer, whose original workaround patch led us to find the
actual cause of the problem.

Signed-off-by: Trent Piepho <xyzzy at speakeasy.org>
Cc: Geoffrey Hausheer <inli3epy93n at phracturedblue.com>
Acked-by: Jarod Wilson <jwilson at redhat.com>
Signed-off-by: Michael Krufky <mkrufky at linuxtv.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab at infradead.org>
---

diff --git a/drivers/media/dvb/b2c2/flexcop-i2c.c
b/drivers/media/dvb/b2c2/flexcop-i2c.c
index 02a0ea6..6bf858a 100644
--- a/drivers/media/dvb/b2c2/flexcop-i2c.c
+++ b/drivers/media/dvb/b2c2/flexcop-i2c.c
@@ -135,6 +135,13 @@ static int flexcop_master_xfer(struct i2c_adapter
*i2c_adap, struct i2c_msg msgs
        struct flexcop_device *fc = i2c_get_adapdata(i2c_adap);
        int i, ret = 0;

+       /* Some drivers use 1 byte or 0 byte reads as probes, which this
+        * driver doesn't support.  These probes will always fail, so this
+        * hack makes them always succeed.  If one knew how, it would of
+        * course be better to actually do the read.  */
+       if (num == 1 && msgs[0].flags == I2C_M_RD && msgs[0].len <= 1)
+               return 1;
+
        if (mutex_lock_interruptible(&fc->i2c_mutex))
                return -ERESTARTSYS;