public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Paulius Zaleckas <paulius.zaleckas@teltonika.lt>
To: video4linux-list@redhat.com
Subject: Re: [RFC] move sensor control out of kernel
Date: Fri, 29 Feb 2008 18:35:18 +0200	[thread overview]
Message-ID: <fq9c8n$hg$1@ger.gmane.org> (raw)
In-Reply-To: <f17812d70802290725o77db19daic50aee0380a1dc59@mail.gmail.com>


eric miao wrote:
>>  > The benefits I can think of now are:
>>  >
>>  > 1. simplified sensor driver design
>>
>>  It is pretty simple with soc-camera framework also. And the best thing
>>  is that it is unified. If you implement sensor driver in userspace then
>>  what kind of interface should V4L data bus driver expose.
>>
> 
> The soc-camera is pretty good. Yet it doesn't solve the issue of
> complicated sensor control. mt9m001.c and mt9v022.c are two
> good examples, but are all too simple. I have a sensor driver here
> with more than 4000 lines of code.

Yes I can imagine driver for analog devices ADV family video decoder :)
Some time ago I wrote driver for ADV...

But I don't imagine application or library with support for all 
devices... Kernel is easy to configure and to compile only drivers you 
need. Also kernel modules are good to load only the driver you need.

Another thing is that there will be different multiple implementations 
of this "4000 lines of code" driver. This means that each of them will 
be tested less than the single one in kernel.

>>  > Another biggest concern to the V4L2 API itself, sensor nowadays has
>>  > more control ability than it used to be, some smart sensor provides
>>  > even more like auto focus control, lens control, flash mode, and many
>>  > other features that current V4L2 API cannot cover.
>>
>>  Implement these controls. Make a patch and send it here. If everything
>>  is OK it should be merged and V4L will be capable to handle these controls.
>>
> 
> Mmm...then do you expect an API growing into a monster in the kernel space?

I guess this will happen anyway, because of tv tuners, dvb-t cards and 
etc. There is no way there to separate "sensor" from the data bus.

And you will have to export data bus interface to the userspace.

>>  My conclusion: your idea just moves problem from kernel to userspace,
>>  but doesn't solve it...
>>
> 
> I'm not trying to solve how to control the sensor but how to simplify
> the kernel effort and maybe with small reward of stability and ease
> of development.

I must agree about kernel stability here.

>>  --
>>  video4linux-list mailing list
>>  Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
>>  https://www.redhat.com/mailman/listinfo/video4linux-list
>>
> 

--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list

  reply	other threads:[~2008-02-29 16:36 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-02-29  4:18 [RFC] move sensor control out of kernel eric miao
2008-02-29  9:23 ` Markus Rechberger
2008-02-29 12:49 ` Paulius Zaleckas
2008-02-29 15:25   ` eric miao
2008-02-29 16:35     ` Paulius Zaleckas [this message]
2008-03-04 14:23       ` Mauro Carvalho Chehab
2008-02-29 23:09     ` Guennadi Liakhovetski

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='fq9c8n$hg$1@ger.gmane.org' \
    --to=paulius.zaleckas@teltonika.lt \
    --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