linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [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).