On Wed, Apr 27, 2011 at 12:07 AM, Mark Brown <broonie@xxxxxxxxxxxxx> wrote: > On Tue, Apr 26, 2011 at 05:43:21PM +0900, MyungJoo Ham wrote: > >> +static int __init dvfs_init(void) >> +{ >> + Â Â platform_driver_register(&dvfs_pm_drv); >> + Â Â platform_device_register(&dvfs_pm_dev); >> + >> + Â Â INIT_DELAYED_WORK_DEFERRABLE(&dvfs_work, dvfs_monitor); >> + Â Â schedule_delayed_work(&dvfs_work, msecs_to_jiffies(DVFS_INTERVAL)); > > This will unconditionally start the timer which polls the devices at > whatever rate. ÂI'd expect to start and stop the poll depending on if we > have devices and/or governor type code (which I don't see anything for > here, though it's hard to know what criteria a generic governor would > use) which require it. > _______________________________________________ > linux-pm mailing list > linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx > https://lists.linux-foundation.org/mailman/listinfo/linux-pm > Thanks. I will let it poll only when there are dvfs-devices with polling interval specified. Cheers! - MyungJoo -- MyungJoo Ham (íëì), Ph.D. Mobile Software Platform Lab, Digital Media and Communications (DMC) Business Samsung Electronics cell: 82-10-6714-2858 _______________________________________________ linux-pm mailing list linux-pm@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/linux-pm