Skip to contentSkip to conversation

omap_hsmmc: Remove unused state variable

PATCH New Sergio Aguirre 2 participants 2 messages Jun 10, 2010 – Aug 27, 2010

Subsystems (2)
MMC, SD and SDIO OMAP hs MMC support
lore
Share & tools
lore

Sergio Aguirre

author
#0 lore raw

PATCH 1/1 omap_hsmmc: Remove unused state variable

This fixes the following warning:

drivers/mmc/host/omap_hsmmc.c: In function 'omap_hsmmc_suspend':
drivers/mmc/host/omap_hsmmc.c:2275: warning: unused variable 'state'

Introduced by commit ID:

commit 1a13f8fa76c880be41d6b1e6a2b44404bcbfdf9e
Author: Matt Fleming <[email protected]>
Date: Wed May 26 14:42:08 2010 -0700

mmc: remove the "state" argument to mmc_suspend_host()

The unique usage of this var was removed there, and missed removing the respective declaration aswell.

Signed-off-by Sergio Aguirre <[email protected]>
Acked-by Matt Fleming <[email protected]>
---
 drivers/mmc/host/omap_hsmmc.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)
drivers/mmc/host/omap_hsmmc.c +0 -1
index b032828..d50e917 100644
@@ -2272,7 +2272,6 @@ static int omap_hsmmc_suspend(struct device *dev)
 	int ret = 0;
 	struct platform_device *pdev = to_platform_device(dev);
 	struct omap_hsmmc_host *host = platform_get_drvdata(pdev);
-	pm_message_t state = PMSG_SUSPEND; /* unused by MMC core */
 
 	if (host && host->suspended)
 		return 0;
Signature · 1 line
1.6.3.3
#1 lore raw

PATCH RESEND 1/1 omap_hsmmc: Remove unused state variable

Hi,

Andrew, please take this one.

From: Sergio Aguirre <[email protected]>
Date: Thu, 10 Jun 2010 09:00:06 -0500

This fixes the following warning:

drivers/mmc/host/omap_hsmmc.c: In function 'omap_hsmmc_suspend':
drivers/mmc/host/omap_hsmmc.c:2275: warning: unused variable 'state'

Introduced by commit ID:

commit 1a13f8fa76c880be41d6b1e6a2b44404bcbfdf9e
Author: Matt Fleming <[email protected]>
Date: Wed May 26 14:42:08 2010 -0700

mmc: remove the "state" argument to mmc_suspend_host()

The unique usage of this var was removed there, and missed removing the respective declaration aswell.

Signed-off-by Sergio Aguirre <[email protected]>
Acked-by Matt Fleming <[email protected]>
---
 drivers/mmc/host/omap_hsmmc.c |    1 -
 1 files changed, 0 insertions(+), 1 deletions(-)
drivers/mmc/host/omap_hsmmc.c +0 -1
index 077bbdb..ba623d6 100644
@@ -2305,7 +2305,6 @@ static int omap_hsmmc_suspend(struct device *dev)
 	int ret = 0;
 	struct platform_device *pdev = to_platform_device(dev);
 	struct omap_hsmmc_host *host = platform_get_drvdata(pdev);
-	pm_message_t state = PMSG_SUSPEND; /* unused by MMC core */
 
 	if (host && host->suspended)
 		return 0;
Signature · 5 lines
1.7.0.1

-- 
Chris Ball   <[email protected]>   <http://printf.net/>
One Laptop Per Child