From patchwork Thu Nov 15 07:02:27 2012 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "Zhang, Rui" X-Patchwork-Id: 1747251 X-Patchwork-Delegate: rui.zhang@intel.com Return-Path: X-Original-To: patchwork-linux-pm@patchwork.kernel.org Delivered-To: patchwork-process-083081@patchwork1.kernel.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by patchwork1.kernel.org (Postfix) with ESMTP id 94A3040E02 for ; Thu, 15 Nov 2012 07:02:44 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2992707Ab2KOHCc (ORCPT ); Thu, 15 Nov 2012 02:02:32 -0500 Received: from mga11.intel.com ([192.55.52.93]:32869 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2992548Ab2KOHCb (ORCPT ); Thu, 15 Nov 2012 02:02:31 -0500 Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga102.fm.intel.com with ESMTP; 14 Nov 2012 23:02:30 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.83,254,1352102400"; d="scan'208";a="247313172" Received: from unknown (HELO [10.255.21.124]) ([10.255.21.124]) by fmsmga001.fm.intel.com with ESMTP; 14 Nov 2012 23:02:28 -0800 Message-ID: <1352962947.2080.10.camel@rzhang1-mobl4> Subject: [RFC PATCH 2/2] Thermal: Exynos : add missing dependency From: Zhang Rui To: Linux PM list Cc: Sachin Kamat , Amit Kachhap , "Zhang, Rui" , patches@linaro.org Date: Thu, 15 Nov 2012 15:02:27 +0800 X-Mailer: Evolution 3.2.3-0ubuntu6 Mime-Version: 1.0 Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org there is still something unclear to me. Amit, the next step should be remove this "depends on CPU_THERMAL" line for EXYNOS driver, right? If we do this, the exynos driver should work w/ or w/o CPU_THERMAL, but should we have something like, "hey, I like CPU_THERMAL, I work better with it set"? "Select" does not work here because it may block exynos driver when CPU_FREQ cleared. any ideas on this? thanks, rui CPU_FREQ_TABLE depends on CPU_FREQ. Selecting CPU_FREQ_TABLE without checking for dependencies gives the following compilation warnings: warning: (ARCH_TEGRA_2x_SOC && ARCH_TEGRA_3x_SOC && UX500_SOC_DB8500 && CPU_THERMAL && EXYNOS_THERMAL) selects CPU_FREQ_TABLE which has unmet direct dependencies (ARCH_HAS_CPUFREQ && CPU_FREQ) Patch-based-on: Sachin Kamat Signed-off-by: Zhang Rui --- drivers/thermal/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/thermal/Kconfig b/drivers/thermal/Kconfig index 937a23d..99b6587 100644 --- a/drivers/thermal/Kconfig +++ b/drivers/thermal/Kconfig @@ -96,7 +96,7 @@ config RCAR_THERMAL config EXYNOS_THERMAL tristate "Temperature sensor on Samsung EXYNOS" depends on (ARCH_EXYNOS4 || ARCH_EXYNOS5) - select CPU_FREQ_TABLE + depends on CPU_THERMAL help If you say yes here you get support for TMU (Thermal Managment Unit) on SAMSUNG EXYNOS series of SoC.