public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Robert Jarzmik <robert.jarzmik@free.fr>
To: Guennadi Liakhovetski <g.liakhovetski@gmx.de>
Cc: Linux Media Mailing List <linux-media@vger.kernel.org>
Subject: Re: [PATCH 0/5] soc-camera: convert to platform device
Date: Thu, 16 Apr 2009 21:04:47 +0200	[thread overview]
Message-ID: <87d4bch12o.fsf@free.fr> (raw)
In-Reply-To: <Pine.LNX.4.64.0904161955140.4947@axis700.grange> (Guennadi Liakhovetski's message of "Thu\, 16 Apr 2009 20\:14\:38 +0200 \(CEST\)")

Guennadi Liakhovetski <g.liakhovetski@gmx.de> writes:

>>  - I unload and reload mt9m111 and pxa_camera
>>     => not any better
>
> Actually, I think, in this case it should be found again, as long as you 
> reload pxa-camera while i2c-pxa is already loaded.
Damn, you're right. I cross-checked, and reloading pxa_camera rescans the
sensor.

>> What I'm getting at is that if soc_camera is loaded before the i2c host driver,
>> no camera will get any chance to work. Is that normal considering the new driver
>> model ?
>> I was naively thinking that there would be a "rescan" when the "control" was
>> being available for a sensor.
>
> Yes, unfortunately, it is "normal":-( On the one hand, we shouldn't really 
> spend _too_ much time on this intermediate version, because, as I said, it 
> is just a preparatory step for v4l2-subdev. We just have to make sure it 
> doesn't introduce any significant regressions and doesn't crash too often. 
OK. So from my side everything is OK (let aside my nitpicking in mioa701.c and
mt9m111.c).

> OTOH, this is also how it is with v4l2-subdev. With it you first must have 
> the i2c-adapter driver loaded. Then, when a match between a camera host 
> and a camera client (sensor) platform device is detected, it is reported 
> to the v4l2-subdev core, which loads the respective camera i2c driver.
OK, why not.

> If you then unload the camera-host and i2c adapter drivers, and then you load
> the camera-host driver, it then fails to get the adapter, and if you then load
> it, nothing else happens. To reprobe you have to unload and reload the camera
> host driver.

So be it. I'm sure we'll be through it once more in the v4l2-subdev transition,
so I'll let aside any objection I could mutter :)

Cheers.

--
Robert

  reply	other threads:[~2009-04-16 19:04 UTC|newest]

Thread overview: 46+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-04-15 12:17 [PATCH 0/5] soc-camera: convert to platform device Guennadi Liakhovetski
2009-04-15 12:17 ` [PATCH 1/5] soc-camera: add a free_bus method to struct soc_camera_link Guennadi Liakhovetski
2009-04-16  3:39   ` Eric Miao
2009-04-15 12:17 ` [PATCH 2/5] soc-camera: host-driver cleanup Guennadi Liakhovetski
2009-04-15 20:36   ` Robert Jarzmik
2009-04-15 12:18 ` [PATCH 3/5] soc-camera: remove an extra device generation from struct soc_camera_host Guennadi Liakhovetski
2009-04-15 12:18 ` [PATCH 4/5] soc-camera: simplify register access routines in multiple sensor drivers Guennadi Liakhovetski
2009-04-15 20:36   ` Robert Jarzmik
2009-04-15 12:20 ` [PATCH 5/5] soc-camera: Convert to a platform driver Guennadi Liakhovetski
2009-04-15 20:36   ` Robert Jarzmik
2009-04-16  2:19   ` Dongsoo, Nathaniel Kim
2009-04-16  8:58     ` Guennadi Liakhovetski
2009-04-16 10:00       ` Dongsoo, Nathaniel Kim
2009-04-16 10:30         ` Guennadi Liakhovetski
2009-04-16 11:09           ` Dongsoo, Nathaniel Kim
2009-04-16 12:06             ` Guennadi Liakhovetski
2009-04-16 12:48               ` Dongsoo, Nathaniel Kim
2009-04-16 12:59                 ` Guennadi Liakhovetski
2009-04-16 14:44                   ` Dongsoo Kim
2009-04-16 14:56                     ` Guennadi Liakhovetski
2009-04-16 15:15                       ` Dongsoo Kim
2009-04-15 20:36 ` [PATCH 0/5] soc-camera: convert to platform device Robert Jarzmik
2009-04-16 17:49   ` Robert Jarzmik
2009-04-16 18:14     ` Guennadi Liakhovetski
2009-04-16 19:04       ` Robert Jarzmik [this message]
2009-04-17  7:29 ` Dongsoo, Nathaniel Kim
2009-04-17  7:48   ` Guennadi Liakhovetski
2009-04-17  7:50     ` Dongsoo, Nathaniel Kim
2009-04-17  7:40 ` Magnus Damm
2009-04-17  7:51   ` Guennadi Liakhovetski
2009-04-17 10:31     ` Magnus Damm
2009-04-17 10:43       ` Guennadi Liakhovetski
2009-04-20  7:14         ` Magnus Damm
2009-04-20  7:22           ` Guennadi Liakhovetski
2009-04-20  8:00             ` Magnus Damm
2009-04-20  8:14               ` Guennadi Liakhovetski
2009-04-20  8:54                 ` Magnus Damm
2009-04-20 13:50                   ` v4l2-subdev " Guennadi Liakhovetski
2009-04-20 14:04                     ` Mark Brown
2009-04-20 14:18                       ` Guennadi Liakhovetski
2009-04-20 14:31                         ` Mark Brown
2009-04-20  0:12       ` Kuninori Morimoto
2009-04-20  7:03         ` Magnus Damm
2009-04-17 18:38   ` [PATCH 5/5 v2] soc-camera: Convert to a platform driver Guennadi Liakhovetski
2009-04-19 20:01     ` Robert Jarzmik
2009-04-20  9:45     ` Darius Augulis

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=87d4bch12o.fsf@free.fr \
    --to=robert.jarzmik@free.fr \
    --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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox