From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753873Ab3EaPNg (ORCPT ); Fri, 31 May 2013 11:13:36 -0400 Received: from comal.ext.ti.com ([198.47.26.152]:45427 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753111Ab3EaPN1 (ORCPT ); Fri, 31 May 2013 11:13:27 -0400 Message-ID: <51A8BE06.70009@ti.com> Date: Fri, 31 May 2013 11:13:10 -0400 From: Eduardo Valentin User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130404 Thunderbird/17.0.5 MIME-Version: 1.0 To: Amit Daniel Kachhap CC: , Zhang Rui , , , , Kukjin Kim , Eduardo Valentin Subject: Re: [PATCH V4 03/30] thermal: exynos: Remove CPU_THERMAL dependency for using TMU driver References: <1368525540-15034-1-git-send-email-amit.daniel@samsung.com> <1368525540-15034-4-git-send-email-amit.daniel@samsung.com> In-Reply-To: <1368525540-15034-4-git-send-email-amit.daniel@samsung.com> X-Enigmail-Version: 1.5.1 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="----enig2NXLPVKHLHSUDKEPTIENI" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org ------enig2NXLPVKHLHSUDKEPTIENI Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 14-05-2013 05:58, Amit Daniel Kachhap wrote: > This patch removes the dependency on CPU_THERMAL for compiling TMU driv= er. > This is useful for cases when only TMU controller needs to be initialis= ed > without cpu cooling action. Agreed with your intention. I just do not know if it makes sense to make this change at this point. Maybe after you have split the code? The above concern is simply from a non-functional perspective. If you do this at this point of your series, your driver may have compilation issues at this specific commit, in case your config does not have CONFIG_CPU_THERMAL. I recommend you moving this patch further in your series, to a place where you have isolated the code that depends on CPU_THERMAL. >=20 > Acked-by: Kukjin Kim > Signed-off-by: Amit Daniel Kachhap > --- > drivers/thermal/samsung/Kconfig | 1 - > 1 files changed, 0 insertions(+), 1 deletions(-) >=20 > diff --git a/drivers/thermal/samsung/Kconfig b/drivers/thermal/samsung/= Kconfig > index 145a55d..b5ab971 100644 > --- a/drivers/thermal/samsung/Kconfig > +++ b/drivers/thermal/samsung/Kconfig > @@ -4,7 +4,6 @@ config ARCH_HAS_TMU > config EXYNOS_THERMAL > tristate "Temperature sensor on Samsung EXYNOS" > depends on ARCH_HAS_TMU > - depends on CPU_THERMAL > help > If you say yes here you get support for TMU (Thermal Management > Unit) on SAMSUNG EXYNOS series of SoC. This helps in registering >=20 --=20 You have got to be excited about what you are doing. (L. Lamport) Eduardo Valentin ------enig2NXLPVKHLHSUDKEPTIENI Content-Type: application/pgp-signature; name="signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iF4EAREIAAYFAlGovgYACgkQCXcVR3XQvP2y3AEApjG7CdQf1WO7FgGuqyKOIjT0 2PfZr6UuS5LMAxzo6uYBALx1GGC+eKxSTR2we62Fe3rPXKUtPnIYpVs6XUAB7TeL =msjL -----END PGP SIGNATURE----- ------enig2NXLPVKHLHSUDKEPTIENI--