From: Mauro Carvalho Chehab <mchehab@infradead.org>
To: Paulius Zaleckas <paulius.zaleckas@teltonika.lt>
Cc: video4linux-list@redhat.com
Subject: Re: [RFC] move sensor control out of kernel
Date: Tue, 4 Mar 2008 11:23:20 -0300 [thread overview]
Message-ID: <20080304112320.6dadf6df@gaivota> (raw)
In-Reply-To: <fq9c8n$hg$1@ger.gmane.org>
On Fri, 29 Feb 2008 18:35:18 +0200
Paulius Zaleckas <paulius.zaleckas@teltonika.lt> wrote:
>
> 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.
Having this at kernel avoids duplicating the code. Also, a kernel driver without
the sensor means a broken driver, since it won't work without the userspace
counterpart.
> >> > 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.
This is just a matter of adding newer controls to VIDIOC_[G/S/ENUM]_CTRL.
This way, generic apps like qv4l2 can properly configure any device. This also
avoids the need of having special userspace apps for certain devices.
If you don't standardize the controls at the API, then you'll have a real mess
at the userspace.
Cheers,
Mauro
--
video4linux-list mailing list
Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
https://www.redhat.com/mailman/listinfo/video4linux-list
next prev parent reply other threads:[~2008-03-04 14:24 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
2008-03-04 14:23 ` Mauro Carvalho Chehab [this message]
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=20080304112320.6dadf6df@gaivota \
--to=mchehab@infradead.org \
--cc=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