Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933080AbbBIPAg (ORCPT ); Mon, 9 Feb 2015 10:00:36 -0500 Received: from comal.ext.ti.com ([198.47.26.152]:39321 "EHLO comal.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1759408AbbBIPAe (ORCPT ); Mon, 9 Feb 2015 10:00:34 -0500 Date: Mon, 9 Feb 2015 09:00:09 -0600 From: Nishanth Menon To: Paul Walmsley CC: Eduardo Valentin , Zhang Rui , , , , Subject: Re: [PATCH] thermal: ti-soc-thermal: fix compilation warnings when !CONFIG_PM_SLEEP Message-ID: <20150209150009.GA21362@kahuna> References: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2382 Lines: 66 On 00:17-20150207, Paul Walmsley wrote: > > When CONFIG_PM_SLEEP=n, the following compilation warnings appear: > > drivers/thermal/ti-soc-thermal/ti-bandgap.c:1478:12: warning: ‘ti_bandgap_suspend’ defined but not used [-Wunused-function] > static int ti_bandgap_suspend(struct device *dev) > ^ > drivers/thermal/ti-soc-thermal/ti-bandgap.c:1492:12: warning: ‘ti_bandgap_resume’ defined but not used [-Wunused-function] > static int ti_bandgap_resume(struct device *dev) > > A few years ago, suspend/resume-related code was protected by CONFIG_PM. > But that was changed and now it's protected by CONFIG_PM_SLEEP. Fix the > warnings by converting the preprocessor test in the TI bandgap thermal > sensor driver from testing CONFIG_PM to testing CONFIG_PM_SLEEP. > > Signed-off-by: Paul Walmsley > Cc: Eduardo Valentin > Cc: Zhang Rui > Cc: linux-pm@vger.kernel.org > Cc: linux-omap@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > > Non-critical. Targeted for either v3.20-rc1 or v3.21-rc1. > > Basic test reports for a series that includes this patch (and a few other > unrelated warning cleanups) is available here: > > http://www.pwsan.com/omap/testlogs/fix-omap-warnings-v3.21/20150206154619/ > > Compare to: > > http://www.pwsan.com/omap/testlogs/test_v3.19-rc7/20150204213018/ > > drivers/thermal/ti-soc-thermal/ti-bandgap.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/thermal/ti-soc-thermal/ti-bandgap.c b/drivers/thermal/ti-soc-thermal/ti-bandgap.c > index 634b6ce0e63a..62a5d449c388 100644 > --- a/drivers/thermal/ti-soc-thermal/ti-bandgap.c > +++ b/drivers/thermal/ti-soc-thermal/ti-bandgap.c > @@ -1402,7 +1402,7 @@ int ti_bandgap_remove(struct platform_device *pdev) > return 0; > } > > -#ifdef CONFIG_PM > +#ifdef CONFIG_PM_SLEEP > static int ti_bandgap_save_ctxt(struct ti_bandgap *bgp) > { > int i; > -- > 2.1.4 I am a little confused: https://patchwork.kernel.org/patch/5792411/ was already posted. would you suggest improvements? -- Regards, Nishanth Menon -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/