On Fri, 15 Jul 2011, Sakari Ailus wrote: > On Fri, Jul 15, 2011 at 12:02:06AM +0200, Guennadi Liakhovetski wrote: > > On Fri, 15 Jul 2011, Sakari Ailus wrote: > > > > > Hi Guennadi, > > > > > > Thanks for the comments. > > > > > > On Thu, Jul 14, 2011 at 07:56:10PM +0200, Guennadi Liakhovetski wrote: > > > > On Thu, 14 Jul 2011, Sakari Ailus wrote: > > > > > > > > > Hi all, > > > > > > > > > > I was thinking about the sensor binning controls. > > > > > > > > What wrong with just doing S_FMT on the subdev pad? Binning does in fact > > > > implement scaling. > > > > > > Nothing really. Supporting setting binning using S_FMT is fine. > > > > > > However, the interface does not express binning capabilities in any way. To > > > effectively use binning settings one must know the capabilities. Binning is > > > scaling but the choices are so coarse that the capabilities are a must. > > > > > > The capabilities could be found implicitly by trying out different formats > > > and looking back at the result. That's still not quite trivial. > > > > > > If there would be a good way to enumerate the binning capabilities, combined > > > with S_FMT it'd be close to perfect. > > > > Then how about something like ENUM_SCALE(S)? > > For enumerating scaler capabilities? > > The capabilities and features vary wildly --- actually binning (and > skipping) are probably the only types of scaling the properties of which can > be fully expressed so tersely. What would you think about enumerating only > binning and skipping? Those are the scalers, that can be easily enumerated, yes. Others simply wouldn't implement this method. Similar to ENUM_FRAMESIZES etc. 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