* [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).