From patchwork Fri Mar 29 15:50:55 2013 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Daniel Lezcano X-Patchwork-Id: 15770 Return-Path: X-Original-To: patchwork@peony.canonical.com Delivered-To: patchwork@peony.canonical.com Received: from fiordland.canonical.com (fiordland.canonical.com [91.189.94.145]) by peony.canonical.com (Postfix) with ESMTP id D955E23E2C for ; Fri, 29 Mar 2013 15:51:02 +0000 (UTC) Received: from mail-ve0-f181.google.com (mail-ve0-f181.google.com [209.85.128.181]) by fiordland.canonical.com (Postfix) with ESMTP id 79DBFA18E3E for ; Fri, 29 Mar 2013 15:51:02 +0000 (UTC) Received: by mail-ve0-f181.google.com with SMTP id pa12so649119veb.26 for ; Fri, 29 Mar 2013 08:51:02 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20120113; h=x-received:x-forwarded-to:x-forwarded-for:delivered-to:x-received :received-spf:x-received:message-id:date:from:user-agent :mime-version:to:cc:subject:references:in-reply-to:content-type :content-transfer-encoding:x-gm-message-state; bh=vAVMj6bmla8SWf1n3KxAJLxWQSH0ndCUhCt/DiaVpHg=; b=lK0yP39kk+vw0zGdx+2gOwSUHpaXiNaJTKyxG40l04WMLAGY4K/0RBuG/Cf7pq+q0C bu7DKkzJ5mNqNKSfuRx+WbCyLNkqRUue7k46/XwYXufI0+oYn+qOrcff32WUEoGL9vb4 jjQ2s8IgTHvTrJtGinkBsX0NRjso44pXDJi8VP/UnbR4p1Cwdl6AISehJ9w4MH2CgrXV 8KboaheVxP5zjD0jd8rPl0RrpK8YHhnIJpR3Q2eUiScd2S3bgK0X21XTWa6UXmqVlFl0 NfYhnSwL5Sj6GUwkMI7Q9iYn6rAgagB2/b1494IWidETzuVVhgt+jEn3M2s32W9hssQN b3fQ== X-Received: by 10.220.153.143 with SMTP id k15mr2192666vcw.33.1364572261976; Fri, 29 Mar 2013 08:51:01 -0700 (PDT) X-Forwarded-To: linaro-patchwork@canonical.com X-Forwarded-For: patch@linaro.org linaro-patchwork@canonical.com Delivered-To: patches@linaro.org Received: by 10.59.4.204 with SMTP id cg12csp65167ved; Fri, 29 Mar 2013 08:51:00 -0700 (PDT) X-Received: by 10.194.22.5 with SMTP id z5mr4423551wje.5.1364572260433; Fri, 29 Mar 2013 08:51:00 -0700 (PDT) Received: from mail-wi0-x22f.google.com ([2a00:1450:400c:c05::22f]) by mx.google.com with ESMTPS id ln8si1235683wjb.146.2013.03.29.08.50.59 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 29 Mar 2013 08:51:00 -0700 (PDT) Received-SPF: neutral (google.com: 2a00:1450:400c:c05::22f is neither permitted nor denied by best guess record for domain of daniel.lezcano@linaro.org) client-ip=2a00:1450:400c:c05::22f; Authentication-Results: mx.google.com; spf=neutral (google.com: 2a00:1450:400c:c05::22f is neither permitted nor denied by best guess record for domain of daniel.lezcano@linaro.org) smtp.mail=daniel.lezcano@linaro.org Received: by mail-wi0-f175.google.com with SMTP id c10so1265wiw.14 for ; Fri, 29 Mar 2013 08:50:59 -0700 (PDT) X-Received: by 10.194.103.72 with SMTP id fu8mr4265426wjb.42.1364572259050; Fri, 29 Mar 2013 08:50:59 -0700 (PDT) Received: from [192.168.1.13] (AToulouse-654-1-486-7.w92-146.abo.wanadoo.fr. [92.146.77.7]) by mx.google.com with ESMTPS id q13sm3939459wie.0.2013.03.29.08.50.56 (version=TLSv1 cipher=ECDHE-RSA-RC4-SHA bits=128/128); Fri, 29 Mar 2013 08:50:58 -0700 (PDT) Message-ID: <5155B85F.6030808@linaro.org> Date: Fri, 29 Mar 2013 16:50:55 +0100 From: Daniel Lezcano User-Agent: Mozilla/5.0 (X11; Linux i686; rv:17.0) Gecko/20130308 Thunderbird/17.0.4 MIME-Version: 1.0 To: Santosh Shilimkar CC: Amit Kucheria , Arnd Bergmann , Olof Johansson , Lists linaro-kernel , Russell King - ARM Linux , swarren@wwwdotorg.org, Patch Tracking , linux-pm@vger.kernel.org, "Nori, Sekhar" , "Rafael J. Wysocki" , Rajendra Nayak , linux-tegra@vger.kernel.org, horms+renesas@verge.net.au, Lists LAKML , Len Brown , Deepthi Dharwar , benh@kernel.crashing.org, lethal@linux-sh.org Subject: How to facilitate the cpuidle drivers to go to the same direction (Was: Re: [PATCH 4/9] ARM: OMAP4: cpuidle: fix wrong driver initialization) References: <1364553095-25110-1-git-send-email-daniel.lezcano@linaro.org> <1364553095-25110-4-git-send-email-daniel.lezcano@linaro.org> <51556F1D.5030208@ti.com> <515570DF.5010608@linaro.org> <51558723.1050904@ti.com> <5155AEE7.106@ti.com> In-Reply-To: <5155AEE7.106@ti.com> X-Gm-Message-State: ALoCoQn8cKRq71HyZ4nm63lZ9YUBSFCB19y+lKkdcuoZj0D9yd4JS0fF+zqrbK2O8BQS0nMyzDP5 On 03/29/2013 04:10 PM, Santosh Shilimkar wrote: > On Friday 29 March 2013 06:20 PM, Amit Kucheria wrote: >> On Fri, Mar 29, 2013 at 5:50 PM, Santosh Shilimkar >> wrote: >>> On Friday 29 March 2013 05:26 PM, Amit Kucheria wrote: >>>> On Fri, Mar 29, 2013 at 4:15 PM, Daniel Lezcano >>>> wrote: >>>>> On 03/29/2013 11:38 AM, Santosh Shilimkar wrote: >>>>>> On Friday 29 March 2013 04:01 PM, Daniel Lezcano wrote: >>>>>>> The driver is initialized several times. This is wrong and if the >>>>>>> return code of the function was checked, it will return -EINVAL. >>>>>>> >>>>>>> Move this initialization out of the loop. >>>>>>> >>>>>>> Signed-off-by: Daniel Lezcano >>>>>>> --- >>>>>> Fix for this is already and v2 of the patch is here [1] >>>>> >>>>> Ah, ok. Thanks for reviewing the patch. >>>>> >>>>> Can we find a solution to have a single entry point to sumbit patches >>>>> for all the cpuidle drivers ? >>>>> >>>>> Otherwise, consolidating them is a pain: a patch for the samsung tree, >>>>> another one for the at91 tree, etc ... and wait for all the trees to >>>>> sync before continuing to consolidate the code. >>>>> >>>>> Wouldn't be worth to move these drivers under the PM umbrella instead of >>>>> the SoC specific code ? >>>>> >>>>> Any idea to simplify the cpuidle consolidation and maintenance ? >>>> >>>> Adding Arnd and Olof to this discussion since atleast the ARM drivers >>>> go through their arm-soc tree. >>>> >>>> Given the work you're putting in to consolidate the drivers, perhaps >>>> they can insist that idle drivers get acked by you? >>>> >>> Not to create controversy but as a general rule there is nothing >>> like *insisting* ack on patches for merge apart from the official >>> maintainers(gate keepers). >>> >>> Having said that, its always good to get more reviews and acks so >>> that better code gets merged. >>> >>> This just my personal opinion. >> >> I'm not asking for special treatment here. :) I'm requesting one set >> of maintainers (arm-soc maintainers) to push back on changes that >> don't get platform idle drivers in sync with the consolidation work >> that is currently ongoing. >> >> This will speed up the process since it is hard to track every >> SoC-specific list for these changes. Some platform maintainers might >> not even be aware of it (those that Daniel hasn't modified yet). A >> similar approach seems to have worked for common clock, DT, pinmux, >> etc. >> > Every patch gets pulled into arm-soc/arm-core has to be posted on > LAKML. So as long as everybody follows that rule, there is no need to > track every SoC lists. And what I have seen so far every this rule > has been followed well. (Added Benjamin, Deepthi and Paul) I don't think everybody is following this rule, patches go to the SoC maintainer's tree without sometimes going through lakml. Furthermore, there is not only ARM, there is also acpi_idle, intel_idle, pseries_idle and sh_idle, respectively located in: drivers/acpi/processor_idle.c drivers/acpi/processor_driver.c drivers/idle/intel_idle.c These ones above are under linux-pm, that is Rafael, like cpuidle, even if it is not marked in the MAINTAINERS file, so that should be ok. And there is also: arch/sh/kernel/cpu/shmobile/cpuidle.c arch/powerpc/platforms/pseries/processor_idle.c And hopefully, some others in the right place, calxeda_idle and kirwood_idle located in drivers/cpuidle. In the maintainer file, there is no information about cpuidle at all. For example, if someone modify the cpuidle framework allowing to consolidate the code across the different drivers, we have to wait for the merge before using the new api into the different drivers. If we follow strictly the path of the merge tree we fall into a scenario where consolidating the drivers takes a loooooong time. >From my POV, *all* the cpuidle drivers must go under drivers/cpuidle, like cpufreq and pass through a single entry point to apply the patches, so the cpuidle framework and the drivers are always synced. If everyone agree and we reach this consensus, then we can work to move these drivers to a single place. I think Amit was suggesting to Cc me in the meantime, so while we are moving these drivers to this place, I can help to ensure we go to the same direction. For example, Arnd Cc'ed me about the zynq cpuidle driver when it has been posted and, after review, it appeared it was totally obsolete wrt the modifications we did this year. I propose first to add an entry in MAINTAINERS: Does it make sense ? Thanks -- Daniel diff --git a/MAINTAINERS b/MAINTAINERS index 4cf5fd3..5b5ab87 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -2206,6 +2206,15 @@ S: Maintained F: drivers/cpufreq/ F: include/linux/cpufreq.h +CPU IDLE DRIVERS +M: Rafael J. Wysocki +L: cpuidle@vger.kernel.org +L: linux-pm@vger.kernel.org +S: Maintained +F: drivers/cpuidle/ +F: include/linux/cpuidle.h + CPUID/MSR DRIVER M: "H. Peter Anvin" S: Maintained