public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Darius <augulis.darius@gmail.com>
To: video4linux-list@redhat.com
Subject: Re: question about SoC Camera driver (Micron)
Date: Thu, 15 May 2008 17:31:45 +0300	[thread overview]
Message-ID: <g0hhpt$jfp$1@ger.gmane.org> (raw)
In-Reply-To: <Pine.LNX.4.64.0805132212530.4988@axis700.grange>

Guennadi Liakhovetski wrote:
> On Tue, 13 May 2008, Darius wrote:
> 
>> Now I see how it works. I2C devices should be created before driver loading.
>> There was my mistake, and driver does not call probe() function. Maybe would
>> be better to create I2C devices by driver itself, not by the board specific
>> config code? Now sensor driver is useless itself, without board specific
>> configuration... Would be correct to do so?
> 
> No. This is not how the driver model works. PCI drivers do not register 
> PCI devices. The PCI host controller scans the PCI bus and adds devices 
> into the system, to be later matched against PCI drivers. Similar for USB 
> devices, etc. The problem with i2c you cannot reliably scan the bus. 
> Therefore the information about devices present on the system has to come 
> from elsewhere: when it is an i2c device embedded into a USB web-camera, 
> its driver "knows", there's an i2c device and registers it. On embedded 
> systems the platform knows what i2c devices are onboard, and registers 
> them using i2c_register_board_info(), on powerpc (and sparc?) you can 
> register i2c devices in your device tree, etc.
> 
> Thanks
> Guennadi
> ---
> Guennadi Liakhovetski, Ph.D.
> Freelance Open-Source Software Developer
> 
> --
> video4linux-list mailing list
> Unsubscribe mailto:video4linux-list-request@redhat.com?subject=unsubscribe
> https://www.redhat.com/mailman/listinfo/video4linux-list
> 

Guennadi, can you please describe more detailed struct soc_camera_device structure? All these members xmin, ymin, etc...
Also soc_camera_data_format structure has member depht. Should this member fit sensor bus bit count or pixel format depht in videodev2.h?
Because most pixel formats are 16 bit and camera sensor interface in most cases is 8 bit.

--
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-05-15 14:37 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-05-12 14:03 question about SoC Camera driver (Micron) Darius
2008-05-12 18:32 ` Guennadi Liakhovetski
2008-05-13  8:31   ` Darius
2008-05-13 20:19     ` Guennadi Liakhovetski
2008-05-15 14:31       ` Darius [this message]
2008-05-15 19:39         ` Guennadi Liakhovetski
2008-05-22 13:19           ` Darius
2008-05-22 19:19             ` Guennadi Liakhovetski
2008-05-23  9:05               ` Darius

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='g0hhpt$jfp$1@ger.gmane.org' \
    --to=augulis.darius@gmail.com \
    --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