Hi Guennadi, On Wednesday 21 March 2012 11:59:59 Guennadi Liakhovetski wrote: > On Wed, 21 Mar 2012, Laurent Pinchart wrote: > > On Wednesday 14 March 2012 16:02:20 Guennadi Liakhovetski wrote: > > > Newer CEU versions, e.g., the one, used on sh7372, support image sizes > > > larger than 2560x1920. Retrieve maximum sizes from platform properties. > > > > Isn't there a way you could query the CEU version at runtime instead ? > > I'm not aware of any. And even if it were possible, I'm not sure putting > tables with "version - feature-set" tables into the driver proper would be > a very good idea. It used to be like that (or almost like that with > dependencies on the chip-type) in other drivers (e.g., shdma) and we > dropped it in favour of platform data.
I would have voted for doing it the other way around. The driver should know about the different hardware versions it supports. Putting that in platform data just moves the burden to board code developers and duplicates the information in lots of places. At worse, if you can't detect the version at runtime, I would put a version field in the platform data and map that to hardware features in the driver. -- Regards, Laurent Pinchart -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
