Re: [PATCH 17/18] cpufreq: powerpc: move cpufreq driver to drivers/cpufreq

From: Rafael J. Wysocki
Date: Fri Jun 07 2013 - 07:49:10 EST


On Friday, June 07, 2013 10:48:21 AM Viresh Kumar wrote:
> On 31 May 2013 16:20, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> > On 20 May 2013 10:10, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> >> On 13 May 2013 11:34, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> >>> On 22 April 2013 12:19, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> >>>> On 9 April 2013 14:05, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> >>>>> On 5 April 2013 12:16, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> >>>>>> On 4 April 2013 18:24, Viresh Kumar <viresh.kumar@xxxxxxxxxx> wrote:
> >>>>>>> This patch moves cpufreq driver of powerpc platform to drivers/cpufreq.
> >>>>>>>
> >>>>>>> Cc: Benjamin Herrenschmidt <benh@xxxxxxxxxxxxxxxxxxx>
> >>>>>>> Cc: Paul Mackerras <paulus@xxxxxxxxx>
> >>>>>>> Cc: Olof Johansson <olof@xxxxxxxxx>
> >>>>>>> Cc: linuxppc-dev@xxxxxxxxxxxxxxxx
> >>>>>>> Signed-off-by: Viresh Kumar <viresh.kumar@xxxxxxxxxx>
> >>>>>>> ---
> >>>>>>> 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.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/