* [PATCH] spi: pl022: remove unused ret and pins_state variables @ 2013-06-17 10:08 Fabio Baltieri 2013-06-17 10:35 ` Mark Brown 2013-06-17 11:22 ` Linus Walleij 0 siblings, 2 replies; 3+ messages in thread From: Fabio Baltieri @ 2013-06-17 10:08 UTC (permalink / raw) To: linux-arm-kernel Removes the warnings: drivers/spi/spi-pl022.c: In function 'pl022_suspend_resources': drivers/spi/spi-pl022.c:2322:24: warning: unused variable 'pins_state' [-Wunused-variable] drivers/spi/spi-pl022.c:2321:6: warning: unused variable 'ret' [-Wunused-variable] drivers/spi/spi-pl022.c: In function 'pl022_resume_resources': drivers/spi/spi-pl022.c:2334:6: warning: unused variable 'ret' [-Wunused-variable] introduced in: f1c9cf0 spi: pl022: use pinctrl PM helpers Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org> --- Hi Linus, this is just a quick build warning fixup patch found while playing with today's linux-next, should apply cleanly on pinctrl/for-next. Thanks, Fabio drivers/spi/spi-pl022.c | 5 ----- 1 file changed, 5 deletions(-) diff --git a/drivers/spi/spi-pl022.c b/drivers/spi/spi-pl022.c index 9652189..abef061 100644 --- a/drivers/spi/spi-pl022.c +++ b/drivers/spi/spi-pl022.c @@ -2318,9 +2318,6 @@ pl022_remove(struct amba_device *adev) */ static void pl022_suspend_resources(struct pl022 *pl022, bool runtime) { - int ret; - struct pinctrl_state *pins_state; - clk_disable(pl022->clk); if (runtime) @@ -2331,8 +2328,6 @@ static void pl022_suspend_resources(struct pl022 *pl022, bool runtime) static void pl022_resume_resources(struct pl022 *pl022, bool runtime) { - int ret; - /* First go to the default state */ pinctrl_pm_select_default_state(&pl022->adev->dev); if (!runtime) -- 1.8.2 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* [PATCH] spi: pl022: remove unused ret and pins_state variables 2013-06-17 10:08 [PATCH] spi: pl022: remove unused ret and pins_state variables Fabio Baltieri @ 2013-06-17 10:35 ` Mark Brown 2013-06-17 11:22 ` Linus Walleij 1 sibling, 0 replies; 3+ messages in thread From: Mark Brown @ 2013-06-17 10:35 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jun 17, 2013 at 12:08:15PM +0200, Fabio Baltieri wrote: > Removes the warnings: > > drivers/spi/spi-pl022.c: In function 'pl022_suspend_resources': > drivers/spi/spi-pl022.c:2322:24: warning: unused variable 'pins_state' [-Wunused-variable] > drivers/spi/spi-pl022.c:2321:6: warning: unused variable 'ret' [-Wunused-variable] > drivers/spi/spi-pl022.c: In function 'pl022_resume_resources': > drivers/spi/spi-pl022.c:2334:6: warning: unused variable 'ret' [-Wunused-variable] > > introduced in: > > f1c9cf0 spi: pl022: use pinctrl PM helpers Acked-by: Mark Brown <broonie@linaro.org> -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 836 bytes Desc: Digital signature URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20130617/78f4d9e4/attachment.sig> ^ permalink raw reply [flat|nested] 3+ messages in thread
* [PATCH] spi: pl022: remove unused ret and pins_state variables 2013-06-17 10:08 [PATCH] spi: pl022: remove unused ret and pins_state variables Fabio Baltieri 2013-06-17 10:35 ` Mark Brown @ 2013-06-17 11:22 ` Linus Walleij 1 sibling, 0 replies; 3+ messages in thread From: Linus Walleij @ 2013-06-17 11:22 UTC (permalink / raw) To: linux-arm-kernel On Mon, Jun 17, 2013 at 12:08 PM, Fabio Baltieri <fabio.baltieri@linaro.org> wrote: > Removes the warnings: > > drivers/spi/spi-pl022.c: In function 'pl022_suspend_resources': > drivers/spi/spi-pl022.c:2322:24: warning: unused variable 'pins_state' [-Wunused-variable] > drivers/spi/spi-pl022.c:2321:6: warning: unused variable 'ret' [-Wunused-variable] > drivers/spi/spi-pl022.c: In function 'pl022_resume_resources': > drivers/spi/spi-pl022.c:2334:6: warning: unused variable 'ret' [-Wunused-variable] > > introduced in: > > f1c9cf0 spi: pl022: use pinctrl PM helpers > > Signed-off-by: Fabio Baltieri <fabio.baltieri@linaro.org> How sloppy of me. Applied with Mark's ACK. Thanks! Linus Walleij ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-17 11:22 UTC | newest] Thread overview: 3+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2013-06-17 10:08 [PATCH] spi: pl022: remove unused ret and pins_state variables Fabio Baltieri 2013-06-17 10:35 ` Mark Brown 2013-06-17 11:22 ` Linus Walleij
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).