mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Johannes Stezenbach <js@linuxtv.org>
To: Markus Rechberger <mrechberger@gmail.com>
Cc: Mauro Carvalho Chehab <mchehab@infradead.org>,
	"linux-dvb@linuxtv.org" <linux-dvb@linuxtv.org>,
	video4linux-list@redhat.com, akpm@linux-foundation.org,
	linux-kernel@vger.kernel.org
Subject: Re: [linux-dvb] [PATCH] Userspace tuner
Date: Thu, 13 Sep 2007 15:13:53 +0200	[thread overview]
Message-ID: <20070913131353.GB26972@linuxtv.org> (raw)
In-Reply-To: <d9def9db0709121610n1a78225cs26380b7ca586344a@mail.gmail.com>

On Thu, Sep 13, 2007, Markus Rechberger wrote:
> Let's add the LKML to this.
> 
> On 9/13/07, Markus Rechberger <mrechberger@gmail.com> wrote:
> > On 9/12/07, Mauro Carvalho Chehab <mchehab@infradead.org> wrote:
> > > I don't see any technical reason why tuner drivers should be moved to
> > > userspace. Looking at xc3028 device, the driver is very simple and
> > > doesn't require any special treatment that it isn't possible to be done
> > > at kernel. There are already some implementations on kernelspace that
> > > works fine.
> >
> > As from my side to support the xceive driver properly it needs a
> > rewrite and a proper API description. Since it's not possible to
> > discuss any API changes

Not possible? We're doing it all the time...

However, your ideas were rejected in this discussion,
and you can't seem to get over it.

> > don't get me wrong but the existing community is rather small and
> > kicking off people who are interested in changing things.

IMHO there is a lack of openness caused by people being burned
in past flamewars. This makes it a bit difficult to see through
what happens and why, and to participate.  However, I think it
is completely wrong to say that the community is "kicking off people".

> > I'm against how the project works out at the moment and how it worked
> > out in history. Exactly this way will kick off companies to be
> > interested in future like Avermedia. A driver can easily be written
> > within a few weeks and I've been struggling with it for 2 years(!!!)
> > now just for nothing finally telling me that some guys want to steal
> > my code and move it to kernelspace although it would raise more
> > complications with upcoming and current devices which have even more
> > requirements.

Oh dear, there we go again... more flame bait...

I reality, 95% of your driver code could have been merged
without problems, but you refused to take the small, objectionable
part out of the picture.

(For those interested:
http://mcentral.de/~mrec/patches/v4l-dvb/hg_v4l-dvb-experimental_01.patch
The patch changed the internal tuner API and required changes
to all tuner drivers.)

Your all-or-nothing approach didn't work out.

Out of curiosity: How does your userspace approach solve
the hybrid (analog/digital TV) tuner problem which was the
only objectionable part of your work? And why are the kernel
parts of your new interface now less objectionable? What changed?


Johannes

  parent reply	other threads:[~2007-09-13 13:14 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     ` 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       ` Johannes Stezenbach [this message]
2007-09-13 14:12         ` [linux-dvb] " 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
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=20070913131353.GB26972@linuxtv.org \
    --to=js@linuxtv.org \
    --cc=akpm@linux-foundation.org \
    --cc=linux-dvb@linuxtv.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mchehab@infradead.org \
    --cc=mrechberger@gmail.com \
    --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

all inboxes | Powered by JetHome®