From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kumar Gala Subject: Re: [PATCH v2 2/5] clocksource: qcom: Move clocksource code out of mach-msm Date: Tue, 4 Feb 2014 17:51:06 -0600 Message-ID: References: <1391553532-27001-1-git-send-email-galak@codeaurora.org> <1391553532-27001-3-git-send-email-galak@codeaurora.org> <20140204230956.GE20528@codeaurora.org> Mime-Version: 1.0 (Mac OS X Mail 7.1 \(1827\)) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8BIT Return-path: Received: from smtp.codeaurora.org ([198.145.11.231]:36835 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933157AbaBDXvK convert rfc822-to-8bit (ORCPT ); Tue, 4 Feb 2014 18:51:10 -0500 In-Reply-To: <20140204230956.GE20528@codeaurora.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Stephen Boyd Cc: linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, Daniel Lezcano On Feb 4, 2014, at 5:09 PM, Stephen Boyd wrote: > On 02/04, Kumar Gala wrote: >> We intent to share the clocksource code for MSM platforms between legacy > > s/intent/intend/ > >> and multiplatform supported qcom SoCs. >> >> Acked-by: Olof Johansson >> Signed-off-by: Kumar Gala > >> diff --git a/drivers/clocksource/Kconfig b/drivers/clocksource/Kconfig >> index cd6950f..81caa16 100644 >> --- a/drivers/clocksource/Kconfig >> +++ b/drivers/clocksource/Kconfig >> @@ -140,3 +140,7 @@ config VF_PIT_TIMER >> bool >> help >> Support for Period Interrupt Timer on Freescale Vybrid Family SoCs. >> + >> +config CLKSRC_QCOM >> + bool >> + depends on ARCH_MSM > > It would be a a little less noisy if you left this as MSM_TIMER > (msm is all over the code in that file anyway). Also, why do we > care about depending on ARCH_MSM here? This is a hidden Kconfig > option anyway. I will drop the depends, but not sure what point in leaving this as MSM_TIMER. - k -- Employee of Qualcomm Innovation Center, Inc. Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by The Linux Foundation