diff options
| author | Yi Zou <yi.zou@intel.com> | 2013-05-18 02:28:17 -0400 |
|---|---|---|
| committer | Robert Love <robert.w.love@intel.com> | 2013-07-09 14:18:26 -0400 |
| commit | 418a8cfe69c3b6bd4598e9870b9f412e2c247214 (patch) | |
| tree | 73fd6a8652e5ebb10d202a534e4a870fee4d3c01 /drivers/scsi/fcoe/fcoe_transport.c | |
| parent | f07d46bbc9ba9a08b338dec7bb858977d4d822fb (diff) | |
fcoe: fix the link error status block sparse warnings
Both fcoe_fc_els_lesb and fc_els_lesb are in __be32 already, and both are
exactly the same size in bytes, with somewhat different member names to
reflect the fact the former is for Ethernet media the latter is for Fiber
Channel, so, remove conversion and use __be32 directly. This fixes the warning
from sparse check.
Signed-off-by: Yi Zou <yi.zou@intel.com>
Reported-by: Fengguang Wu <fengguang.wu@intel.com>
Tested-by: Jack Morgan <jack.morgan@intel.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Diffstat (limited to 'drivers/scsi/fcoe/fcoe_transport.c')
| -rw-r--r-- | drivers/scsi/fcoe/fcoe_transport.c | 22 |
1 files changed, 4 insertions, 18 deletions
diff --git a/drivers/scsi/fcoe/fcoe_transport.c b/drivers/scsi/fcoe/fcoe_transport.c index f3a5a53e8631..bedd42211727 100644 --- a/drivers/scsi/fcoe/fcoe_transport.c +++ b/drivers/scsi/fcoe/fcoe_transport.c | |||
| @@ -180,24 +180,10 @@ void fcoe_ctlr_get_lesb(struct fcoe_ctlr_device *ctlr_dev) | |||
| 180 | { | 180 | { |
| 181 | struct fcoe_ctlr *fip = fcoe_ctlr_device_priv(ctlr_dev); | 181 | struct fcoe_ctlr *fip = fcoe_ctlr_device_priv(ctlr_dev); |
| 182 | struct net_device *netdev = fcoe_get_netdev(fip->lp); | 182 | struct net_device *netdev = fcoe_get_netdev(fip->lp); |
| 183 | struct fcoe_fc_els_lesb *fcoe_lesb; | 183 | struct fc_els_lesb *fc_lesb; |
| 184 | struct fc_els_lesb fc_lesb; | 184 | |
| 185 | 185 | fc_lesb = (struct fc_els_lesb *)(&ctlr_dev->lesb); | |
| 186 | __fcoe_get_lesb(fip->lp, &fc_lesb, netdev); | 186 | __fcoe_get_lesb(fip->lp, fc_lesb, netdev); |
| 187 | fcoe_lesb = (struct fcoe_fc_els_lesb *)(&fc_lesb); | ||
| 188 | |||
| 189 | ctlr_dev->lesb.lesb_link_fail = | ||
| 190 | ntohl(fcoe_lesb->lesb_link_fail); | ||
| 191 | ctlr_dev->lesb.lesb_vlink_fail = | ||
| 192 | ntohl(fcoe_lesb->lesb_vlink_fail); | ||
| 193 | ctlr_dev->lesb.lesb_miss_fka = | ||
| 194 | ntohl(fcoe_lesb->lesb_miss_fka); | ||
| 195 | ctlr_dev->lesb.lesb_symb_err = | ||
| 196 | ntohl(fcoe_lesb->lesb_symb_err); | ||
| 197 | ctlr_dev->lesb.lesb_err_block = | ||
| 198 | ntohl(fcoe_lesb->lesb_err_block); | ||
| 199 | ctlr_dev->lesb.lesb_fcs_error = | ||
| 200 | ntohl(fcoe_lesb->lesb_fcs_error); | ||
| 201 | } | 187 | } |
| 202 | EXPORT_SYMBOL_GPL(fcoe_ctlr_get_lesb); | 188 | EXPORT_SYMBOL_GPL(fcoe_ctlr_get_lesb); |
| 203 | 189 | ||