Re: finalising soc-camera conversion to v4l2-subdev

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Laurent

On Thu, 29 Oct 2009, Laurent Pinchart wrote:

> Hi Guennadi,
> 
> On Wednesday 28 October 2009 17:37:09 Guennadi Liakhovetski wrote:
> > Hi all
> > 
> > As some of you will know, soc-camera framework is undergoing a conversion to
> > the v4l2-subdev API. Most of the legacy soc-camera client API has been
> > ported over to v4l2-subdev. Final conversion is blocked by missing
> > functionality in the current v4l2 subsystem. Namely video bus configuration
> > and data format negotiation. And from the progress of respective RFCs it
> > looks like this could take a while to get them into the mainline, which is
> > also understandable, given the amount of work. So, the question is - can we
> > work out a way to finalise the porting yet before the final versions of
> > those RFCs make it upstream? OTOH, we certainly do not want to have to
> > create a solution, which will have to be thrown away completely later.
> 
> Right, but we could design a temporary solution that goes in the right 
> direction and "fix" the code later. In that case the temporary solution must 
> be clearly marked as such, as we don't want to keep it around for API and ABI 
> compatibility reasons.

Agree.

> > We could decide to
> > 
> > 1. make bus configuration optional. If no data provided - use defaults.
> 
> Would that really work ?

Well, we should be able to make at least one (USB or whatever) camera work 
per soc-camera sensor driver:-) Which means, soc-camera native 
configurations set bus configuration explicitly anyway, and we make 
default match that one non-soc-camera card. It is relatively improbable, 
that some driver will get used by more than one card and that they will 
have incompatible configurations;) Then we'll have to think how to solve 
that.

> > 2. use something like the proposed imagebus API for data format negotiation.
> > Even if it will be eventually strongly modified for new "Media Controller &
> > Co." APIs, it already exists, so, the time has already been spent on it, and
> > mainlining it will not require much more time. But I'm open to other ideas
> > too.
> > 
> > OR
> > 
> > 3. use some intermediate solution - something, that we think will later
> > allow an easy enough extension to the new APIs when they appear.
> 
> 2 and 3 are similar in my opinion.

The numbering is not very logical, it should have been 2.1 instead of 2, 
and 2.2 instead of 3.

That's good that that's also your opinion:-) That means my imagebus is not 
too far off the track.

> The current imagebus API proposal controls 
> whole subdevices while it should act at the pad level. Pads will be introduced 
> with the media controller, so we could
> 
> - use a subdev-level imagebus API, allowing the soc-camera conversion to 
> subdev, and port the code to pad level latter, or
> 
> - introduce subdev pads operations now and use them for the imagebus API
> 
> The second solution would take more time as we need to agree on the subdev 
> pads operations. I'm ok with the first solution, as long as you agree to port 
> the code to the new subdev pads operations later :-)

You know, noone can see the future:-) But so far I don't see anything that 
would hinder me from doing that.

Thanks
Guennadi
---
Guennadi Liakhovetski, Ph.D.
Freelance Open-Source Software Developer
http://www.open-technology.de/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Input]     [Video for Linux]     [Gstreamer Embedded]     [Mplayer Users]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]
  Powered by Linux