On Tue, Nov 9, 2010 at 5:48 PM, Kevin Hilman <khilman@xxxxxxxxxxxxxxxxxxx> wrote: >> #if defined(CONFIG_OMAP_MBOX_FWK) || defined(CONFIG_OMAP_MBOX_FWK_MODULE) >> +static struct omap_device_pm_latency mbox_latencies[] = { >> + [0] = { >> + .activate_func = omap_device_enable_clocks, >> + .activate_lat = 50000, /* FIXME random value */ >> + .deactivate_func = omap_device_enable_clocks, >> + .deactivate_lat = 50000, /* FIXME random value */ >> + }, >> +}; > > hmm, I'm hoping you're not expecting this to merge when using random values. This was changed on the second iteration of the patch [1], and is still a WIP as some more comments have been made. I'll keep your current comments though. Regards, Omar --- [1] http://www.mail-archive.com/linux-omap@xxxxxxxxxxxxxxx/msg38169.html -- 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