On Tue, Jun 07, 2011 at 05:25:32PM +0200, Laurent Pinchart wrote: > Hi Guennadi, > > Thanks for the patch. > > On Monday 06 June 2011 14:31:57 Guennadi Liakhovetski wrote: > > Add media bus configuration types and two subdev operations to get > > supported mediabus configurations and to set a specific configuration. > > Subdevs can support several configurations, e.g., they can send video data > > on 1 or several lanes, can be configured to use a specific CSI-2 channel, > > in such cases subdevice drivers return bitmasks with all respective bits > > set. When a set-configuration operation is called, it has to specify a > > non-ambiguous configuration. > > > > Signed-off-by: Stanimir Varbanov <svarbanov@xxxxxxxxxx> > > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski@xxxxxx> > > --- > > > > This change would allow a re-use of soc-camera and "standard" subdev > > drivers. It is a modified and extended version of > > > > http://article.gmane.org/gmane.linux.drivers.video-input-infrastructure/294 > > 08 > > > > therefore the original Sob. After this we only would have to switch to the > > control framework:) Please, comment. > > I still believe we shouldn't use any set operation :-) The host/bridge driver > should just use the get operation before starting the stream to configure it's > sensor interface. I agree with Laurent. What about implementing just get first? Regards, -- Sakari Ailus sakari.ailus@xxxxxx -- 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
