diff options
author | Robert Love <robert.w.love@intel.com> | 2009-02-27 13:55:55 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2009-03-10 10:05:35 -0400 |
commit | fc47ff6b1b27fb736f255ed8cd490356e0cd228f (patch) | |
tree | 6be2387ebfcff7a1a729061b35f1ea6fdd115a46 /drivers/scsi/fcoe/fcoe_sw.c | |
parent | b2ab99c9a300e572105d6db7f6efe0a4d1572167 (diff) |
[SCSI] libfc, fcoe: Remove unnecessary cast by removing inline wrapper
Comment from "Andrew Morton <akpm@linux-foundation.org>"
> +{
> + return (struct fcoe_softc *)lport_priv(lp);
unneeded/undesirable cast of void*. There are probably zillions of
instances of this - there always are.
This whole inline function was unnecessary. The FCoE layer knows
that it's data structure is stored in the lport private data, it
can just access it from lport_priv().
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/scsi/fcoe/fcoe_sw.c')
-rw-r--r-- | drivers/scsi/fcoe/fcoe_sw.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/fcoe/fcoe_sw.c b/drivers/scsi/fcoe/fcoe_sw.c index 007d1fbae808..f667dce1bbba 100644 --- a/drivers/scsi/fcoe/fcoe_sw.c +++ b/drivers/scsi/fcoe/fcoe_sw.c | |||
@@ -297,7 +297,7 @@ static int fcoe_sw_destroy(struct net_device *netdev) | |||
297 | if (!lp) | 297 | if (!lp) |
298 | return -ENODEV; | 298 | return -ENODEV; |
299 | 299 | ||
300 | fc = fcoe_softc(lp); | 300 | fc = lport_priv(lp); |
301 | 301 | ||
302 | /* Logout of the fabric */ | 302 | /* Logout of the fabric */ |
303 | fc_fabric_logoff(lp); | 303 | fc_fabric_logoff(lp); |