public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Aidan Thornton <makosoft@googlemail.com>
Cc: Michael Krufky <mkrufky@linuxtv.org>,
	Manu Abraham <abraham.manu@gmail.com>,
	"video4linux-list@redhat.com" <video4linux-list@redhat.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
	"linux-dvb@linuxtv.org" <linux-dvb@linuxtv.org>
Subject: Re: [linux-dvb] [PATCH] Userspace tuner
Date: Fri, 14 Sep 2007 18:16:09 -0300	[thread overview]
Message-ID: <1189804569.2363.93.camel@gaivota> (raw)
In-Reply-To: <c8b4dbe10709141407p757d0909s5a4d49a7eb44d929@mail.gmail.com>


> > There is no reason why the Xceive driver cannot be merged into the
> > current development tree using the hybrid tuner framework as it stands
> > today.
> 
> I'm not convinced this is entirely true. In order to avoid unnecessary
> reinitialisation of the device, the driver needs to know whether the
> device is in analog or digital mode, and I can't see a way of doing it
> with the current API. (I think existing drivers, such as the xc2028
> driver in one branch, use the older analog API and make the digital
> driver a wrapper around it.) Again, I may be missing something.

For sure there are some ways. One dirty way would be to add an static
lock at xc3028 driver. You can rise the lock when firmware is being
uploaded, removing it at the end. This would prevent the troubles you've
mentioned.

A cleaner way would be to create a tasklet for firmware upload. This
will also prevent the driver for a long load time, due to firmware
initialization (a similar change were recently added at ivtv driver).

For sure there are other ways of doing this.
 
Cheers,
Mauro


  reply	other threads:[~2007-09-14 21:16 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <46C1BCC5.9090709@amd.com>
     [not found] ` <1189626560.5160.57.camel@gaivota>
     [not found]   ` <d9def9db0709121546h67d3d1ceuf001ab1d798fde92@mail.gmail.com>
2007-09-12 23:10     ` [linux-dvb] [PATCH] Userspace tuner Markus Rechberger
2007-09-13  4:58       ` Dâniel Fraga
     [not found]         ` <d9def9db0709122258l4b05546dq2e370e67a2a1404@mail.gmail.com>
2007-09-13 12:40           ` Johannes Stezenbach
2007-09-13 13:17             ` Markus Rechberger
2007-09-17 22:05         ` Bill Davidsen
2007-09-13 13:13       ` [linux-dvb] " Johannes Stezenbach
2007-09-13 14:12         ` Markus Rechberger
2007-09-13 15:50           ` Manu Abraham
2007-09-13 16:08             ` Markus Rechberger
2007-09-13 16:22               ` Manu Abraham
2007-09-13 16:32                 ` Markus Rechberger
2007-09-13 16:52                   ` Manu Abraham
2007-09-13 20:36               ` Steven Toth
2007-09-13 23:17                 ` Markus Rechberger
2007-09-14  1:03                   ` Steven Toth
2007-09-14  6:38                     ` Markus Rechberger
2007-09-14  7:57                       ` Manu Abraham
2007-09-14  9:15                         ` Joerg Roedel
2007-09-14 12:00                           ` Manu Abraham
2007-09-14 16:09                             ` Markus Rechberger
2007-09-14 18:52                               ` Alex Deucher
2007-09-14 18:59                                 ` Markus Rechberger
2007-09-14 19:38                                   ` Alex Deucher
2007-09-14 16:13                             ` Joerg Roedel
2007-09-14 16:56                               ` Manu Abraham
2007-09-14 17:40                                 ` Joerg Roedel
2007-09-14 11:38                       ` Johannes Stezenbach
2007-09-14 16:36                         ` Markus Rechberger
2007-09-14 17:32                           ` Mauro Carvalho Chehab
2007-09-14 17:46                             ` Markus Rechberger
2007-09-14 18:29                               ` Mauro Carvalho Chehab
2007-09-14 19:20                                 ` Michael Krufky
2007-09-14 21:07                                   ` Aidan Thornton
2007-09-14 21:16                                     ` Mauro Carvalho Chehab [this message]
2007-09-14 21:53                                     ` Manu Abraham
2007-09-14 20:43                           ` Johannes Stezenbach
2007-09-15  1:29                             ` Markus Rechberger
2007-09-15  1:49                               ` Mauro Carvalho Chehab
2007-09-15 13:16                               ` Johannes Stezenbach
2007-09-15 13:38                                 ` Markus Rechberger
2007-09-15 14:04                                   ` Mauro Carvalho Chehab
2007-09-15 14:33                                     ` Markus Rechberger
2007-09-15 14:45                                       ` Mauro Carvalho Chehab
2007-09-15 16:58                                     ` Bernard Jungen
2007-09-16 11:30                                       ` Hans Verkuil
2007-09-17  8:46                                       ` Markus Rechberger
2007-09-15  1:49                             ` Markus Rechberger
2007-09-15  3:09                               ` Mauro Carvalho Chehab
2007-09-15 12:59                                 ` Markus Rechberger
2007-09-15  8:52                               ` Manu Abraham
2007-09-15 13:34                               ` Johannes Stezenbach
2007-09-15 14:56                                 ` Steven Toth
2007-09-14 13:52                   ` Alan Cox
2007-09-14 16:18                     ` Markus Rechberger
2007-09-18  8:19                       ` Jelle Foks
2007-09-18  8:55                         ` Markus Rechberger
2007-09-18 22:56                         ` Mauro Carvalho Chehab
2007-09-18 23:00                           ` Alan Cox
2007-09-14  0:49                 ` hermann pitton
2007-09-14 12:10                 ` Alan Cox

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=1189804569.2363.93.camel@gaivota \
    --to=mchehab@infradead.org \
    --cc=abraham.manu@gmail.com \
    --cc=akpm@linux-foundation.org \
    --cc=linux-dvb@linuxtv.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=makosoft@googlemail.com \
    --cc=mkrufky@linuxtv.org \
    --cc=video4linux-list@redhat.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