diff options
author | Jörn Engel <joern@logfs.org> | 2012-05-09 13:04:23 -0400 |
---|---|---|
committer | Nicholas Bellinger <nab@linux-iscsi.org> | 2012-05-09 12:33:26 -0700 |
commit | 3269d930536f4dd9a431cfaca877c1c2cccb0fe6 (patch) | |
tree | e5dbb834d6269ea858cf77f4b423fc715bb588e1 | |
parent | 8ff5b325cd42d423e29827bfc6346b8d16b10b30 (diff) | |
download | lio-core-3269d930536f4dd9a431cfaca877c1c2cccb0fe6.tar.gz |
qla2xxx: use WWN_SIZE instead of hard-coded constants
Signed-off-by: Joern Engel <joern@logfs.org>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
-rw-r--r-- | drivers/scsi/qla2xxx/qla_target.c | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/drivers/scsi/qla2xxx/qla_target.c b/drivers/scsi/qla2xxx/qla_target.c index f4ca3c15dc1893..77e179f603600a 100644 --- a/drivers/scsi/qla2xxx/qla_target.c +++ b/drivers/scsi/qla2xxx/qla_target.c @@ -4358,17 +4358,17 @@ static void qlt_lport_dump(struct scsi_qla_host *vha, u64 wwpn, int i; pr_debug("qla2xxx HW vha->node_name: "); - for (i = 0; i < 8; i++) + for (i = 0; i < WWN_SIZE; i++) pr_debug("%02x ", vha->node_name[i]); pr_debug("\n"); pr_debug("qla2xxx HW vha->port_name: "); - for (i = 0; i < 8; i++) + for (i = 0; i < WWN_SIZE; i++) pr_debug("%02x ", vha->port_name[i]); pr_debug("\n"); pr_debug("qla2xxx passed configfs WWPN: "); put_unaligned_be64(wwpn, b); - for (i = 0; i < 8; i++) + for (i = 0; i < WWN_SIZE; i++) pr_debug("%02x ", b[i]); pr_debug("\n"); } @@ -4390,7 +4390,7 @@ int qlt_lport_register(struct qla_tgt_func_tmpl *qla_tgt_ops, u64 wwpn, struct Scsi_Host *host; unsigned long flags; int rc; - u8 b[8]; + u8 b[WWN_SIZE]; mutex_lock(&qla_tgt_mutex); list_for_each_entry(tgt, &qla_tgt_glist, tgt_list_entry) { @@ -4424,7 +4424,7 @@ int qlt_lport_register(struct qla_tgt_func_tmpl *qla_tgt_ops, u64 wwpn, } qlt_lport_dump(vha, wwpn, b); - if (memcmp(vha->port_name, b, 8)) { + if (memcmp(vha->port_name, b, WWN_SIZE)) { scsi_host_put(host); continue; } |