From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Markus Rechberger <mrechberger@gmail.com>
Cc: Johannes Stezenbach <js@linuxtv.org>,
"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: Sat, 15 Sep 2007 00:09:08 -0300 [thread overview]
Message-ID: <1189825748.5359.77.camel@gaivota> (raw)
In-Reply-To: <d9def9db0709141849r4dc489c2h88ac99e3d4f56d50@mail.gmail.com>
> The main discussion in this thread was about drivers in userspace
> are bad because the API will allow binary drivers.
No. The focus is that userspace API is not needed at all, and the
community believe that this is a regression from all efforts that are
being done by the community to have good quality OSS software.
> The guy
> who works for Hauppauge (again I also have good contacts
> at Hauppauge Europe) writes it's bad - for no technical reason.
Please stop with personal attacks.
> I (again) don't intend to
> release binary drivers. (also keep in mind that ubuntu takes
> everything which makes things work - this matters to the enduser)
Markus, you are thinking that all the community are fool? You used to
state on your website that your intention were to release binary-only
xc5000 drivers. So, please stop with childish and assume what you've
said.
> Mauro is copying the logic of my code and writes I told him I'm ok with
> taking my code without just adding a single line of how his driver
> got developed. (I still wonder why he skipped some significant parts
> of the driver .. because he used the existing one as logic template)
>
> http://linuxtv.org/hg/~mchehab/tm6000-new/log/14352ab89146/linux/drivers/media/video/tuner-xc2028.c
The basis for tuner-xc2028 driver is tea5767. The xc2028/xc3028 drivers
is very simple:
a) load the proper firmware;
b) send one 32 bits command to select frequency + the frequency divider.
All the rest is the common logic of a tuner driver.
If you take a look at my driver, you will see that the implementation is
different, providing also those functionalities:
- provides a sync during frequency setting, needed by tm6000;
- has the logic to retrieve signal status;
- part of the firmware need to be reload every time you change a freq
(tm6000 driver needs it);
- supports just the firmwares I've identified as being used by tm6000
driver;
The only thing I used is your usbreplay.pl, as properly stated at
README.first (properly pointing to your site).
Again, please stop with personal attacks. This leads to nowhere.
---
>From my side, I've nacked your userspace tuner.
However, I keep open to accept your kernelspace em28xx/xc3028 drivers,
providing that they fits at the current V4L/DVB core.
Changes at common APIs, and especially at Kernel to userspace API should
be discussed with the community. If you accept this fact, you may also
propose improvements at the APIs.
If, after all that were discussed, you're willing to do a serious work,
please send us the patches for em28xx/xc3028 kernelspace drivers.
Otherwise, I'll kindly ask you to take your own way and stop with those
flamewars.
Cheers,
Mauro
next prev parent reply other threads:[~2007-09-15 3:09 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
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 [this message]
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=1189825748.5359.77.camel@gaivota \
--to=mchehab@infradead.org \
--cc=akpm@linux-foundation.org \
--cc=js@linuxtv.org \
--cc=linux-dvb@linuxtv.org \
--cc=linux-kernel@vger.kernel.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