| Message ID | 4180fd5b43906421fc155126e8b7bea5fb3ff375.1364229828.git.viresh.kumar@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
On 25 March 2013 22:24, Viresh Kumar <viresh.kumar@linaro.org> wrote: > This patch moves cpufreq driver of powerpc platform to drivers/cpufreq. > > Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> > Cc: Paul Mackerras <paulus@samba.org> > Cc: Olof Johansson <olof@lixom.net> > Cc: linuxppc-dev@lists.ozlabs.org > Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > --- > arch/powerpc/platforms/Kconfig | 31 ---------------------- > arch/powerpc/platforms/pasemi/Makefile | 1 - > arch/powerpc/platforms/powermac/Makefile | 2 -- > drivers/cpufreq/Kconfig.powerpc | 26 ++++++++++++++++++ > drivers/cpufreq/Makefile | 3 +++ > .../cpufreq.c => drivers/cpufreq/pasemi-cpufreq.c | 0 > .../cpufreq/pmac32-cpufreq.c | 0 > .../cpufreq/pmac64-cpufreq.c | 0 > 8 files changed, 29 insertions(+), 34 deletions(-) > rename arch/powerpc/platforms/pasemi/cpufreq.c => drivers/cpufreq/pasemi-cpufreq.c (100%) > rename arch/powerpc/platforms/powermac/cpufreq_32.c => drivers/cpufreq/pmac32-cpufreq.c (100%) > rename arch/powerpc/platforms/powermac/cpufreq_64.c => drivers/cpufreq/pmac64-cpufreq.c (100%) Benjamin/Paul/Olof, Any comments on this?
On 31 March 2013 09:33, Viresh Kumar <viresh.kumar@linaro.org> wrote: > On 25 March 2013 22:24, Viresh Kumar <viresh.kumar@linaro.org> wrote: >> This patch moves cpufreq driver of powerpc platform to drivers/cpufreq. >> >> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> >> Cc: Paul Mackerras <paulus@samba.org> >> Cc: Olof Johansson <olof@lixom.net> >> Cc: linuxppc-dev@lists.ozlabs.org >> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> >> --- >> arch/powerpc/platforms/Kconfig | 31 ---------------------- >> arch/powerpc/platforms/pasemi/Makefile | 1 - >> arch/powerpc/platforms/powermac/Makefile | 2 -- >> drivers/cpufreq/Kconfig.powerpc | 26 ++++++++++++++++++ >> drivers/cpufreq/Makefile | 3 +++ >> .../cpufreq.c => drivers/cpufreq/pasemi-cpufreq.c | 0 >> .../cpufreq/pmac32-cpufreq.c | 0 >> .../cpufreq/pmac64-cpufreq.c | 0 >> 8 files changed, 29 insertions(+), 34 deletions(-) >> rename arch/powerpc/platforms/pasemi/cpufreq.c => drivers/cpufreq/pasemi-cpufreq.c (100%) >> rename arch/powerpc/platforms/powermac/cpufreq_32.c => drivers/cpufreq/pmac32-cpufreq.c (100%) >> rename arch/powerpc/platforms/powermac/cpufreq_64.c => drivers/cpufreq/pmac64-cpufreq.c (100%) > > Benjamin/Paul/Olof, > > Any comments on this? Ping!!
On Wed, 2013-04-03 at 15:00 +0530, Viresh Kumar wrote: > On 31 March 2013 09:33, Viresh Kumar <viresh.kumar@linaro.org> wrote: > > On 25 March 2013 22:24, Viresh Kumar <viresh.kumar@linaro.org> wrote: > >> This patch moves cpufreq driver of powerpc platform to drivers/cpufreq. > >> > >> Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> > >> Cc: Paul Mackerras <paulus@samba.org> > >> Cc: Olof Johansson <olof@lixom.net> > >> Cc: linuxppc-dev@lists.ozlabs.org > >> Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> > >> --- > >> arch/powerpc/platforms/Kconfig | 31 ---------------------- > >> arch/powerpc/platforms/pasemi/Makefile | 1 - > >> arch/powerpc/platforms/powermac/Makefile | 2 -- > >> drivers/cpufreq/Kconfig.powerpc | 26 ++++++++++++++++++ > >> drivers/cpufreq/Makefile | 3 +++ > >> .../cpufreq.c => drivers/cpufreq/pasemi-cpufreq.c | 0 > >> .../cpufreq/pmac32-cpufreq.c | 0 > >> .../cpufreq/pmac64-cpufreq.c | 0 > >> 8 files changed, 29 insertions(+), 34 deletions(-) > >> rename arch/powerpc/platforms/pasemi/cpufreq.c => drivers/cpufreq/pasemi-cpufreq.c (100%) > >> rename arch/powerpc/platforms/powermac/cpufreq_32.c => drivers/cpufreq/pmac32-cpufreq.c (100%) > >> rename arch/powerpc/platforms/powermac/cpufreq_64.c => drivers/cpufreq/pmac64-cpufreq.c (100%) > > > > Benjamin/Paul/Olof, > > > > Any comments on this? > > Ping!! I'm on vacation until end of April. No objection to the patch but somebody needs to test it. Cheers, Ben.
On 3 April 2013 16:00, Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote: > I'm on vacation until end of April. No objection to the patch but > somebody needs to test it. Any input on whom i can ask for that?
On 3 April 2013 16:00, Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote: > On Wed, 2013-04-03 at 15:00 +0530, Viresh Kumar wrote: >> On 31 March 2013 09:33, Viresh Kumar <viresh.kumar@linaro.org> wrote: >> > Benjamin/Paul/Olof, >> > >> > Any comments on this? >> >> Ping!! > > I'm on vacation until end of April. No objection to the patch but > somebody needs to test it. Hi, Can somebody else from powerpc world give it a try? OR @Rafael: Can we get this pushed in linux-next as is and then people would be forced to test it and in case there are any complains, i will fix them or you can revert it?
On Thu, Apr 4, 2013 at 11:55 AM, Viresh Kumar <viresh.kumar@linaro.org> wrote: > On 3 April 2013 16:00, Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote: >> On Wed, 2013-04-03 at 15:00 +0530, Viresh Kumar wrote: >>> On 31 March 2013 09:33, Viresh Kumar <viresh.kumar@linaro.org> wrote: > >>> > Benjamin/Paul/Olof, >>> > >>> > Any comments on this? >>> >>> Ping!! >> >> I'm on vacation until end of April. No objection to the patch but >> somebody needs to test it. > > Hi, > > Can somebody else from powerpc world give it a try? Deepthi, can you help? > OR > > @Rafael: Can we get this pushed in linux-next as is and then people would > be forced to test it and in case there are any complains, i will fix them or > you can revert it?
On Thu, Apr 04, 2013 at 11:55:56AM +0530, Viresh Kumar wrote: > On 3 April 2013 16:00, Benjamin Herrenschmidt <benh@kernel.crashing.org> wrote: > > On Wed, 2013-04-03 at 15:00 +0530, Viresh Kumar wrote: > >> On 31 March 2013 09:33, Viresh Kumar <viresh.kumar@linaro.org> wrote: > > >> > Benjamin/Paul/Olof, > >> > > >> > Any comments on this? > >> > >> Ping!! > > > > I'm on vacation until end of April. No objection to the patch but > > somebody needs to test it. > > Hi, > > Can somebody else from powerpc world give it a try? > > OR > > @Rafael: Can we get this pushed in linux-next as is and then people would > be forced to test it and in case there are any complains, i will fix them or > you can revert it? That's not very nice this late in the staging cycle. Give the powerpc guys a little more time than a single day to come back with test results, please. -Olof
On 11 April 2013 12:29, Olof Johansson <olof@lixom.net> wrote: > That's not very nice this late in the staging cycle. Give the powerpc > guys a little more time than a single day to come back with test results, > please. Yes we are waiting indefinitely for an Ack from powerpc guys.. If we can get a Ack soon, we will get it in 3.10 otherwise next cycle.
On Thursday, April 11, 2013 12:32:50 PM Viresh Kumar wrote: > On 11 April 2013 12:29, Olof Johansson <olof@lixom.net> wrote: > > That's not very nice this late in the staging cycle. Give the powerpc > > guys a little more time than a single day to come back with test results, > > please. > > Yes we are waiting indefinitely for an Ack from powerpc guys.. If we can > get a Ack soon, we will get it in 3.10 otherwise next cycle. Well, I'll be traveling from Saturday morning onwards until the end of the next week and my window for taking stuff into v3.10 closes in 16 hours. Whatever is not ready (i.e. ACKed in this particular case) before that time, won't be taken. Thanks, Rafael
diff --git a/arch/powerpc/platforms/Kconfig b/arch/powerpc/platforms/Kconfig index 52de8bc..46a223f 100644 --- a/arch/powerpc/platforms/Kconfig +++ b/arch/powerpc/platforms/Kconfig @@ -194,37 +194,6 @@ config PPC_IO_WORKAROUNDS source "drivers/cpufreq/Kconfig" -menu "CPU Frequency drivers" - depends on CPU_FREQ - -config CPU_FREQ_PMAC - bool "Support for Apple PowerBooks" - depends on ADB_PMU && PPC32 - select CPU_FREQ_TABLE - help - This adds support for frequency switching on Apple PowerBooks, - this currently includes some models of iBook & Titanium - PowerBook. - -config CPU_FREQ_PMAC64 - bool "Support for some Apple G5s" - depends on PPC_PMAC && PPC64 - select CPU_FREQ_TABLE - help - This adds support for frequency switching on Apple iMac G5, - and some of the more recent desktop G5 machines as well. - -config PPC_PASEMI_CPUFREQ - bool "Support for PA Semi PWRficient" - depends on PPC_PASEMI - default y - select CPU_FREQ_TABLE - help - This adds the support for frequency switching on PA Semi - PWRficient processors. - -endmenu - menu "CPUIdle driver" source "drivers/cpuidle/Kconfig" diff --git a/arch/powerpc/platforms/pasemi/Makefile b/arch/powerpc/platforms/pasemi/Makefile index ce6d789..8e8d4ca 100644 --- a/arch/powerpc/platforms/pasemi/Makefile +++ b/arch/powerpc/platforms/pasemi/Makefile @@ -1,3 +1,2 @@ obj-y += setup.o pci.o time.o idle.o powersave.o iommu.o dma_lib.o misc.o obj-$(CONFIG_PPC_PASEMI_MDIO) += gpio_mdio.o -obj-$(CONFIG_PPC_PASEMI_CPUFREQ) += cpufreq.o diff --git a/arch/powerpc/platforms/powermac/Makefile b/arch/powerpc/platforms/powermac/Makefile index ea47df6..52c6ce1 100644 --- a/arch/powerpc/platforms/powermac/Makefile +++ b/arch/powerpc/platforms/powermac/Makefile @@ -9,8 +9,6 @@ obj-y += pic.o setup.o time.o feature.o pci.o \ sleep.o low_i2c.o cache.o pfunc_core.o \ pfunc_base.o udbg_scc.o udbg_adb.o obj-$(CONFIG_PMAC_BACKLIGHT) += backlight.o -obj-$(CONFIG_CPU_FREQ_PMAC) += cpufreq_32.o -obj-$(CONFIG_CPU_FREQ_PMAC64) += cpufreq_64.o # CONFIG_NVRAM is an arch. independent tristate symbol, for pmac32 we really # need this to be a bool. Cheat here and pretend CONFIG_NVRAM=m is really # CONFIG_NVRAM=y diff --git a/drivers/cpufreq/Kconfig.powerpc b/drivers/cpufreq/Kconfig.powerpc index e76992f..2e5a007 100644 --- a/drivers/cpufreq/Kconfig.powerpc +++ b/drivers/cpufreq/Kconfig.powerpc @@ -5,3 +5,29 @@ config CPU_FREQ_MAPLE help This adds support for frequency switching on Maple 970FX Evaluation Board and compatible boards (IBM JS2x blades). + +config CPU_FREQ_PMAC + bool "Support for Apple PowerBooks" + depends on ADB_PMU && PPC32 + select CPU_FREQ_TABLE + help + This adds support for frequency switching on Apple PowerBooks, + this currently includes some models of iBook & Titanium + PowerBook. + +config CPU_FREQ_PMAC64 + bool "Support for some Apple G5s" + depends on PPC_PMAC && PPC64 + select CPU_FREQ_TABLE + help + This adds support for frequency switching on Apple iMac G5, + and some of the more recent desktop G5 machines as well. + +config PPC_PASEMI_CPUFREQ + bool "Support for PA Semi PWRficient" + depends on PPC_PASEMI + select CPU_FREQ_TABLE + default y + help + This adds the support for frequency switching on PA Semi + PWRficient processors. diff --git a/drivers/cpufreq/Makefile b/drivers/cpufreq/Makefile index 0203a06..fa4b5f2 100644 --- a/drivers/cpufreq/Makefile +++ b/drivers/cpufreq/Makefile @@ -75,6 +75,9 @@ obj-$(CONFIG_ARCH_TEGRA) += tegra-cpufreq.o ################################################################################## # PowerPC platform drivers obj-$(CONFIG_CPU_FREQ_MAPLE) += maple-cpufreq.o +obj-$(CONFIG_CPU_FREQ_PMAC) += pmac32-cpufreq.o +obj-$(CONFIG_CPU_FREQ_PMAC64) += pmac64-cpufreq.o +obj-$(CONFIG_PPC_PASEMI_CPUFREQ) += pasemi-cpufreq.o ################################################################################## # Other platform drivers
This patch moves cpufreq driver of powerpc platform to drivers/cpufreq. Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org> Cc: Paul Mackerras <paulus@samba.org> Cc: Olof Johansson <olof@lixom.net> Cc: linuxppc-dev@lists.ozlabs.org Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org> --- arch/powerpc/platforms/Kconfig | 31 ---------------------- arch/powerpc/platforms/pasemi/Makefile | 1 - arch/powerpc/platforms/powermac/Makefile | 2 -- drivers/cpufreq/Kconfig.powerpc | 26 ++++++++++++++++++ drivers/cpufreq/Makefile | 3 +++ .../cpufreq.c => drivers/cpufreq/pasemi-cpufreq.c | 0 .../cpufreq/pmac32-cpufreq.c | 0 .../cpufreq/pmac64-cpufreq.c | 0 8 files changed, 29 insertions(+), 34 deletions(-) rename arch/powerpc/platforms/pasemi/cpufreq.c => drivers/cpufreq/pasemi-cpufreq.c (100%) rename arch/powerpc/platforms/powermac/cpufreq_32.c => drivers/cpufreq/pmac32-cpufreq.c (100%) rename arch/powerpc/platforms/powermac/cpufreq_64.c => drivers/cpufreq/pmac64-cpufreq.c (100%) diff --git a/arch/powerpc/platforms/pasemi/cpufreq.c b/drivers/cpufreq/pasemi-cpufreq.c similarity index 100% rename from arch/powerpc/platforms/pasemi/cpufreq.c rename to drivers/cpufreq/pasemi-cpufreq.c diff --git a/arch/powerpc/platforms/powermac/cpufreq_32.c b/drivers/cpufreq/pmac32-cpufreq.c similarity index 100% rename from arch/powerpc/platforms/powermac/cpufreq_32.c rename to drivers/cpufreq/pmac32-cpufreq.c diff --git a/arch/powerpc/platforms/powermac/cpufreq_64.c b/drivers/cpufreq/pmac64-cpufreq.c similarity index 100% rename from arch/powerpc/platforms/powermac/cpufreq_64.c rename to drivers/cpufreq/pmac64-cpufreq.c