* Felipe Balbi <balbi@xxxxxx> [110810 05:31]: > > On Wed, Aug 10, 2011 at 05:55:20PM +0530, Keerthy wrote: > > + > > +int __init omap_devinit_temp_sensor(void) > > +{ > > + if (!cpu_is_omap446x()) > > + return 0; > > + > > + return omap_hwmod_for_each_by_class("temperature_sensor", > > + temp_sensor_dev_init, NULL); > > +} > > + > > +arch_initcall(omap_devinit_temp_sensor); > > I really dislike people adding more and more *initcall() to their pieces > of code. But Tony is the final Judge. Yes how about making this just a regular device driver and have it live under drivers/ somewhere? Or is there some reason why this could not be a loadable module? Regards, Tony -- 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