From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paul Gortmaker Subject: Re: [PATCH] drivers: qcom: spm: avoid module usage in non-modular SPM driver Date: Mon, 4 Jan 2016 13:30:17 -0500 Message-ID: <20160104183017.GO12458@windriver.com> References: <1451927725-7257-1-git-send-email-lina.iyer@linaro.org> <20160104173012.GM12458@windriver.com> <20160104174030.GG3342@linaro.org> <20160104175233.GN12458@windriver.com> <20160104180239.GH3342@linaro.org> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Return-path: Received: from mail1.windriver.com ([147.11.146.13]:34287 "EHLO mail1.windriver.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751972AbcADScT (ORCPT ); Mon, 4 Jan 2016 13:32:19 -0500 Content-Disposition: inline In-Reply-To: <20160104180239.GH3342@linaro.org> Sender: linux-arm-msm-owner@vger.kernel.org List-Id: linux-arm-msm@vger.kernel.org To: Lina Iyer Cc: andy.gross@linaro.org, sboyd@codeaurora.org, linux-pm@vger.kernel.org, linux-arm-msm@vger.kernel.org [Re: [PATCH] drivers: qcom: spm: avoid module usage in non-modular SPM driver] On 04/01/2016 (Mon 11:02) Lina Iyer wrote: > On Mon, Jan 04 2016 at 10:52 -0700, Paul Gortmaker wrote: [...] > >Looking at my queue, there are still two others in SOC I'll need to send > >eventually: > > > >drivers-soc-make-sunxi_sram.c-explicitly-non-modular.patch > >drivers-soc-tegra-make-fuse-tegra.c-explicitly-non-m.patch > > > >...and there are lots of others that I need to do a final review and test > >on before sending, so one less here and there is definitely welcome. > > > I figured if you are making changes to drivers, then you would do it for > SPM but it needs to be tested out. I tested it out on a IFC6540 Dragonboard. Well, if the compiled code is invariant due to substitution of equivalent macros as it is in this case, then compile testing for any fat finger issues is sufficient, as the run time will be invariant. But thanks for testing it regardless. Paul. -- > > Thanks. > > -- Lina > > >Thanks, > >Paul. > >-- > > > >> > >>-- Lina > >> > >>> * This program is free software; you can redistribute it and/or modify > >>> * it under the terms of the GNU General Public License version 2 and > >>> * only version 2 as published by the Free Software Foundation. > >>> > >>> > >>>Paul. > >>>-- > >>> > >>>>Signed-off-by: Lina Iyer > >>>>--- > >>>> drivers/soc/qcom/spm.c | 7 +------ > >>>> 1 file changed, 1 insertion(+), 6 deletions(-) > >>>> > >>>>diff --git a/drivers/soc/qcom/spm.c b/drivers/soc/qcom/spm.c > >>>>index b04b05a..199c085 100644 > >>>>--- a/drivers/soc/qcom/spm.c > >>>>+++ b/drivers/soc/qcom/spm.c > >>>>@@ -12,7 +12,6 @@ > >>>> * GNU General Public License for more details. > >>>> */ > >>>> > >>>>-#include > >>>> #include > >>>> #include > >>>> #include > >>>>@@ -378,8 +377,4 @@ static struct platform_driver spm_driver = { > >>>> .of_match_table = spm_match_table, > >>>> }, > >>>> }; > >>>>-module_platform_driver(spm_driver); > >>>>- > >>>>-MODULE_LICENSE("GPL v2"); > >>>>-MODULE_DESCRIPTION("SAW power controller driver"); > >>>>-MODULE_ALIAS("platform:saw"); > >>>>+builtin_platform_driver(spm_driver); > >>>>-- > >>>>2.1.4 > >>>>