public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Martin Strubel <hackfin@section5.ch>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-media@vger.kernel.org
Subject: Re: v4l2 device property framework in userspace
Date: Tue, 31 May 2011 13:33:32 +0200	[thread overview]
Message-ID: <4DE4D20C.1090206@section5.ch> (raw)
In-Reply-To: <201105311255.02481.hverkuil@xs4all.nl>

Hi,

> 
> Not religion, it's experience. I understand what you want to do and it is
> just a bad idea in the long term. Mind you, it's great for prototyping and
> experimentation. But if you want to get stable sensor support in the kernel,
> then it has to conform to the rules. Having some sensor drivers in the kernel
> and some in userspace will be a maintenance disaster.

Sorry, from our perspective the current v4l2 system *has* already been a
maintenance disaster. No offense, but that is exactly the reason why we
had to internally circumvent it.
You're free to use the system for early prototyping stage as well as for
a stable release (the framework is in fact running since 2006 in medical
imaging devices). It certainly cost us less maintenance so far than
syncing up to the changing v4l2 APIs.

> 
> Besides, how is your sensor driver supposed to work when used in a USB webcam?
> Such a USB bridge driver expects a subdevice sensor driver. Since you use a
> different API the two can't communicate. Hence no code reuse.
> 

I don't see a problem there either. Because you just put your register
access code into the kernel. That's merely a matter of two functions.
The sensor daemon doesn't really care *how* you access registers.


>>
>> Not sure if you understand: I do not have to implement or generate ioctl
>> handlers and call them. This is definitely less expensive in terms of
>> coding. All the register access is handled *automatically* using the HW
>> description layer.
> 
> Using what? /dev/i2c-X? That's using ioctls (I2C_RDWR).
> 

Yes. But I have to write exactly two wrappers for access. Not create
tables with ioctl reqcodes.

> 
> Well, you clearly want *your* solution. I've been working in the v4l subsystem
> for many years now ensuring that we can support the widest range of very
> practical and non-academic hardware, both consumer hardware and embedded system
> hardware, and working together with companies like TI, Samsung, Nokia, etc.

Nope. I want any solution that does the job for our requirements. So far
it hasn't been doing it. It's not just getting an image from a sensor
and supporting v4l2 modes, but I think I've been mentioning the dirty
stuff already.

> 
> You (or your company/organization) designed a system without as far as I am aware
> consulating the people responsible for the relevant kernel subsystem (V4L in this
> case). And now you want to get your code in with a minimum of change. Sorry, that's
> not the way it works.
> 

Just that you understand: I'm not wanting to get our code into
somewhere. I'd rather avoid it, one reason being lengthy discussions :-)
Bottomline again: I'm trying to find a solution to avoid bloated and
potentially unstable kernel drivers. Why do you think we (and our
customers) spent the money to develop alternative solutions?


Cheers,

- Martin

  reply	other threads:[~2011-05-31 11:33 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-05-29 13:07 v4l2 device property framework in userspace Martin Strubel
2011-05-30  7:32 ` Hans Verkuil
2011-05-30  9:38   ` Martin Strubel
2011-05-30 10:52     ` Hans Verkuil
2011-05-30 12:07       ` Martin Strubel
2011-05-30 12:53         ` Hans Verkuil
2011-05-30 13:30           ` Martin Strubel
2011-05-31  8:01             ` Hans Verkuil
2011-05-31  8:27               ` Martin Strubel
2011-05-31 10:55                 ` Hans Verkuil
2011-05-31 11:33                   ` Martin Strubel [this message]
2011-05-31 12:58                     ` Mauro Carvalho Chehab
2011-06-06 11:00                     ` Sakari Ailus
2011-05-31  6:38         ` Kim, HeungJun

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=4DE4D20C.1090206@section5.ch \
    --to=hackfin@section5.ch \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    /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