All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jonathan Cameron <jic23@cam.ac.uk>
To: Jonathan Corbet <corbet@lwn.net>
Cc: linux-media@vger.kernel.org, g.liakhovetski@gmx.de
Subject: Re: RFC: ov7670 soc-camera driver
Date: Mon, 16 Mar 2009 14:43:10 +0000	[thread overview]
Message-ID: <49BE657E.7040109@cam.ac.uk> (raw)
In-Reply-To: <20090315162338.3be11fec@bike.lwn.net>

Jonathan Corbet wrote:
> On Sun, 15 Mar 2009 17:10:01 +0000
> Jonathan Cameron <jic23@cam.ac.uk> wrote:
> 
>> The primary control on this chip related to shutter rate is actualy
>> the frame rate. There are rather complex (and largerly undocumented)
>> interactions between this setting and the auto brightness controls
>> etc. Anyone have any suggestions on a better way of specifying this?
> 
> Welcome to the world of the ov7670!  My conclusion, after working with
> this sensor, is that is consists of something like 150 analog tweakers
> disguised as digital registers.  Everything interacts with everything
> else, many of the settings are completely undocumented, and that's not
> to mention the weird multiplexor at 0x79.  It's hard to make this thing
> work if you don't have a blessed set of settings from OmniVision.
Hmm... And the grape vine / rumour says that they get most of their
'magic' values from customers who tweak the chips enough to get something
working.

Thanks for all the good work you put in.  Only other useful info was
the tinyos driver and that was a port of yours in the first place ;)

I'm particularly fond of the apparently obvious registers that won't
take a write unless something else is in a particular state.
> 
>> Clearly this driver shares considerable portions of code with
>> Jonathan Corbet's driver (in tree). It would be complex to combine
>> the two drivers, but perhaps people feel this would be worthwhile?
> 
> I think it's necessary, really.  Having two drivers for the same device
> seems like a bad idea.  As Hans noted, he's already put quite a bit of
> work into generalizing the ov7670 driver; I think it would be best to
> work with him to get a driver that works for everybody.
That sounds like a good plan.  Now all we need is some time ;)

Jonathan

      parent reply	other threads:[~2009-03-16 14:43 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-15 17:10 RFC: ov7670 soc-camera driver Jonathan Cameron
2009-03-15 18:50 ` Guennadi Liakhovetski
2009-03-15 19:05   ` Hans Verkuil
2009-03-15 20:21     ` Guennadi Liakhovetski
2009-03-16 14:39       ` Jonathan Cameron
2009-03-15 22:23 ` Jonathan Corbet
2009-03-16  7:46   ` Hans Verkuil
2009-03-20  6:32     ` Mauro Carvalho Chehab
2009-03-16 14:43   ` Jonathan Cameron [this message]

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=49BE657E.7040109@cam.ac.uk \
    --to=jic23@cam.ac.uk \
    --cc=corbet@lwn.net \
    --cc=g.liakhovetski@gmx.de \
    --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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.