From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752997AbbKYMu0 (ORCPT ); Wed, 25 Nov 2015 07:50:26 -0500 Received: from mout.kundenserver.de ([212.227.17.10]:58226 "EHLO mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752787AbbKYMuY (ORCPT ); Wed, 25 Nov 2015 07:50:24 -0500 From: Arnd Bergmann To: Daniel Lezcano Cc: linux-arm-kernel@lists.infradead.org, Stephen Boyd , linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, Andy Gross Subject: Re: [PATCH 1/3] clocksource: defbool CLKSRC_QCOM=y on ARCH_QCOM and make it visible Date: Wed, 25 Nov 2015 13:49:52 +0100 Message-ID: <2558718.Gz71lz6d4y@wuerfel> User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; ) In-Reply-To: <5655ABA1.6030206@linaro.org> References: <1448413710-8101-1-git-send-email-sboyd@codeaurora.org> <3550727.W9y7mWx54D@wuerfel> <5655ABA1.6030206@linaro.org> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" X-Provags-ID: V03:K0:vLHXeol1mlglsmfYIDdjueCJseTsvIa2aSAJbyVhwC8OCJzyx62 WMf7f6XXpjLMheibSsIZRM8X7QATcytasmehWZmGL4ifD66ZN+iuz4z374FyF1pWuBlaW/I RuzIOJNhjL5+uXhKOGb1eWPiHZ37l9vtSPhOEHc2IDZcptqf8/u6O5fWfC0Zu2MTfrkv6Sp qhdx9B2/DlGu0hcbKXDRA== X-UI-Out-Filterresults: notjunk:1;V01:K0:i6rmrL8ctqQ=:PfNIpTOyxkrhdiUrtWpxTO u1l/M0DvzlIpYTN7P7PYSUcfBRt9Qu8Hr5STtV8RvgNEQLTKnuLLpaLrSsgyGYK8yZ3Tj9lOt ftsGuSv8kAF+G1PV+MmGCiZ+NYf7X9MLVd+wEkzoGZTvXw04lLDsZJ1yWjl18eFUafR+dgV4Q XA+MSadcWnEP4a0WSjxYzMtua0Z/aBnBO/iaHNNqi3xBVByPtjpUNaDP8DJB+dR2JNY3WSkeR PxTgEo4HgyoVVA+g/kqDG2QwXxqmVF43gViM204YRQNsvBDvVjCknpnEI6G0wALhr3j3TwvGY 9djO2KsCa8lRo9tRTD1+9TVbXOGCvZ+TIAXsZ2llDSPIN+LbwQ/1+/mR+h+pARJYs5LcF+SOd Hm0NmCO1hFbqHgDUV2cj5ZuSgrAKPzrFAbng35u99PlrTdG5b9S419A9U3TjzgKaQiCT2E3pP bgqQCJPfWQrqn6I4QBWnXGIWe7cytwoZSNSHWIDx5F8nZmwIp/eYwgpIi+rexIM3M3SKf6iVI mAfOCOVQCyVGrP3UhYWoGpn9nxDCE5+jvDmdzT1bj4cZn8RVWVMDlM7C70NvJcCwxJ+9uE5UJ 29GTmaLYgpAM6N9h/5H35NbHRZyA/BxTE8YUonPxp5AuiqGVWZT7StPRxOyyf76uRbzw9KmTA SmncXhoyyFom9k7L1Ilj/9Yvo6Rij221g/XwpsnFWLdq0Na0RG1whjfW46DfkSc1CsHMWylu2 I2han8dRuqsRXq4Q Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wednesday 25 November 2015 13:37:53 Daniel Lezcano wrote: > On 11/25/2015 11:17 AM, Arnd Bergmann wrote: > > On Wednesday 25 November 2015 11:10:49 Daniel Lezcano wrote: > >> On 11/25/2015 02:08 AM, Stephen Boyd wrote: > >>> We want to remove the ARCH_MSM* configs in mach-qcom/Kconfig > >>> because they are mostly proxy configs for selecting the right > >>> clocksource driver. Therefore, make CLKSRC_QCOM default to the > >>> value of ARCH_QCOM, but also make it visible if ARCH_QCOM=y so > >>> that we can turn it off when we don't want it. > >> > >> I have been removing the ARCH dependencies in the Kconfig file. > >> > >> Why do you have to turn it off manually ? > > > > The background is that this is used only on some of the older > > MSM SoCs, while the newer ones use the arch timer. > > > > We decided to remove the SoC-specific top-level options from > > mach-msm as they are becoming rather meaningless these days > > and just a burden to maintain at the rate that new variants > > get released, so being able to turn off this driver helps make > > the kernel slightly smaller if you are building a kernel for > > only the more recent models. > > Ok, thanks for the clarification. > > I don't really like this approach even if it is correct because it > breaks the current approach I am trying to make consistent across the > drivers. > > I would like to have the COMPILE_TEST option available for all the > drivers and move this option under the menu config. This patch will > prevent to do this code factoring. How about moving the option to arch/arm/mach-qcom/Kconfig then? We could have a user-selectable "allow use of qcom clocksource" option there, which would then select the driver. > On the other side, this option is supposed to have a slightly smaller > kernel when it is not used. But when does it happen ? When > ARCH_MSM8X60=n and ARCH_MSM8960=n. With this patchset, I don't see the > ability to turn these SoCs off as the options are removed. So the > associated code is not removed, right ? > > So why allow to turn off the timer but disallow that for the entire SoC ? The timer is the only code that is controlled by those two options at the moment, all the other differences between SoCs are already handled by enabling the respective device drivers. Arnd