Linux Media Controller development
 help / color / mirror / Atom feed
From: Andy Walls <awalls@md.metrocast.net>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Jean-Francois Moine <moinejf@free.fr>,
	linux-media@vger.kernel.org
Subject: Re: [PATCH] Illuminators control
Date: Mon, 13 Sep 2010 08:49:32 -0400	[thread overview]
Message-ID: <1284382172.2031.45.camel@morgan.silverblock.net> (raw)
In-Reply-To: <201009131007.58665.hverkuil@xs4all.nl>

On Mon, 2010-09-13 at 10:07 +0200, Hans Verkuil wrote:
> On Monday, September 13, 2010 09:08:36 Laurent Pinchart wrote:
> > Hi,
> > 
> > On Saturday 11 September 2010 11:03:50 Jean-Francois Moine wrote:
> > 
> > > @@ -419,6 +421,8 @@ void v4l2_ctrl_fill(u32 id, const char **name, enum 
> > v4l2_ctrl_type *type,
> > >  	case V4L2_CID_AUDIO_LIMITER_ENABLED:
> > >  	case V4L2_CID_AUDIO_COMPRESSION_ENABLED:
> > >  	case V4L2_CID_PILOT_TONE_ENABLED:
> > > +	case V4L2_CID_ILLUMINATORS_1:
> > > +	case V4L2_CID_ILLUMINATORS_2:
> > >  		*type = V4L2_CTRL_TYPE_BOOLEAN;
> > >  		*min = 0;
> > >  		*max = *step = 1;
> > 
> > I would prefer integer controls for this, as we will need to support dimmable 
> > illuminators.
> > 
> > 
> 
> Don't. That should be a separate control. I expect that the brightness is
> something you touch a lot less than the on/of controls. Anyway, I think it
> makes a lot more sense to separate these two functions.



I would need to research what computerized microscopes today actually
have.

A quick look reveals: 
	http://www.bodelin.com/proscopehr/how_it_works/
	http://www.ecoscopestore.com/main.sc
	http://www.microscopeworld.com/MSWorld/Microscope-World/Digital-Microscopes/

handheld units likely have only on/off control for illumination.
Eyepiece cameras rely on what the existing laboratory microscope already
has for illumination.


In my limited experience, there are some microscope controls that a
human is going to have his hands on to get the desired image:
magnification, focus, and specimen position.  Illumination intensity
seems like it would fit in that group.

When I had a job in college fixing medical equipment, most microscopes
limited light using a mechanical iris controlled by hand.  The
illuminator was simply an on/off switch.  (BTW a mechanical iris is a
pain to take apart and rebuild.)  Light filters could be installed
between the illuminator and the iris. 

On some of the "digital microscopes" in the links above, you can see the
iris assembly underneath the stage.  The DMBA210 microscope has both a
manual iris and an illumination intensity knob.


So for digital microscopes we may not see illumination intensity
controllable by software.  Maybe for other types of cameras we will.

Regards,
Andy




  reply	other threads:[~2010-09-13 12:49 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-11  9:03 [PATCH] Illuminators control Jean-Francois Moine
2010-09-11  9:31 ` Hans Verkuil
2010-09-11 14:20 ` Andy Walls
2010-09-13  7:08 ` Laurent Pinchart
2010-09-13  8:07   ` Hans Verkuil
2010-09-13 12:49     ` Andy Walls [this message]
  -- strict thread matches above, loose matches on Subject: below --
2010-09-11  8:24 Jean-Francois Moine
2010-09-11  8:43 ` Hans Verkuil

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=1284382172.2031.45.camel@morgan.silverblock.net \
    --to=awalls@md.metrocast.net \
    --cc=hverkuil@xs4all.nl \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-media@vger.kernel.org \
    --cc=moinejf@free.fr \
    /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