On Fri, Sep 09, 2011 at 02:30:24AM +0200, Cousson, Benoit wrote: > On 9/8/2011 8:09 PM, Grant Likely wrote: > >>+- compatible : Should be: > >>+ - "ti,ivahd", "ti,iva" for OMAP4 > >>+ - "ti,iva2", "ti,iva" for OMAP3 > >>+ - "ti,iva1", "ti,iva" for OMAP2 > > > >Again, be specific to the instantiation and encode the omap version > >into the compatible value. Use the form "ti,omap4-*", etc. > > In that case, I do not necessarily agree. The IVA is a generic TI IP > used in various chips. The IP version is well documented in the TRM. > > OK, in fact I didn't pick the proper version:-( > It should be: > > + - "ti,ivahd", for OMAP4 > + - "ti,iva2.2", for OMAP3 > + - "ti,iva2.1", for OMAP2430 > + - "ti,iva1", for OMAP2420 > > But still, no need to rely on OMAP version to identify the IVA version. Fair enough, providing that the "ti,iva" value is dropped. g. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
