From: David S. Miller <davem@davemloft.net>
Date: Mon, 3 Nov 2008 08:04:24 +0000 (-0800)
Subject: SMC91x: Fix compilation on some platforms.
X-Git-Tag: v2.6.28-rc4~86^2~10
X-Git-Url: http://pilppa.com/gitweb/?a=commitdiff_plain;h=55c8eb6c8eaa5009eed1557b296da5d4ea9c369a;p=linux-2.6-omap-h63xx.git

SMC91x: Fix compilation on some platforms.

This reverts 51ac3beffd4afaea4350526cf01fe74aaff25eff ('SMC91x: delete
unused local variable "lp"') and adds __maybe_unused markers to these
(potentially) unused variables.

The issue is that in some configurations SMC_IO_SHIFT evaluates
to '(lp->io_shift)', but in some others it's plain '0'.

Based upon a build failure report from Manuel Lauss.

Signed-off-by: David S. Miller <davem@davemloft.net>
---

diff --git a/drivers/net/smc91x.c b/drivers/net/smc91x.c
index 6f9895d4e5b..fc80f250da3 100644
--- a/drivers/net/smc91x.c
+++ b/drivers/net/smc91x.c
@@ -2060,6 +2060,7 @@ static int smc_request_attrib(struct platform_device *pdev,
 			      struct net_device *ndev)
 {
 	struct resource * res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "smc91x-attrib");
+	struct smc_local *lp __maybe_unused = netdev_priv(ndev);
 
 	if (!res)
 		return 0;
@@ -2074,6 +2075,7 @@ static void smc_release_attrib(struct platform_device *pdev,
 			       struct net_device *ndev)
 {
 	struct resource * res = platform_get_resource_byname(pdev, IORESOURCE_MEM, "smc91x-attrib");
+	struct smc_local *lp __maybe_unused = netdev_priv(ndev);
 
 	if (res)
 		release_mem_region(res->start, ATTRIB_SIZE);