summaryrefslogtreecommitdiff
path: root/include
diff options
context:
space:
mode:
authorRobert Love <robert.w.love@intel.com>2009-02-27 21:55:55 +0300
committerJames Bottomley <James.Bottomley@HansenPartnership.com>2009-03-10 17:05:35 +0300
commitfc47ff6b1b27fb736f255ed8cd490356e0cd228f (patch)
tree6be2387ebfcff7a1a729061b35f1ea6fdd115a46 /include
parentb2ab99c9a300e572105d6db7f6efe0a4d1572167 (diff)
downloadlinux-fc47ff6b1b27fb736f255ed8cd490356e0cd228f.tar.xz
[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 'include')
-rw-r--r--include/scsi/libfcoe.h8
1 files changed, 1 insertions, 7 deletions
diff --git a/include/scsi/libfcoe.h b/include/scsi/libfcoe.h
index 89fdbb9a6a1b..f43d3833a7a8 100644
--- a/include/scsi/libfcoe.h
+++ b/include/scsi/libfcoe.h
@@ -58,16 +58,10 @@ struct fcoe_softc {
u8 address_mode;
};
-static inline struct fcoe_softc *fcoe_softc(
- const struct fc_lport *lp)
-{
- return (struct fcoe_softc *)lport_priv(lp);
-}
-
static inline struct net_device *fcoe_netdev(
const struct fc_lport *lp)
{
- return fcoe_softc(lp)->real_dev;
+ return ((struct fcoe_softc *)lport_priv(lp))->real_dev;
}
static inline struct fcoe_hdr *skb_fcoe_header(const struct sk_buff *skb)