From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933462AbcINWJZ (ORCPT ); Wed, 14 Sep 2016 18:09:25 -0400 Received: from mout.kundenserver.de ([212.227.17.24]:60153 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1762024AbcINWJX (ORCPT ); Wed, 14 Sep 2016 18:09:23 -0400 From: Arnd Bergmann To: Will Deacon Cc: Leo Yan , Catalin Marinas , Simon Horman , Thierry Reding , Krzysztof Kozlowski , Alim Akhtar , Kuninori Morimoto , Riku Voipio , Srinivas Kandagatla , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v2] arm64: defconfig: enable common modules for power management Date: Thu, 15 Sep 2016 00:08:30 +0200 Message-ID: <5111815.vFFFHkSD4j@wuerfel> User-Agent: KMail/5.1.3 (Linux/4.4.0-34-generic; KDE/5.18.0; x86_64; ; ) In-Reply-To: <20160908102035.GD1493@arm.com> References: <1472705469-25599-1-git-send-email-leo.yan@linaro.org> <5498297.rN0szKFv4p@wuerfel> <20160908102035.GD1493@arm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:lK6cxJUl8tiW1D6aW0Jry55n2+9SalvfkU7sabL5AJkd5RQCqLf KH5HcLutUPLvJ8U8RtnMDWaeZcQhnTnhnDIj6QKOpCeZNvh+G2sHf7OHMT98Jx9Z+gZYHrh sSb+xtAkBYeAx/qADpyyjNb3PbhL0FDkA6Mm81woJNKwYHS/kZQMlw/l2cm53T/oDhyb64h 17COcRgfnPBYVhCe7b48Q== X-UI-Out-Filterresults: notjunk:1;V01:K0:DctTZ6h0vFY=:OPj/600uimVnAgjBhYVjCe 9IuQ/S+Ts64FPy9MdjjnNn5dMVv42IrA7/wOuum3OY/XIIJ8ZnXz/KLrtKMEzSsjnic2h9SK2 prCh+mHf8dxLxMd99BlZ4JK/xxUONIyFijhCnDZGSocl6bi0WSM7xfipgGs0fu1Lbz6HBRa8a xeA/OyzaE83lOtHGk3XyGGu3Mt2BgpusCfLoY6NtSIMM8io/YSQv06lLtxcvzM4JoioB7euPz HkhTCUWtNjR6O5OzjJPV2L+XksajG9Ow4knMlJ3IxLycubbZRTK3N0RDxgkvFFMMWb8/sZQIa SwWsXseeEu/rG2cki6Rl0dVerG4FmOoMDf4P7t66VQY5izV3YjXh2WfkmGxnT69zxcyvIWGs6 h+pmDBbo4SmX5tMhO8RTTZ0hzdTImz3XrQKtvPJDFSRpm/8Os+P4FNtchIy5eDDe3wU1kIN+Q LlvQwoL87bCl48EvR+qe2ItQHPdtcT4cP3x6Xgyugr989l9hijDL0xHEKShzW6Ix7Y8miFt1e kpwiFUai2TyvU1HPb4TgmRJ4fiwy5JqTfB/ueOq3asfLhoPEGI+Y8Ctpo5xbECbDLR9cHFJ/H Lqp6bP+OGU/NVE5+LnbdqXAxLhqdjHPh++Wbsj1FMoRYW9/sibMT9l6h+rYlTodRmPxf7860j FvnN1wArjqnzr7cq/blv9FM2q9K11RR+p7JLSJyYnLsia53E+YihpUt3A0ssFx/SNGT2LGj9Y n6LWggZMLmhFOWn3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday, September 8, 2016 11:20:35 AM CEST Will Deacon wrote: > Hi Arnd, > > On Thu, Sep 01, 2016 at 09:33:38AM +0200, Arnd Bergmann wrote: > > On Thursday, September 1, 2016 12:51:09 PM CEST Leo Yan wrote: > > > Enable common modules for power management; one is to enable > > > CPUFREQ_DT driver; the driver is used by many platforms by passing OPP > > > table from device tree. > > > > > > Also enables thermal related drivers. Firstly we need enable > > > configuration CPU_THERMAL for CPU cooling device driver, this will bind > > > thermal zone with CPU cooling device; and enable 'power allocator' > > > thermal governor. > > > > > > Signed-off-by: Leo Yan > > > > > > > Looks good, > > > > Acked-by: Arnd Bergmann > > > > Catalin and Will: I never know who should merge this kind of patch > > as we have a number of other arm64 defconfig patches going through > > arm-soc, but this patch is not really platform specific. > > > > Do you want to pick it up in the arm64 tree or should we take > > this and others patches like it in general? > > I'm perfectly happy with you taking this, but thanks for asking the > question. I think that, outside of the architectural bits, it makes > sense for arm-soc to manage config options that are requested by the > submaintainers. It also helps reduce defconfig churn. I typically wait > until -rc1 before making any .config changes, unless they're urgent > (e.g. systemd suddenly starts needing some options enabled in order to > boot), so if you queue this for the merge window then we should be good. Ok, thanks for the detailed answer. I've applied the patch to our next/arm64 branch now. Arnd