On Mon, 23 May 2011, KY Srinivasan wrote: > > > +module_init(init_hv_clocksource); > > > +MODULE_DESCRIPTION("HyperV based clocksource"); > > > +MODULE_AUTHOR("K. Y. Srinivasan <ksrinivasan@xxxxxxxxxx>"); > > > +MODULE_LICENSE("GPL"); > > > > Why do we need to build this as a module? > > No particular reason. This is the way, I had it in the staging directory. > Do you want me to build this as part of the kernel? I would then not have > to worry about auto-loading issues. If we get rid of the DMI/PCI stuff then the whole module code is larger than the real code. So no point in building it modular, just make it depend on CONFIG_WHATEVER_MEANS_HYPERV Thanks, tglx _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel