public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Bill Davidsen <davidsen@tmr.com>
To: Thomas Koeller <thomas.koeller@baslerweb.com>
Cc: linux-kernel@vger.kernel.org, alan@lxorguk.ukuu.org.uk,
	linux-mips@linux-mips.org
Subject: Re: [PATCH] Image capturing driver for Basler eXcite smart camera
Date: Fri, 18 Aug 2006 09:19:51 -0400	[thread overview]
Message-ID: <44E5BE77.9040200@tmr.com> (raw)
In-Reply-To: <200608172230.30682.thomas.koeller@baslerweb.com>

Thomas Koeller wrote:
> On Thursday 17 August 2006 17:31, Pavel Machek wrote:
>> Well, I guess v4l api will need to be improved, then. That is still
>> not a reason to introduce completely new api...
> 
> The API as implemented by the driver I submitted is very minimalistic,
> because it is just a starting point. There's more to be added in future,
> like controlling flashes, interfacing to line-scan cameras clocked by
> incremental encodes attached to some conveyor, and other stuff which
> is common in industrial image processing applications. You really do
> not want to clutter the v4l2 API with these things; that would hardly
> be an 'improvement'.
> 
> Different interfaces, designed to serve different purposes...
> 
If you look at Pavel's posts WRT swsusp2, he has taken this position 
before, that lack of functionality in {something} is no justification to 
introduce a new solution, and that the limitations of {something} can be 
addressed by incremental improvement. Like any good idea, this can be 
carried to extremes.

Don't take it personally, just write working code people can patch in. 
When your code has the features you mentioned it will be highly useful 
and hopefully ported to many devices. I guess security monitoring is an 
"industrial image processing application," which interests me. At the 
moment I would call it an impressive proof of concept, but you have many 
useful ideas for its future.

-- 
Bill Davidsen <davidsen@tmr.com>
   Obscure bug of 2004: BASH BUFFER OVERFLOW - if bash is being run by a
normal user and is setuid root, with the "vi" line edit mode selected,
and the character set is "big5," an off-by-one errors occurs during
wildcard (glob) expansion.

  parent reply	other threads:[~2006-08-18 13:16 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-08-10 21:18 [PATCH] Image capturing driver for Basler eXcite smart camera Thomas Koeller
2006-08-11 19:35 ` Randy.Dunlap
2006-08-11 20:09 ` Alan Cox
2006-08-12 17:27   ` Thomas Koeller
2006-08-14 15:53     ` Éric Piel
2006-08-14 19:26       ` Thomas Koeller
2006-08-17 15:31         ` Pavel Machek
2006-08-17 20:30           ` Thomas Koeller
2006-08-18  8:27             ` Pavel Machek
2006-08-18 13:19             ` Bill Davidsen [this message]
2006-08-22 21:36               ` Thomas Koeller
2006-08-11 20:48 ` Dave Jones

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=44E5BE77.9040200@tmr.com \
    --to=davidsen@tmr.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mips@linux-mips.org \
    --cc=thomas.koeller@baslerweb.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