Re: [PATCH] ARM: EXYNOS5: Support Exynos5-bus devfreq driver

2012-12-30 Thread Abhilash Kesavan
On Sun, Dec 30, 2012 at 11:45 AM, Olof Johansson wrote: > Hi, Hi Olof, >> +# ppmu support >> + >> +obj-$(CONFIG_EXYNOS5250_PPMU)+= exynos_ppmu.o exynos5_ppmu.o > > Quite obvious that it's ppmu support from the file names. No need for > a comment. Will improve the commenting in the

Re: [PATCH] ARM: EXYNOS5: Support Exynos5-bus devfreq driver

2012-12-29 Thread Olof Johansson
Hi, Just a quick hit of review, I haven't done anything more in-depth yet. On Fri, Dec 28, 2012 at 04:24:09AM -0500, Abhilash Kesavan wrote: > - Setup the INT clock ops to control/vary INT frequency > - Add PPMU support for Exynos5250 > - Add mappings initially for the PPMU device > > Signed-of