summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRobert Love <robert.w.love@intel.com>2013-06-11 07:28:19 +0000
committerRobert Love <robert.w.love@intel.com>2013-07-09 11:18:47 -0700
commita2ceb1fbaa875172345f611a3d37c3662f7e5082 (patch)
tree0f9a67e501749d07729a8319edf6c7d5b841738e
parent3a2926054acf876313afaaddac00a8ad255f6d68 (diff)
downloadop-kernel-dev-a2ceb1fbaa875172345f611a3d37c3662f7e5082.zip
op-kernel-dev-a2ceb1fbaa875172345f611a3d37c3662f7e5082.tar.gz
libfcoe: Fix meaningless log statement
ctlr_dev was initialized to NULL, and never re-assigned. This caused the log statement to always report failure. This patch removes the unused variable and fixes the log statement to always report 'success', as that is what should be logged if the code reaches this point. Signed-off-by: Robert Love <robert.w.love@intel.com> Tested-by: Jack Morgan <jack.morgan@intel.com> Acked-by: Neil Horman <nhorman@tuxdriver.com>
-rw-r--r--drivers/scsi/fcoe/fcoe_transport.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/scsi/fcoe/fcoe_transport.c b/drivers/scsi/fcoe/fcoe_transport.c
index bedd422..f1ae5ed 100644
--- a/drivers/scsi/fcoe/fcoe_transport.c
+++ b/drivers/scsi/fcoe/fcoe_transport.c
@@ -707,7 +707,6 @@ ssize_t fcoe_ctlr_create_store(struct bus_type *bus,
{
struct net_device *netdev = NULL;
struct fcoe_transport *ft = NULL;
- struct fcoe_ctlr_device *ctlr_dev = NULL;
int rc = 0;
int err;
@@ -754,9 +753,8 @@ ssize_t fcoe_ctlr_create_store(struct bus_type *bus,
goto out_putdev;
}
- LIBFCOE_TRANSPORT_DBG("transport %s %s to create fcoe on %s.\n",
- ft->name, (ctlr_dev) ? "succeeded" : "failed",
- netdev->name);
+ LIBFCOE_TRANSPORT_DBG("transport %s succeeded to create fcoe on %s.\n",
+ ft->name, netdev->name);
out_putdev:
dev_put(netdev);
OpenPOWER on IntegriCloud