From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753417Ab3KUJUr (ORCPT ); Thu, 21 Nov 2013 04:20:47 -0500 Received: from cantor2.suse.de ([195.135.220.15]:37912 "EHLO mx2.suse.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751226Ab3KUJUp (ORCPT ); Thu, 21 Nov 2013 04:20:45 -0500 Date: Thu, 21 Nov 2013 10:20:43 +0100 Message-ID: From: Takashi Iwai To: Stephen Boyd Cc: Pavel Machek , linux-kernel@vger.kernel.org, daniel.lezcano@linaro.org, will.deacon@arm.com, catalin.marinas@arm.com, olof@lixom.net, sudeep.karkadanagesha@arm.com Subject: Re: [PATCH] drivers: clocksource: Hide arm_arch_timer eventstream Kconfig on non-ARM In-Reply-To: <1384977723-32344-1-git-send-email-sboyd@codeaurora.org> References: <20131120192100.GA11723@amd.pavel.ucw.cz> <1384977723-32344-1-git-send-email-sboyd@codeaurora.org> User-Agent: Wanderlust/2.15.9 (Almost Unreal) SEMI/1.14.6 (Maruoka) FLIM/1.14.9 (=?UTF-8?B?R29qxY0=?=) APEL/10.8 Emacs/24.3 (x86_64-suse-linux-gnu) MULE/6.0 (HANACHIRUSATO) MIME-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org At Wed, 20 Nov 2013 12:02:03 -0800, Stephen Boyd wrote: > > Pavel Machek reports that this config is exposed on x86 where the > ARM architected timers aren't even present. Make it depend on the > ARM architected timers being selected so that non-ARM builds > aren't asked about it. > > Reported-by: Pavel Machek > Signed-off-by: Stephen Boyd > --- > drivers/clocksource/Kconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig > index bdb953e..5c07a56 100644 > --- a/drivers/clocksource/Kconfig > +++ b/drivers/clocksource/Kconfig > @@ -87,6 +87,7 @@ config ARM_ARCH_TIMER > config ARM_ARCH_TIMER_EVTSTREAM > bool "Support for ARM architected timer event stream generation" > default y if ARM_ARCH_TIMER > + depends on ARM_ARCH_TIMER We can drop "if ARM_ARCH_TIMER" in "default y" line now. But do we need "default y" at the first place? Takashi