From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754162Ab3FGLtH (ORCPT ); Fri, 7 Jun 2013 07:49:07 -0400 Received: from hydra.sisk.pl ([212.160.235.94]:59471 "EHLO hydra.sisk.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752047Ab3FGLtC (ORCPT ); Fri, 7 Jun 2013 07:49:02 -0400 From: "Rafael J. Wysocki" To: Viresh Kumar Cc: Robin Randhawa , Steve.Bannister@arm.com, Liviu.Dudau@arm.com, charles.garcia-tobin@arm.com, arvind.chauhan@arm.com, cpufreq@vger.kernel.org, linux-pm@vger.kernel.org, linux-kernel@vger.kernel.org, linaro-kernel@lists.linaro.org, patches@linaro.org, Paul Mackerras , Olof Johansson , linuxppc-dev@lists.ozlabs.org, Benjamin Herrenschmidt , deepthi@linux.vnet.ibm.com, Andrew Morton Subject: Re: [PATCH 17/18] cpufreq: powerpc: move cpufreq driver to drivers/cpufreq Date: Fri, 07 Jun 2013 13:58:05 +0200 Message-ID: <36081985.E8ct76IG4n@vostro.rjw.lan> User-Agent: KMail/4.9.5 (Linux/3.10.0-rc4+; KDE/4.9.5; x86_64; ; ) In-Reply-To: References: MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Friday, June 07, 2013 10:48:21 AM Viresh Kumar wrote: > On 31 May 2013 16:20, Viresh Kumar wrote: > > On 20 May 2013 10:10, Viresh Kumar wrote: > >> On 13 May 2013 11:34, Viresh Kumar wrote: > >>> On 22 April 2013 12:19, Viresh Kumar wrote: > >>>> On 9 April 2013 14:05, Viresh Kumar wrote: > >>>>> On 5 April 2013 12:16, Viresh Kumar wrote: > >>>>>> On 4 April 2013 18:24, Viresh Kumar wrote: > >>>>>>> This patch moves cpufreq driver of powerpc platform to drivers/cpufreq. > >>>>>>> > >>>>>>> Cc: Benjamin Herrenschmidt > >>>>>>> Cc: Paul Mackerras > >>>>>>> Cc: Olof Johansson > >>>>>>> Cc: linuxppc-dev@lists.ozlabs.org > >>>>>>> Signed-off-by: Viresh Kumar > >>>>>>> --- > >>>>>>> Compile Tested only. > >>>>>>> > >>>>>>> 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%) > >>>>>> > >>>>>> Hi Deepthi, > >>>>>> > >>>>>> Can you help testing this please? > >>>>> > >>>>> Ping!! > >>>> > >>>> Ping!! > >>> > >>> Hi Benjamin, > >>> > >>> Hope you are back from your vacations. Can you give it a try now? > >> > >> Ping!! > > > > Ping!! > > Hi Rafael, > > Its been more than 2 months now that this patch was first posted. > And the response from Maintainers isn't so great, irrespective of > how many times I pinged them. > > This is what I think: > - It looked functionally correct to Benjamin but he wanted somebody > to actually test it. > - Arnd gave his Ack (So it looked functionally correct to him too) > - We can probably push this into linux-next now and see if somebody > complains of any breakage it has done. If not we can get it pushed for > 3.11. I agree, applied to bleeding-edge. Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from hydra.sisk.pl (hydra.sisk.pl [212.160.235.94]) by ozlabs.org (Postfix) with ESMTP id 11F902C0098 for ; Fri, 7 Jun 2013 21:49:04 +1000 (EST) From: "Rafael J. Wysocki" To: Viresh Kumar Subject: Re: [PATCH 17/18] cpufreq: powerpc: move cpufreq driver to drivers/cpufreq Date: Fri, 07 Jun 2013 13:58:05 +0200 Message-ID: <36081985.E8ct76IG4n@vostro.rjw.lan> In-Reply-To: References: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Cc: deepthi@linux.vnet.ibm.com, Robin Randhawa , linux-pm@vger.kernel.org, patches@linaro.org, Liviu.Dudau@arm.com, linux-kernel@vger.kernel.org, cpufreq@vger.kernel.org, Steve.Bannister@arm.com, Paul Mackerras , Olof Johansson , Andrew Morton , arvind.chauhan@arm.com, linuxppc-dev@lists.ozlabs.org, linaro-kernel@lists.linaro.org, charles.garcia-tobin@arm.com List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Friday, June 07, 2013 10:48:21 AM Viresh Kumar wrote: > On 31 May 2013 16:20, Viresh Kumar wrote: > > On 20 May 2013 10:10, Viresh Kumar wrote: > >> On 13 May 2013 11:34, Viresh Kumar wrote: > >>> On 22 April 2013 12:19, Viresh Kumar wrote: > >>>> On 9 April 2013 14:05, Viresh Kumar wrote: > >>>>> On 5 April 2013 12:16, Viresh Kumar wrote: > >>>>>> On 4 April 2013 18:24, Viresh Kumar wrote: > >>>>>>> This patch moves cpufreq driver of powerpc platform to drivers/cpufreq. > >>>>>>> > >>>>>>> Cc: Benjamin Herrenschmidt > >>>>>>> Cc: Paul Mackerras > >>>>>>> Cc: Olof Johansson > >>>>>>> Cc: linuxppc-dev@lists.ozlabs.org > >>>>>>> Signed-off-by: Viresh Kumar > >>>>>>> --- > >>>>>>> Compile Tested only. > >>>>>>> > >>>>>>> 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%) > >>>>>> > >>>>>> Hi Deepthi, > >>>>>> > >>>>>> Can you help testing this please? > >>>>> > >>>>> Ping!! > >>>> > >>>> Ping!! > >>> > >>> Hi Benjamin, > >>> > >>> Hope you are back from your vacations. Can you give it a try now? > >> > >> Ping!! > > > > Ping!! > > Hi Rafael, > > Its been more than 2 months now that this patch was first posted. > And the response from Maintainers isn't so great, irrespective of > how many times I pinged them. > > This is what I think: > - It looked functionally correct to Benjamin but he wanted somebody > to actually test it. > - Arnd gave his Ack (So it looked functionally correct to him too) > - We can probably push this into linux-next now and see if somebody > complains of any breakage it has done. If not we can get it pushed for > 3.11. I agree, applied to bleeding-edge. Thanks, Rafael -- I speak only for myself. Rafael J. Wysocki, Intel Open Source Technology Center.