From: Antti Palosaari <crope@iki.fi>
To: Malcolm Priestley <tvboxspy@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab@redhat.com>,
linux-media <linux-media@vger.kernel.org>
Subject: Re: [PATCH FOR 3.2 FIX] af9015: limit I2C access to keep FW happy
Date: Tue, 29 Nov 2011 02:09:57 +0200 [thread overview]
Message-ID: <4ED422D5.60701@iki.fi> (raw)
In-Reply-To: <4ec1955e.e813b40a.37be.3fce@mx.google.com>
On 11/15/2011 12:25 AM, Malcolm Priestley wrote:
> I have tried this patch, while it initially got MythTV working, there is
> too many call backs and some failed to acquire the lock. The device
> became unstable on both single and dual devices.
>
> The callbacks
>
> af9015_af9013_read_status,
> af9015_af9013_init
> af9015_af9013_sleep
>
> had to be removed.
>
> I take your point, a call back can be an alternative.
>
> The patch didn't stop the firmware fails either.
>
> The af9015 usb bridge on the whole is so unstable in its early stages,
> especially on a cold boot and when the USB controller has another device
> on it, such as card reader or wifi device.
>
> I am, at the moment looking to see if the fails are due to interface 1
> being claimed by HID.
I just got af9013 rewrite ready. Feel free to test.
http://git.linuxtv.org/anttip/media_tree.git/shortlog/refs/heads/misc
It reduces typical statistics polling load maybe 1/4 from the original.
I see still small glitch when switching to channel. That seems to come
from tuner driver I2C load. There is 3 tuners used for dual devices;
MXL5005S, MXL5007T and TDA18271. I have only MXL5005S and MXL5007T dual
devices here. MXL5005S is worse than MXL5007T but both makes still
rather much I/O.
regards
Antti
--
http://palosaari.fi/
next prev parent reply other threads:[~2011-11-29 0:10 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-11-13 19:05 [PATCH FOR 3.2 FIX] af9015: limit I2C access to keep FW happy Antti Palosaari
2011-11-13 19:19 ` Antti Palosaari
2011-11-14 22:25 ` Malcolm Priestley
2011-11-14 22:56 ` Antti Palosaari
2011-11-29 0:09 ` Antti Palosaari [this message]
2011-11-30 21:31 ` Malcolm Priestley
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4ED422D5.60701@iki.fi \
--to=crope@iki.fi \
--cc=linux-media@vger.kernel.org \
--cc=mchehab@redhat.com \
--cc=tvboxspy@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox