On 01/29/2014 01:29 PM, Robert Nelson wrote: > On Wed, Jan 29, 2014 at 12:19 PM, Nishanth Menon <nm@xxxxxx> wrote: >> OMAP34xx, AM3517 and OMAP36xx platforms use dpll1 clock. >> >> OMAP443x, OMAP446x, OMAP447x, OMAP5, DRA7, AM43xx platforms use >> dpll_mpu clock. >> >> Latency used is the generic latency defined in omap-cpufreq >> driver. >> >> Signed-off-by: Nishanth Menon <nm@xxxxxx> > > Hi Nishanth, > > After this patch, do you see any limitation to finally enabling 1Ghz > operation on the beagle-xm by default? Or are we still missing a > dependicy somewhere? yes, there is: a) ABB dt series - i will repost this in a few mins b) AVS conversion from non-dt mode to dt supported mode. (which by itself depends on VC/VP conversion). c) clk notifier based dvfs for cpufreq-cpu0 -> this allows us to introduce the necessary plumbing for mpu voltage domain such that the TWL4030 regulator, AVS and ABB are rightly sequenced. What you have done in the patch below is to introduce ABB regulator - but no one is actually using it -> this might actually work on certain samples at 1GHz, but prolonged operation will either damage the device or fail on other samples - I have tried numerous times Internally to get approval for non ABB/AVS configuration for 1GHz - but I have a clear feedback that it cannot be done with the constraints of DM3730/OMAP3630. Lets do this a series at a time and build up the necessary support - we get clock nodes for dvfs (using i2c1) here with cpufreq-cpu0 with this series. If folks can ack and queue this up, we can get in ABB dts nodes in place - allowing us to work on the next set -> sequencing using clock notifier. in parallel we could work on converting AVS back to dt based solution. yes, the road is long. -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html