From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753196AbbBWVVY (ORCPT ); Mon, 23 Feb 2015 16:21:24 -0500 Received: from mail-we0-f181.google.com ([74.125.82.181]:45979 "EHLO mail-we0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751591AbbBWVVX (ORCPT ); Mon, 23 Feb 2015 16:21:23 -0500 Message-ID: <54EB99CE.1030806@linaro.org> Date: Mon, 23 Feb 2015 22:21:18 +0100 From: Daniel Lezcano User-Agent: Mozilla/5.0 (X11; Linux i686; rv:31.0) Gecko/20100101 Thunderbird/31.4.0 MIME-Version: 1.0 To: Oleksij Rempel , Guenter Roeck CC: linux-kernel@vger.kernel.org, tglx@linutronix.de Subject: Re: [PATCH] clockevents: asm9260: Fix compilation error with sparc/sparc64 allyesconfig References: <20150223175716.GA22430@roeck-us.net> <1424718051-18946-1-git-send-email-daniel.lezcano@linaro.org> <20150223191023.GA25441@roeck-us.net> <54EB8D15.3040208@rempel-privat.de> <20150223203417.GB32536@roeck-us.net> <54EB91BB.9020800@rempel-privat.de> In-Reply-To: <54EB91BB.9020800@rempel-privat.de> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/23/2015 09:46 PM, Oleksij Rempel wrote: > Am 23.02.2015 um 21:34 schrieb Guenter Roeck: >> On Mon, Feb 23, 2015 at 09:27:01PM +0100, Oleksij Rempel wrote: >>> Am 23.02.2015 um 20:10 schrieb Guenter Roeck: >>>> On Mon, Feb 23, 2015 at 08:00:51PM +0100, Daniel Lezcano wrote: >>>>> The Kconfig options for the asm9260 timer is wrong as it can be selected by >>>>> another platform with allyes config and thus leading to a compilation failure >>>>> as some non arch related code is pulled by the compilation. >>>>> >>>>> Fix this by having the platform Kconfig to select the timer as it is done for >>>>> the others drivers. >>>>> >>>>> Signed-off-by: Daniel Lezcano >>>> >>>> Acked-by: Guenter Roeck >>>> >>> >>> the thing is, this SoC or this company has no own IP cores. All of them >>> are from other companies and providers, including this timer. This is >>> why all modules for this SoC are selectable. >>> >> Maybe the modules should be selected by the architectures using the IP cores ? > > At the moment of pushing patches to arch/arm, maintainers preferred to > have minimalistic Kconfig. > >> Either case, we'll need _some_ fix upstream, whatever is acceptable for the >> maintainer. > > What is with initial white list patch? The Kconfig policy for the timers is to let the arch to select the timer they need and not enable them from the clocksource/Kconfig file. That is the general policy even there are a couple of exceptions. So if you are ok with the patch I sent I will merge it as a fix. -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog