Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756899AbeAHQaT (ORCPT + 1 other); Mon, 8 Jan 2018 11:30:19 -0500 Received: from mail-ot0-f195.google.com ([74.125.82.195]:35825 "EHLO mail-ot0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751426AbeAHQaN (ORCPT ); Mon, 8 Jan 2018 11:30:13 -0500 X-Google-Smtp-Source: ACJfBot3VJ4oCawfFFu/BSbykGnxMxc/w6sreT7XHpxNCr9UDjaojXWJRi+DDGBGQsmOpQmcbhiERg9YxvLoM0jwIjc= MIME-Version: 1.0 In-Reply-To: <78dc7813-524d-c108-0e3d-516f8f4dabfe@linaro.org> References: <1513057621-19084-1-git-send-email-rickchen36@gmail.com> <1513057621-19084-2-git-send-email-rickchen36@gmail.com> <1e75edb3-8f7b-796c-6871-1612b027050e@linaro.org> <78dc7813-524d-c108-0e3d-516f8f4dabfe@linaro.org> From: Arnd Bergmann Date: Mon, 8 Jan 2018 17:30:11 +0100 X-Google-Sender-Auth: 9Vu6YI842umO_tlabVxKDYMSnIA Message-ID: Subject: Re: [PATCH v5 1/3] clocksource/drivers/atcpit100: Add andestech atcpit100 timer To: Daniel Lezcano Cc: Greentime Hu , Greentime , Rick Chen , Rick Chen , Linux Kernel Mailing List , Linus Walleij , linux-arch , Thomas Gleixner , Jason Cooper , Marc Zyngier , Rob Herring , netdev , Vincent Chen , DTML , Al Viro , David Howells , Will Deacon , linux-serial@vger.kernel.org, John Stultz Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Mon, Jan 8, 2018 at 5:08 PM, Daniel Lezcano wrote: > On 08/01/2018 16:26, Arnd Bergmann wrote: >> On Fri, Jan 5, 2018 at 10:31 AM, Daniel Lezcano >> wrote: >>> >>> etc ... >> >> I'd actually prefer to not do it for ARM either: Most other subsystems >> don't do that, and I don't see a strong reason why clocksource should >> be special here. > > The majority doing the opposite does not mean it is right. > > Do you know which clock belongs to which board ? Who will unselect a > clock ? I'm pretty sure nobody. Everyone relies on the platform Kconfig > and expect it to select the right drivers. The point is that there is no platform specific Kconfig that could select it, as the concept doesn't make a lot of sense here. If you require the driver to always be selected by the architecture code, it adds a little bit of bloat on systems that don't need it. This is possible, but I think it's preferable to give users a way of tuning a kernel for a particular chip. > We don't expect the hackers to have a deep knowledge of the hardware and > the driver dependencies. It is very convenient to not care about that > and let the platform's Kconfig to select the right drivers. > > And that is the behavior I would like to keep. I'm not worried about the driver bloating the kernel here when it isn't disabled, this is no different from enabling the USB controller by default for a board that doesn't have a USB connector, or enabling ten different pinctrl drivers for all members of a chip family even though you know which particular chip you are running on. >> Selecting 'TIMER_OF' from the individual drivers that need it (as you >> suggest) makes sense, but I think for ARM we treat SoC families >> as a bit too special, in the end they are for the most part collections >> of individual hardware blocks that may or may not be present on >> some chip. >> >> In case of risc-v and nds32, I expect that the separation will be >> even less visibile in the hardware, as a typical model here is >> that one company designs SoCs for multiple customers that each >> have different requirements. Some of them may have one >> timer and some have another timer or multiple timers, but there >> is no strict separation between SoC families as I understand. >> Here we'd be better off not having a per-SoC Kconfig option at >> all, just a generic defconfig that enables all the drivers that might >> be used, and integrators can have a defconfig file that only >> enables the stuff they actually use on a given chip. > > Yes, the result is the same, the option is not showed in the menu. > > However, I can understand it could be interesting to have the ability to > unselect a driver for experts. > > I'm wondering if we can create a bool_expert which shows up only when > CONFIG_EXPERT is set. Having a dependency on CONFIG_EXPERT means that a more users will end up having to set that for a shipping system. It's something we can do (even without a special Kconfig keyword), but it should be used carefully for stuff that 99% of the users want to enable. Why not just: config CLKSRC_ATCPIT100 bool "Clocksource for AE3XX platform" depends on NDS32 || COMPILE_TEST depends on HAS_IOMEM default NDS32 help This option enables support for the Andestech AE3XX platform timers. That way, it's simply enabled on NDS32 by default, but just as easy to disable for systems that don't need it. Arnd