Hi Jon, On Mon, May 07, 2012 at 21:42:35, Hunter, Jon wrote: > > Clk_prd is a platform data passed to the driver, so platform code > > updates it, where else can it be done ? > > The point is that you can pass what ever you like. You do not need to > pass the frequency you can pass the clock handle instead. As clk rate is required in platform code for timing calculation, and already available, period was passed > > What happens it the clk_get() of the gpmc_l3_clk fails during the init? Thanks for bringing this point, invalid clk_prd has to be handled by driver. > > In fact if you migrate to runtime pm then we should not have the clk_get > in the gpmc_init any more. Even if converted to RPM, to get clk rate, clk_get has to be done somewhere, right ? Regards Afzal -- 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