All of lore.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: 47+ 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
     [not found]                       ` <20090420140411.GE8776-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2009-04-20 14:18                         ` Guennadi Liakhovetski
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 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.