From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH] cpuidle: add maintainer entry Date: Wed, 24 Apr 2013 16:23:12 +0200 Message-ID: <20130424142312.GA2824@lunn.ch> References: <1366810463-17495-1-git-send-email-daniel.lezcano@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from londo.lunn.ch ([80.238.139.98]:42370 "EHLO londo.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753978Ab3DXOXa (ORCPT ); Wed, 24 Apr 2013 10:23:30 -0400 Content-Disposition: inline In-Reply-To: <1366810463-17495-1-git-send-email-daniel.lezcano@linaro.org> Sender: linux-pm-owner@vger.kernel.org List-Id: linux-pm@vger.kernel.org To: Daniel Lezcano Cc: rjw@sisk.pl, linux-pm@vger.kernel.org, linaro-kernel@lists.linaro.org, khilman@deeprootsystems.com, rob.herring@calxeda.com, jason@lakedaemon.net, andrew@lunn.ch, nicolas.ferre@atmel.com, plagnioj@jcrosoft.com, linux@maxim.org.za, kernel@pengutronix.de, swarren@wwwdotorg.org, horms+renesas@verge.net.au, josephl@nvidia.com, arnd@arndb.de, linux-arm-kernel@lists.infradead.org, horms@verge.net.au, magnus.damm@gmail.com, benh@kernel.crashing.org, paulus@samba.org, kgene.kim@samsung.com, linus.walleij@linaro.org, viresh.kumar@linaro.org, lenb@kernel.org, nsekhar@ti.com, ben-linux@fluff.org On Wed, Apr 24, 2013 at 03:34:23PM +0200, Daniel Lezcano wrote: > Currently the cpuidle drivers are spread across the different archs. > > The patch submission for cpuidle are following different path: the cpuidle core > code goes to linux-pm, the ARM drivers goes to arm-soc or the SoC specific > tree, sh goes through sh arch tree, pseries goes through PowerPC and > finally intel goes through Len's tree while acpi_idle goes under linux-pm. > > That makes difficult to consolidate the code and to propagate modifications > from the cpuidle core to the different drivers. > > Hopefully, a movement has initiated to put the cpuidle drivers into the > drivers/cpuidle directory like cpuidle-calxeda.c and cpuidle-kirkwood.c > > Add an explicit maintainer entry in the MAINTAINER to clarify the situation > and prevent new cpuidle drivers to goes to an arch directory. > > The upstreaming process is unchanged: Rafael takes the patches to merge them > into its tree but with the acked-by from the driver's maintainer. So the header > must contains the name of the maintainer. > > This organization will be the same than cpufreq. > > Signed-off-by: Daniel Lezcano > --- > MAINTAINERS | 7 +++++++ > drivers/cpuidle/cpuidle-calxeda.c | 4 +++- > drivers/cpuidle/cpuidle-kirkwood.c | 5 +++-- > 3 files changed, 13 insertions(+), 3 deletions(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 61677c3..effa0f3 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -2206,6 +2206,13 @@ S: Maintained > F: drivers/cpufreq/ > F: include/linux/cpufreq.h > > +CPUIDLE DRIVERS > +M: Rafael J. Wysocki > +L: linux-pm@vger.kernel.org > +S: Maintained > +F: drivers/cpuidle/* > +F: include/linux/cpuidle.h > + > CPU FREQUENCY DRIVERS - ARM BIG LITTLE > M: Viresh Kumar > M: Sudeep KarkadaNagesha > diff --git a/drivers/cpuidle/cpuidle-calxeda.c b/drivers/cpuidle/cpuidle-calxeda.c > index e344b56..2378c39 100644 > --- a/drivers/cpuidle/cpuidle-calxeda.c > +++ b/drivers/cpuidle/cpuidle-calxeda.c > @@ -1,7 +1,6 @@ > /* > * Copyright 2012 Calxeda, Inc. > * > - * Based on arch/arm/plat-mxc/cpuidle.c: > * Copyright 2012 Freescale Semiconductor, Inc. > * Copyright 2012 Linaro Ltd. > * > @@ -16,6 +15,9 @@ > * > * You should have received a copy of the GNU General Public License along with > * this program. If not, see . > + * > + * Author : Rob Herring > + * Maintainer: Rob Herring > */ > > #include > diff --git a/drivers/cpuidle/cpuidle-kirkwood.c b/drivers/cpuidle/cpuidle-kirkwood.c > index 53290e1..521b0a7 100644 > --- a/drivers/cpuidle/cpuidle-kirkwood.c > +++ b/drivers/cpuidle/cpuidle-kirkwood.c > @@ -1,6 +1,4 @@ > /* > - * arch/arm/mach-kirkwood/cpuidle.c > - * > * CPU idle Marvell Kirkwood SoCs > * > * This file is licensed under the terms of the GNU General Public > @@ -11,6 +9,9 @@ > * to implement two idle states - > * #1 wait-for-interrupt > * #2 wait-for-interrupt and DDR self refresh > + * > + * Maintainer: Jason Cooper > + * Maintainer: Andrew Lunn > */ > For the Kirkwood part Acked-by: Andrew Lunn Andrew From mboxrd@z Thu Jan 1 00:00:00 1970 From: andrew@lunn.ch (Andrew Lunn) Date: Wed, 24 Apr 2013 16:23:12 +0200 Subject: [PATCH] cpuidle: add maintainer entry In-Reply-To: <1366810463-17495-1-git-send-email-daniel.lezcano@linaro.org> References: <1366810463-17495-1-git-send-email-daniel.lezcano@linaro.org> Message-ID: <20130424142312.GA2824@lunn.ch> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, Apr 24, 2013 at 03:34:23PM +0200, Daniel Lezcano wrote: > Currently the cpuidle drivers are spread across the different archs. > > The patch submission for cpuidle are following different path: the cpuidle core > code goes to linux-pm, the ARM drivers goes to arm-soc or the SoC specific > tree, sh goes through sh arch tree, pseries goes through PowerPC and > finally intel goes through Len's tree while acpi_idle goes under linux-pm. > > That makes difficult to consolidate the code and to propagate modifications > from the cpuidle core to the different drivers. > > Hopefully, a movement has initiated to put the cpuidle drivers into the > drivers/cpuidle directory like cpuidle-calxeda.c and cpuidle-kirkwood.c > > Add an explicit maintainer entry in the MAINTAINER to clarify the situation > and prevent new cpuidle drivers to goes to an arch directory. > > The upstreaming process is unchanged: Rafael takes the patches to merge them > into its tree but with the acked-by from the driver's maintainer. So the header > must contains the name of the maintainer. > > This organization will be the same than cpufreq. > > Signed-off-by: Daniel Lezcano > --- > MAINTAINERS | 7 +++++++ > drivers/cpuidle/cpuidle-calxeda.c | 4 +++- > drivers/cpuidle/cpuidle-kirkwood.c | 5 +++-- > 3 files changed, 13 insertions(+), 3 deletions(-) > > diff --git a/MAINTAINERS b/MAINTAINERS > index 61677c3..effa0f3 100644 > --- a/MAINTAINERS > +++ b/MAINTAINERS > @@ -2206,6 +2206,13 @@ S: Maintained > F: drivers/cpufreq/ > F: include/linux/cpufreq.h > > +CPUIDLE DRIVERS > +M: Rafael J. Wysocki > +L: linux-pm at vger.kernel.org > +S: Maintained > +F: drivers/cpuidle/* > +F: include/linux/cpuidle.h > + > CPU FREQUENCY DRIVERS - ARM BIG LITTLE > M: Viresh Kumar > M: Sudeep KarkadaNagesha > diff --git a/drivers/cpuidle/cpuidle-calxeda.c b/drivers/cpuidle/cpuidle-calxeda.c > index e344b56..2378c39 100644 > --- a/drivers/cpuidle/cpuidle-calxeda.c > +++ b/drivers/cpuidle/cpuidle-calxeda.c > @@ -1,7 +1,6 @@ > /* > * Copyright 2012 Calxeda, Inc. > * > - * Based on arch/arm/plat-mxc/cpuidle.c: > * Copyright 2012 Freescale Semiconductor, Inc. > * Copyright 2012 Linaro Ltd. > * > @@ -16,6 +15,9 @@ > * > * You should have received a copy of the GNU General Public License along with > * this program. If not, see . > + * > + * Author : Rob Herring > + * Maintainer: Rob Herring > */ > > #include > diff --git a/drivers/cpuidle/cpuidle-kirkwood.c b/drivers/cpuidle/cpuidle-kirkwood.c > index 53290e1..521b0a7 100644 > --- a/drivers/cpuidle/cpuidle-kirkwood.c > +++ b/drivers/cpuidle/cpuidle-kirkwood.c > @@ -1,6 +1,4 @@ > /* > - * arch/arm/mach-kirkwood/cpuidle.c > - * > * CPU idle Marvell Kirkwood SoCs > * > * This file is licensed under the terms of the GNU General Public > @@ -11,6 +9,9 @@ > * to implement two idle states - > * #1 wait-for-interrupt > * #2 wait-for-interrupt and DDR self refresh > + * > + * Maintainer: Jason Cooper > + * Maintainer: Andrew Lunn > */ > For the Kirkwood part Acked-by: Andrew Lunn Andrew