public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Norbert van Nobelen <Norbert@edusupport.nl>
To: "Nemosoft Unv." <nemosoft@smcc.demon.nl>
Cc: Linux Kernel Mailing List <linux-kernel@vger.kernel.org>
Subject: Re: pwc+pwcx is not illegal
Date: Sun, 29 Aug 2004 19:16:26 +0200	[thread overview]
Message-ID: <200408291916.26053.Norbert@edusupport.nl> (raw)
In-Reply-To: <200408291833.37808@smcc.demon.nl>

Hi,

A part of this discussion has to do with the expiration of the NDA covering 
pwcx. Can you disclose the NDA?
Also a person on the list tried to contact the correct person within Philips. 
Can you disclose the contact person or department which you used about 3 
years ago?

Best regards,

Norbert van Nobelen

On Sunday 29 August 2004 18:33, Nemosoft Unv. wrote:
> Hello,
>
> On Sunday 29 August 2004 16:00, Alan Cox wrote:
> > On Gwe, 2004-08-27 at 20:29, Linus Torvalds wrote:
> > > So stop whining about it. The driver got removed because the author
> > > asked for it.
> >
> > Please put it back, minus the hooks so the rest of the world can use it.
>
> No, don't! There is one very practial reason for that: the utter confusion
> it will cause when suddenly PWCX cannot be loaded anymore, because users
> will assume that since PWC is in the kernel, PWCX will work too. I really
> would not like to be at the receiving end of the support mailbox when 2.6.9
> comes out with such a crippled version of PWC.
>
> That's one of the reasons I requested PWC to be removed. For me, it's also
> a matter of quality: what good is a half-baked driver in the kernel when
> you need to patch it first to get it working fully again? I don't want my
> name attached to that.
>
> > If not please remove every line of code I've even written because I
> > don't like the new attitude .. so ner..
> >
> > Point made ? We can't go around throwing out drivers because the author
> > had a tantrum.
>
> I'm not having a tantrum. If it is, it has been one in the making for 3
> years.
>
> > Its also trivial to move the decompressor to user space
> > where it should be anyway.
>
> *sigh* As I have been saying a 100 times before, it is illogical,
> cumbersome for both users and developers, and will probably take a very
> long time to adopt (notwithstanding V4L2 [*]).
>
> I mean, I still remember when the YUV->RGB conversion code was snipped from
> PWC when I supplied it for inclusing in the kernel, back in 2001. It took a
> long, long time for webcam tools to adjust their code to check for the YUV
> palette and do the conversion themselves, and _to_this_very_day_ I'm
> getting mails about programs who still don't get it right.
>
> *IF* there was a commonly accepted video "middle-layer", this would not
> pose much of a problem. But there is no such thing yet.
>
> (maybe that's something for a 2.7 kernel...)
>
> > Similarly the driver is useful without the binary stuff.
>
> True. But judging from the mails I have received the last couple of days,
> people don't really care about the binary stuff, as long as it works. They
> want to use the cam to its full potential, so PWCX is more or less a
> necessity. However, there's has now been added an extra hurdle in getting
> it work, for reasons I find questionable, and really, 3 years too late.
>
> Seriously, this probably would not have happened if, back in 2001, the
> driver was rejected on the basis of this hook (you were there, Alan...) I
> never made a secret of it, it has been in the driver from day 1 and its
> purpose was clearly spelled out. If it had been rejected, I would probably
> have just switched to '3rd party module' mode and maintained it outside the
> kernel indefinetely. I would not have liked it, but it would have been
> acceptable.
>
> Another acceptable solution would have been, if after the 'discovery' of
> the hook, Greg or anybody else had said: "Look, we really don't want this
> kind of thing in the kernel. However, since we're a bit late to react,
> we'll leave it in the 2.4 and 2.6 series, but versions beyond that
> (2.7-devel, etc) will not have PWC included in this form. In the mean time,
> we're asking you to think of a solution". Chances are the situation would
> have been fully resolved before that (and I mean fully *hint*).
>
> > Or do we need a -ac tree again where this time -ac is "added camera" ;)
>
> *lol* The code is still floating around on the Net, so nobody's stopping
> you...
>
>  - Nemosoft
>
>
> [*] Some advice: if you really want to speed up V4L2 adoption by video
> tools, start disabling V4L1 in the kernel...
>
> -
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/

  parent reply	other threads:[~2004-08-29 17:17 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-08-27 19:18 pwc+pwcx is not illegal Albert Cahalan
2004-08-27 19:29 ` Linus Torvalds
2004-08-27 20:06   ` Kenneth Lavrsen
2004-08-27 20:21     ` Linus Torvalds
2004-08-27 20:24     ` David S. Miller
2004-08-27 20:26     ` Paul Jakma
2004-08-30 17:41       ` Brian Litzinger
2004-08-27 20:38     ` David Ford
2004-08-27 20:57   ` Albert Cahalan
2004-08-27 21:04     ` Greg KH
2004-08-27 21:09     ` Linus Torvalds
2004-08-29 14:00   ` Alan Cox
2004-08-29 16:33     ` Nemosoft Unv.
2004-08-29 15:42       ` Alan Cox
2004-08-29 17:17         ` [linux-usb-devel] " Randy.Dunlap
2004-08-29 17:16       ` Norbert van Nobelen [this message]
2004-08-27 19:34 ` Paulo Marques
2004-08-27 21:34   ` Albert Cahalan
  -- strict thread matches above, loose matches on Subject: below --
2004-08-28  8:15 Gabucino
2004-08-28 10:30 ` Vojtech Pavlik
2004-08-28 12:18 Kenneth Lavrsen
2004-08-28 13:11 ` Adrian Bunk
2004-08-28 15:24   ` Kenneth Lavrsen
2004-08-29 14:02   ` Alan Cox
2004-09-01 22:51     ` Rogier Wolff

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=200408291916.26053.Norbert@edusupport.nl \
    --to=norbert@edusupport.nl \
    --cc=linux-kernel@vger.kernel.org \
    --cc=nemosoft@smcc.demon.nl \
    /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