public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: "Michael Büsch" <m@bues.ch>
To: Antti Palosaari <crope@iki.fi>
Cc: linux-media <linux-media@vger.kernel.org>
Subject: Re: [PATCH] fc0011: Reduce number of retries
Date: Mon, 7 May 2012 23:02:48 +0200	[thread overview]
Message-ID: <20120507230248.5c211f8d@milhouse> (raw)
In-Reply-To: <4FA81A23.4000102@iki.fi>

[-- Attachment #1: Type: text/plain, Size: 1149 bytes --]

On Mon, 07 May 2012 21:53:23 +0300
Antti Palosaari <crope@iki.fi> wrote:

> On 03.04.2012 18:33, Michael Büsch wrote:
> > On another thing:
> > The af9035 driver doesn't look multi-device safe. There are lots of static
> > variables around that keep device state. So it looks like this will
> > blow up if multiple devices are present in the system. Unlikely, but still... .
> > Are there any plans to fix this up?
> > If not, I'll probably take a look at this. But don't hold your breath.
> 
> I fixed what was possible by moving af9033 and af9035 configurations for 
> the state. That at least resolves most significant issues - like your 
> fc0011 tuner callback.

Cool, thanks a lot!

> But there is still some stuff in "static struct 
> dvb_usb_device_properties" which cannot be fixed. Like dynamic remote 
> configuration, dual mode, etc. I am going to make RFC for those maybe 
> even this week after some analysis is done.

Thanks. I'm currently lacking the time to do this kind of intrusive changes,
so I'm happy to see you looking into this.

-- 
Greetings, Michael.

PGP encryption is encouraged / 908D8B0E

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

      reply	other threads:[~2012-05-07 21:02 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-04-03  9:05 [PATCH] fc0011: Reduce number of retries Michael Büsch
2012-04-03  9:58 ` David Cohen
2012-04-03 10:07   ` Michael Büsch
2012-04-03 14:12     ` David Cohen
2012-04-03 15:24 ` Antti Palosaari
2012-04-03 15:33   ` Michael Büsch
2012-04-03 15:41     ` Antti Palosaari
2012-05-07 18:53     ` Antti Palosaari
2012-05-07 21:02       ` Michael Büsch [this message]

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=20120507230248.5c211f8d@milhouse \
    --to=m@bues.ch \
    --cc=crope@iki.fi \
    --cc=linux-media@vger.kernel.org \
    /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